Allow to deploy Zeebe workers with zbchaos #247
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
29 tasks
lenaschoenburg
approved these changes
Nov 22, 2022
| // k8Deployments holds our static k8 manifests, which are copied with the go:embed directive | ||
| // | ||
| //go:embed manifests/* | ||
| var k8Deployments embed.FS |
ChrisKujawa
added a commit
that referenced
this pull request
Nov 22, 2022
With #247 we are able to deploy zeebe workers with zbchaos. With this PR we are able to restart and terminate workers, either one or all existing ones. In this PR I also iterated over the terminate an restart commands and realized that gateway restart was missing. I added this. Furthermore a new subcommand is added on terminate and restart for the broker, this makes it more understandable what is restarted, instead of using the root restart/terminate command for that. I plan to do some more refactoring like extracting the duplicated code and reusing logic, but I will do this in a separate PR. -------- **Example:** Terminate one: ``` $ ./zbchaos terminate worker -v Connecting to zell-chaos Running experiment in self-managed environment. Terminated worker-99cd57f8b-hmhb5 ``` Terminate all: ``` $ ./zbchaos terminate worker -v --all Connecting to zell-chaos Running experiment in self-managed environment. Terminated worker-99cd57f8b-lfbck Terminated worker-99cd57f8b-vjb9l ``` Restart one ``` $ ./zbchaos restart worker Restart worker-99cd57f8b-4ght7 ``` Restart all ``` $ ./zbchaos restart worker --all Restart worker-99cd57f8b-4ght7 Restart worker-99cd57f8b-7t2d9 Restart worker-99cd57f8b-kmrhv ``` New restart sub-commands: ``` $ ./zbchaos restart broker Deleted zell-chaos-zeebe-2 $ ./zbchaos restart gateway Restarted zell-chaos-zeebe-gateway-5f5c7cb8d6-9529r $ ./zbchaos restart worker Restart worker-99cd57f8b-cr6bs ``` New terminate sub-commands: ``` $ ./zbchaos terminate broker Terminated zell-chaos-zeebe-0 $ ./zbchaos terminate gateway Terminated zell-chaos-zeebe-gateway-5f5c7cb8d6-6jlxn $ ./zbchaos terminate worker Terminated worker-99cd57f8b-55tt2 ```
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add new sub-command to deploy Zeebe workers into a Zeebe cluster, which can be used for several chaos experiments. This allows completing process instances etc.
closes #235
Example:
Non-verbose: Deploy worker:
Verbose: Deploy worker
$ ./zbchaos deploy worker -v Connecting to zell-chaos Running experiment in self-managed environment. Deploy worker deployment to the current namespace: zell-chaos Worker successfully deployed to the current namespace: zell-chaosError when deployment already exists: