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

Kill all docker containers as well #639

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

howardjohn
Copy link
Member

For istio/test-infra#1988

We should kill running docker containers at the end of the job. Pruning
doesn't stop running containers.

For istio/test-infra#1988

We should kill running docker containers at the end of the job. Pruning
doesn't stop running containers.
@howardjohn howardjohn requested a review from a team as a code owner December 18, 2019 01:23
@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Dec 18, 2019
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 18, 2019
@howardjohn
Copy link
Member Author

/retest

1 similar comment
@howardjohn
Copy link
Member Author

/retest

@istio-testing istio-testing merged commit 202f6b6 into istio:master Dec 18, 2019
Shuanglu pushed a commit to Shuanglu/istio-tools that referenced this pull request Jun 30, 2022
For istio/test-infra#1988

We should kill running docker containers at the end of the job. Pruning
doesn't stop running containers.
Shuanglu pushed a commit to Shuanglu/istio-tools that referenced this pull request Jul 6, 2022
For istio/test-infra#1988

We should kill running docker containers at the end of the job. Pruning
doesn't stop running containers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants