Skip to content

adding rabbit related things for khronos#164

Merged
anandkumarpatel merged 6 commits intomasterfrom
khronos-workers
Oct 14, 2015
Merged

adding rabbit related things for khronos#164
anandkumarpatel merged 6 commits intomasterfrom
khronos-workers

Conversation

@bkendall
Copy link
Copy Markdown
Contributor

@bkendall bkendall commented Oct 6, 2015

I can add cron to this PR if you would like, but I want to verify that it works fine before I start regularly hammering it.

@bkendall
Copy link
Copy Markdown
Contributor Author

bkendall commented Oct 6, 2015

wait on this khronos pr, but this will be needed then!

Comment thread ansible/group_vars/alpha-khronos.yml Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

?

@anandkumarpatel
Copy link
Copy Markdown

  • deploy to beta before merge

@anandkumarpatel
Copy link
Copy Markdown

code looks good, deploy to beta before merging

@bkendall
Copy link
Copy Markdown
Contributor Author

bkendall commented Oct 7, 2015

ohhh yeah. that worked like a charm. I'm so happy :)

@cflynn07
Copy link
Copy Markdown
Contributor

cflynn07 commented Oct 9, 2015

lgtm. Good that we have default command set to json-filtered output

@anandkumarpatel
Copy link
Copy Markdown

for the cron since we are not doing it inside khronos itself, you will need to also create some type of monitoring mechanism (maybe datadog if event was not seen after x time)

anandkumarpatel pushed a commit that referenced this pull request Oct 14, 2015
adding rabbit related things for khronos
@anandkumarpatel anandkumarpatel merged commit 5972b9c into master Oct 14, 2015
@anandkumarpatel anandkumarpatel deleted the khronos-workers branch October 14, 2015 17:43
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.

3 participants