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

[23.0 backport] volumes: prune: add --all / -a option #4229

Merged
merged 2 commits into from
Apr 26, 2023

Conversation

thaJeztah
Copy link
Member

These tests were deliberately producing errors as part of the test, but
printing those errors could be confusing / make it more difficult to find
actual test-failures.

Before this patch:

    === RUN   TestVolumeCreateErrors
    Error: conflicting options: either specify --name or provide positional arg, not both
    Error: "create" requires at most 1 argument.
    See 'create --help'.

    Usage:  create [OPTIONS] [VOLUME] [flags]

    Create a volume
    Error: error creating volume
    --- PASS: TestVolumeCreateErrors (0.00s)
    PASS

With this patch applied:

    === RUN   TestVolumeCreateErrors
    --- PASS: TestVolumeCreateErrors (0.00s)
    PASS

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit db827d5)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 0dec5d2)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Apr 26, 2023

Codecov Report

Merging #4229 (86e79b5) into 23.0 (ced769f) will increase coverage by 0.00%.
The diff coverage is 71.42%.

Additional details and impacted files
@@           Coverage Diff           @@
##             23.0    #4229   +/-   ##
=======================================
  Coverage   58.83%   58.84%           
=======================================
  Files         572      572           
  Lines       49610    49638   +28     
=======================================
+ Hits        29188    29208   +20     
- Misses      18648    18656    +8     
  Partials     1774     1774           

@thaJeztah thaJeztah merged commit 776388c into docker:23.0 Apr 26, 2023
@thaJeztah thaJeztah deleted the 23.0_backport_volumes_prune_all branch April 26, 2023 11:01
archlinux-github pushed a commit to archlinux/infrastructure that referenced this pull request Nov 25, 2023
"docker system prune --volumes" does no longer prune named volumes in
Docker 23.0[1][2], so use "docker volume prune --all"[3] for pruning
named volumes.

[1] docker/cli#4028
[2] moby/moby#44259
[3] docker/cli#4229
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants