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

feature: add container-index flag for container chaos #36

Merged
merged 1 commit into from
Sep 21, 2020

Conversation

xcaspar
Copy link
Member

@xcaspar xcaspar commented Sep 21, 2020

Signed-off-by: xcaspar x.caspar@gmail.com

Describe what this PR does / why we need it

Support dynamic container names. Use container-index to specify the container.

Does this pull request fix one issue?

Describe how you did it

Add --container-index flag starting from 0.

Describe how to verify it

blade c k8s container-cpu fullload --names frontend-6c887c56c8-4g7sh --container-index 0 --namespace default --kubeconfig ~/.kube/config

Special notes for reviews

Signed-off-by: xcaspar <x.caspar@gmail.com>
@xcaspar xcaspar added the type/enhancement New feature or request label Sep 21, 2020
@xcaspar xcaspar added this to the v0.7.0 milestone Sep 21, 2020
@xcaspar xcaspar requested a review from tiny-x September 21, 2020 07:06
@tiny-x tiny-x merged commit e2f83d4 into master Sep 21, 2020
@xcaspar xcaspar deleted the add-containerindex branch September 21, 2020 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants