Skip to content

Please, document environment variables #189

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
imaia opened this issue Jul 10, 2017 · 6 comments
Closed

Please, document environment variables #189

imaia opened this issue Jul 10, 2017 · 6 comments

Comments

@imaia
Copy link

imaia commented Jul 10, 2017

Please, document the variables below:

  • MONGO_INITDB_DATABASE
  • MONGO_INITDB_ROOT_USERNAME
  • MONGO_INITDB_ROOT_PASSWORD
@tianon
Copy link
Member

tianon commented Aug 1, 2017 via email

@imaia
Copy link
Author

imaia commented Aug 1, 2017

@tianon how can we help?

@yosifkit
Copy link
Member

yosifkit commented Aug 2, 2017

@imaia thanks for any help! Any documentation would go in the content.md file over here.

I think the best option we thought to improve the reliability of these variables was to add a small yaml parser to the image to parse the MongoDB config file and use it with our cli flag parsing to ensure we properly detect ssl usage and other conflicting config. I don't recall what @tianon had found when searching for a small yaml parser. We just haven't been able to get back to give the MongoDB image more attention.

Here is the section that we have to ensure certain config flags are set for mongod. It all is a little hacky, and I'd love a better way to do it. The biggest annoyance is mongod handling of config; it just fails if config options exist twice (two flags or one flag and one in the config file); it can't handle allowSSL if there is no ssl config, or disabled sslMode when there are ssl certs.

Have any ideas where you would like to help?

@geekscrapy
Copy link

Has the actual OP issue been resolved yet? Still can't find doco for setting the auth in environment variables

@tianon
Copy link
Member

tianon commented Feb 15, 2018

Parsing the config file was added in #225, which was the main driver for considering this functionality to still be experimental, so we're definitely ready to document this.

It looks like there's been a stab taken at doing so in docker-library/docs#1127, but we haven't had a chance to review it in depth yet.

@tianon
Copy link
Member

tianon commented Apr 20, 2018

Closing in favor of docker-library/docs#1127 (since this is really a documentation issue, not an image issue). 👍

@tianon tianon closed this as completed Apr 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants