Get rid of chef_gem #39
Conversation
chef_gem fails because build-essential isn't installed at compile time. The Chef guys are recommending that complex things like replicaset and mongos become LWRPs, which is probably in progress but this fixes the chef_gem issue quickly.
| @@ -21,7 +21,15 @@ | |||
| node.set['mongodb']['cluster_name'] = node['mongodb']['cluster_name'] | |||
|
|
|||
| include_recipe 'mongodb::install' | |||
| include_recipe 'mongodb::mongo_gem' | |||
|
|
|||
| ruby_block 'chef_gem_at_converge_time' do· | |||
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
|
wt… another dot @josephholsten all PRs are branches within this repo, you can checkout pr/370 and commit there to avoid duplicated PRs. I'm closing the original PR |
|
Aside from the slight duplication anything else stopping a merge on this? |
|
I thought the middle dot should break everything, I don't understand why CI raises no issue. The code duplication is minor and should not be blocking. |
remove dot
|
I removed the dot, once travis gives us the OK I will merge. |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
fix the middle dot in #21