Skip to content

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.

@twildeboer twildeboer force-pushed the support_suspending_chaos branch from 1526a1f to 39c68fe Compare December 8, 2017 20:08
@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.

3 participants