-
Notifications
You must be signed in to change notification settings - Fork 48
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
fixes #15058 - enable mongo auth #126
Conversation
5825192
to
50a2ef5
Compare
ACK |
@@ -46,6 +46,10 @@ | |||
# | |||
# $repo_export_dir:: Directory to create for repository exports | |||
# | |||
# $pulp_db_username:: Username for mongo | |||
# | |||
# $pulp_db_password:: Password for mongo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We aren't doing this kind of exposure for the Candlepin database, maybe we should. Also, if you change this to just database we won't have to update it when Pulp switches off mongo.
@ehelms Renamed mongo to database in the docs |
On the other PR's, too |
ACK |
I don't think this works on el6, there is a new el6 bats failure:
additionally,
|
I'm not sure this is related, but... This fails on CentOS7 (centos7-devel box):
This works, however (with quotes):
I'm just executing these commands on the centos7-devel box after running |
The quotes would probably be removed when it shows you the command but they should be there when puppet's actually executing it. I think. It's quirky. I've seen this error intermittently, and @beav saw some other Mongo-related issues. I wonder if we just end up turning this back off all together. It only works on one OS, and apparently not very well. Mongo is going away eventually anyway. |
Companion PR's:
Test box: