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.

Sign up for GitHub

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

Jump to bottom

purge minio test container at the end of tests #993

Merged
merged 1 commit into from
Dec 23, 2022
Merged

purge minio test container at the end of tests #993

merged 1 commit into from
Dec 23, 2022

Conversation

darkowlzz
Copy link
Contributor

@darkowlzz darkowlzz commented Dec 22, 2022 •
edited
Loading

Purge minio test container at the end of tests.
Also, add container option to auto-remove a container when stopped, in case a container is left behind due to some reason.

Without it minio containers are left behind:

CONTAINER ID   IMAGE                                      COMMAND                  CREATED        STATUS        PORTS                                                                                      NAMES
e83053caebc5   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   46 hours ago   Up 46 hours   0.0.0.0:49167->9000/tcp, :::49166->9000/tcp, 0.0.0.0:49166->9001/tcp, :::49165->9001/tcp   suspicious_noether
b44cf9e4082d   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   46 hours ago   Up 46 hours   0.0.0.0:49165->9000/tcp, :::49164->9000/tcp, 0.0.0.0:49164->9001/tcp, :::49163->9001/tcp   silly_hermann
7708ff92e24f   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   3 days ago     Up 3 days     0.0.0.0:49163->9000/tcp, :::49162->9000/tcp, 0.0.0.0:49161->9001/tcp, :::49161->9001/tcp   fervent_mendeleev
bc41c28ed6e5   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   3 days ago     Up 3 days     0.0.0.0:49160->9000/tcp, :::49160->9000/tcp, 0.0.0.0:49159->9001/tcp, :::49159->9001/tcp   gracious_bartik
f99d91caa08e   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   3 days ago     Up 3 days     0.0.0.0:49158->9000/tcp, :::49158->9000/tcp, 0.0.0.0:49157->9001/tcp, :::49157->9001/tcp   nervous_spence
24a4e7586a76   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   3 days ago     Up 3 days     0.0.0.0:49156->9000/tcp, :::49156->9000/tcp, 0.0.0.0:49155->9001/tcp, :::49155->9001/tcp   vibrant_pare
16917945c2d2   minio/minio:RELEASE.2022-12-12T19-27-27Z   "/usr/bin/docker-ent…"   3 days ago     Up 3 days     0.0.0.0:49154->9000/tcp, :::49154->9000/tcp, 0.0.0.0:49153->9001/tcp, :::49153->9001/tcp   serene_meitner

Follow-up of #981

@darkowlzz
purge minio test container at the end of tests
3a17461 
Also, add container option to auto-remove a container when stopped, in
case a container is left behind due to some reason.

Signed-off-by: Sunny <darkowlzz@protonmail.com>
@darkowlzz darkowlzz added the area/testing Testing related issues and pull requests label Dec 22, 2022
@darkowlzz darkowlzz merged commit c8feb3a into main Dec 23, 2022
@darkowlzz darkowlzz deleted the cleanup-minio-test-container branch December 23, 2022 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hiddeco hiddeco hiddeco approved these changes

Assignees
No one assigned
Labels
area/testing Testing related issues and pull requests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@darkowlzz @hiddeco