sous-chefs / sc-mongodb

Development repository for the sc-mongodb cookbook
https://supermarket.chef.io/cookbooks/sc-mongodb
Apache License 2.0
75 stars 91 forks source link

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

Open josephholsten opened 9 years ago

josephholsten commented 9 years ago

From https://github.com/edelight/chef-mongodb/issues/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:

shortdudey123 commented 7 years ago

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

josephholsten commented 7 years ago

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 commented 7 years ago

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

github-actions[bot] commented 4 years ago

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.