Skip to content
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

Updating mongodb.conf template to make it work on RedHat derivatives as well #168

Merged
merged 3 commits into from
Apr 2, 2021

Conversation

silveraid
Copy link
Contributor

Hi,

The role currently deploys a mongodb.conf file which does not work on CentOS or any other RedHat derivatives, nor on RedHat itself.

In this PR mongodb.conf's template have been updated with matching configuration from the original rpm config file to ensure that the role works on RedHat (and derivatives) as well.

https://github.com/mongodb/mongo/blob/master/rpm/mongod.conf#L21

Thanks,
Frank

@CLAassistant
Copy link

CLAassistant commented Mar 31, 2021

CLA assistant check
All committers have signed the CLA.

@silveraid
Copy link
Contributor Author

Travis is just horrible, I will try to restart the job again later :-(

@silveraid
Copy link
Contributor Author

Also added the fix for the mongodb test case on Ubuntu 20.04 so that the CI can pass the change.

@malcyon malcyon self-requested a review April 2, 2021 13:15
@malcyon
Copy link
Contributor

malcyon commented Apr 2, 2021

Thanks for cleaning that up, @silveraid!

@malcyon malcyon merged commit 9d0f8c4 into Graylog2:master Apr 2, 2021
jrunu pushed a commit to jrunu/graylog-ansible-role that referenced this pull request Apr 16, 2021
…as well (Graylog2#168)

* Updating mongodb.conf template to work with CentOS

* Applying recommeded mongodb thread configuration for RedHat derivatives

* Fix failing mongodb test on ubuntu 20.04
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

Successfully merging this pull request may close these issues.

3 participants