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

Remove references to internal CEP/SCF stuff #17

Merged
merged 2 commits into from
Nov 4, 2015
Merged

Remove references to internal CEP/SCF stuff #17

merged 2 commits into from
Nov 4, 2015

Conversation

solarkennedy
Copy link
Contributor

This changes references to CEP/SCF and puts soa-configs docs in it's own file so it has room to grow and is easier to link to. cc @tomelm

@@ -8,7 +8,7 @@ in yelpsoa-configs.
Monitoring.yaml
===============

`yelpsoa-configs (CEP 319) <https://docs.google.com/a/yelp.com/document/d/1ZBg5ykniRU30UXj4YcsKfmmnuegQbtR2VuqCAIGi-50/view#>`_ houses a file called monitoring.yaml whose keys are exposed through service_configuration_lib. You can use it to specify information relevant to monitoring for your service, such as who is responsible for the service, who to contact when it has problems, and even what to look for to ensure it stays up.
The ``soa-configs`` house a file called monitoring.yaml whose keys are exposed through service_configuration_lib. You can use it to specify information relevant to monitoring for your service, such as who is responsible for the service, who to contact when it has problems, and even what to look for to ensure it stays up.
Copy link
Contributor

Choose a reason for hiding this comment

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

link to s_c_l github (I believe this exists now / will be public when the rest of this goes public)

@mrtyler
Copy link
Contributor

mrtyler commented Nov 4, 2015

fix'n'ship

solarkennedy added a commit that referenced this pull request Nov 4, 2015
Remove references to internal CEP/SCF stuff
@solarkennedy solarkennedy merged commit 120767b into master Nov 4, 2015
@solarkennedy solarkennedy deleted the no_cep branch November 4, 2015 21:20
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

Successfully merging this pull request may close these issues.

2 participants