Skip to content

Cleanup docker cache clear script

username-removed-609185 requested to merge vilhelmen/gitlab-ci-multi-runner:master into master

What does this MR do?

Cleanup/tweak the docker cache cleanup script. Filters containers by status so it doesn't try to delete new or running containers. Containers are removed in batches of 10 at a time.

Why was this MR needed?

Existing filtering is clunky (4 greps(!)) and does not filter by status.

Are there points in the code the reviewer needs to double check?

Not really, it's pretty minor and rather straightforward.

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

#151 (closed) but only tangentially. It would still be best if volume cleanup was built in.

Merge request reports