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

Mongo 3.4 and (partial) Xenial support #32

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vesse
Copy link

@vesse vesse commented Dec 28, 2016

This pull request would:

  • Add MongoDB 3.4 support with configurable signing_key
  • Use ansible_distribution_release in APT sources, or fallback to the previously hard-coded values

MongoDB Xenial repositories now provide the needed systemd unit at least for 3.2 and 3.4 so there's no need to build one manually.

The service in Ansible system module seems to enable the systemd service and start it properly when run on real server, but when running Ubuntu 16.04 in Docker the setup fails with no service or tool found for: mongod which is probably because Ansible shows upstart in ansible_service_mgr when run on image built from Ubuntu official 16.04 base image.

I did deploy a working MongoDB 3.4 on Xenial with the currently release version with this in my group_vars:

mongodb_version: "3.4.1"
mongodb_repository_config:
  v3.4:
    repository: deb http://repo.mongodb.org/apt/ubuntu xenial/mongodb-org/3.4 multiverse
    package: mongodb-org
mongodb_version_config:
  v3.4:
    daemon: mongod
    config_file: mongod.conf

which is what this pull request should produce.

mongodb_transparent_hugepage_disable does not work with this pull request on systemd because it creates an upstart file and I didn't do anything to it. And as mentioned there's a Docker problem, hopefully you have better insight in it than I do.

Partial support because mongodb_transparent_hugepage_disable creates
an upstart init file that does not work with systemd
@pigulla
Copy link

pigulla commented Jan 27, 2017

Any chance of this getting merged?

@wangsha
Copy link

wangsha commented Feb 13, 2017

+1

@pigulla
Copy link

pigulla commented Feb 13, 2017

Paging @klen

@bradcypert
Copy link

I, too, would like to see this merged.

@JMLX42
Copy link

JMLX42 commented Feb 10, 2018

I would like to see this merged too plz.

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.

5 participants