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 #179

Closed
josephholsten opened this issue Nov 27, 2013 · 3 comments
Closed

Comments

@josephholsten
Copy link

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!
@jamesonjlee
Copy link

well, ideally default would install from wherever but otherwise do nothing else.

@josephholsten
Copy link
Author

Sounds reasonable, most cookbooks install a minimal client from their default, and the standard package is the only way to get the normal mongo client.

http://josephholsten.com

On Nov 26, 2013, at 18:01, Jameson [email protected] wrote:

well, ideally default would install from wherever but otherwise do nothing else.


Reply to this email directly or view it on GitHub.

@josephholsten
Copy link
Author

Moved to sous-chefs/sc-mongodb#75

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

2 participants