Skip to content
This repository has been archived by the owner on Jul 11, 2020. It is now read-only.

Add info about roles usage for monitoring settings #7

Merged
merged 2 commits into from
Feb 14, 2015
Merged

Add info about roles usage for monitoring settings #7

merged 2 commits into from
Feb 14, 2015

Conversation

evverx
Copy link
Contributor

@evverx evverx commented Jan 19, 2015

No description provided.

Verified

This commit was signed with the committer’s verified signature.
lukekarrys Luke Karrys
@@ -12,7 +12,7 @@ This repository contains chef artifacts used to deploy all the different pieces
To run chef on all nodes (staging, common, production) run: `knife ssh "*:*" "sudo chef-client" -x $(whoami)`.

#### Roles
We use roles as minimally as possible. All top-level functionality should be in the appropriate role-specific cookbook (i.e. rubygems-app). We only use roles for easing the use of search.
We use roles as minimally as possible. All top-level functionality should be in the appropriate role-specific cookbook (i.e. rubygems-app). We only use roles for easing the use of search and for monitoring [settings](https://github.com/rubygems/rubygems-infrastructure/blob/master/cookbooks/rubygems-sensu/recipes/default.rb#L25).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"settings" probably isn't the right word here. How about "configuration"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@skottler , yes, 'configuration' would be better.

Verified

This commit was signed with the committer’s verified signature.
lukekarrys Luke Karrys
skottler pushed a commit that referenced this pull request Feb 14, 2015

Verified

This commit was signed with the committer’s verified signature.
lukekarrys Luke Karrys
Add info about roles usage for monitoring settings
@skottler skottler merged commit 182c59a into rubygems:master Feb 14, 2015
@skottler
Copy link
Member

Thanks!

@evverx evverx deleted the patch-1 branch February 14, 2015 23:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants