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

Add support to configure times to suspend chaos. #50

Closed
wants to merge 4 commits into from

Conversation

twildeboer
Copy link

@twildeboer twildeboer commented Dec 8, 2017

This implements the feature as briefly discussed in #48 with some minor name changes.
Please Note:

  • I made an assumption on the version number changes: 0.6.2
  • Addition of a helm chart example
  • Dockerfile change required to include TZ database

This has been tested in our environment and appears to work as advertised.

@linki
Copy link
Owner

linki commented Jan 8, 2018

@twildeboer Thanks, I'll try to look at this as soon as I can and compare it to @klautcomputing's version. For the sake of a small PR can you split out the chart related stuff in a separate PR?

@klautcomputing
Copy link
Contributor

FYI: I don't think my code is mergeable as is and I don't intend to put more time into it. @linki

@twildeboer
Copy link
Author

@linki - Will do.

@twildeboer
Copy link
Author

Breaking this up into two PRs as requested.

@twildeboer twildeboer closed this Jan 8, 2018
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.

None yet

3 participants