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

[breaking change] Stop creating a mongodb_instance in mongodb::default #75

Open
josephholsten opened this issue Jul 17, 2015 · 4 comments
Milestone

Comments

@josephholsten
Copy link
Contributor

From edelight/chef-mongodb#179

the logic of https://github.com/edelight/chef-mongodb/blob/master/recipes/default.rb#L42 is absolutely maddening in it's odd dependence on knowing exactly what recipes are in the run list. In particular, it doesn't play nice with wrapper cookbooks which may set up replicasets via include_recipe, not through node or role run lists. Don't get me started on mongos's search.

I propose:

  • renaming helper recipes to begin with an underscore, including: install -> _install_package, 10gen_repo -> _mongodb_repo,
  • keep 'standard' usage recipes top level: configserver, mongos, replicaset, shard, and move the default instance into its own mongod recipe
  • also move mongo_gem -> ruby to match opscode's mysql, xml, &c cookbooks
  • have default not do anything!
@shortdudey123
Copy link
Contributor

shortdudey123 commented Apr 12, 2017

@josephholsten counter point: i want to setup a single mongo instance without much config
thoughts?

@josephholsten
Copy link
Contributor Author

then I'd prefer the doc said something like:

## The simplest thing that could possibly work

To spin up a minimal mongodb service with reasonable defaults, add `recipe[mongodb::mongod]` to your run list.

@shortdudey123
Copy link
Contributor

That works, going to move this to v2 to avoid making v1 too complicated

@shortdudey123 shortdudey123 modified the milestones: 2.0.0, 1.0.0 Apr 12, 2017
@github-actions
Copy link

Marking stale due to inactivity. Remove stale label or comment or this will be closed in 7 days. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, Sous-Chefs.

@github-actions github-actions bot added the Stale label Sep 30, 2020
@xorimabot xorimabot removed the Stale label Sep 30, 2020
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