Skip to content

Conversation

@thaJeztah
Copy link
Member

The value was calculated, but due to 0af2962 changing to a non-pointer, the value was not written back to the resulting slice.

Before this patch:

docker pull nginx:alpine
docker pull alpine

docker system df -v
Images space usage:

REPOSITORY   TAG       IMAGE ID       CREATED       SIZE      SHARED SIZE   UNIQUE SIZE   CONTAINERS
nginx        alpine    cbad6347cca2   4 weeks ago   53.4MB    N/A           N/A           0
alpine       latest    171e65262c80   7 weeks ago   8.51MB    N/A           N/A           0

With this patch:

docker system df -v
Images space usage:

REPOSITORY   TAG       IMAGE ID       CREATED       SIZE      SHARED SIZE   UNIQUE SIZE   CONTAINERS
nginx        alpine    cbad6347cca2   4 weeks ago   53.4MB    8.512MB       44.91MB       0
alpine       latest    171e65262c80   7 weeks ago   8.51MB    8.512MB       0B            0

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

Fix `docker system df` showing `N/A` for shared size and unique size when using graph-drivers as storage.

- A picture of a cute animal (not mandatory but encouraged)

The value was calculated, but due to 0af2962
changing to a non-pointer, the value was not written back to the resulting
slice.

Before this patch:

    docker pull nginx:alpine
    docker pull alpine

    docker system df -v
    Images space usage:

    REPOSITORY   TAG       IMAGE ID       CREATED       SIZE      SHARED SIZE   UNIQUE SIZE   CONTAINERS
    nginx        alpine    cbad6347cca2   4 weeks ago   53.4MB    N/A           N/A           0
    alpine       latest    171e65262c80   7 weeks ago   8.51MB    N/A           N/A           0

With this patch:

    docker system df -v
    Images space usage:

    REPOSITORY   TAG       IMAGE ID       CREATED       SIZE      SHARED SIZE   UNIQUE SIZE   CONTAINERS
    nginx        alpine    cbad6347cca2   4 weeks ago   53.4MB    8.512MB       44.91MB       0
    alpine       latest    171e65262c80   7 weeks ago   8.51MB    8.512MB       0B            0

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 29.1.2 milestone Dec 1, 2025
@github-actions github-actions bot added the area/daemon Core Engine label Dec 1, 2025
@thaJeztah thaJeztah marked this pull request as ready for review December 1, 2025 17:25
@cpuguy83 cpuguy83 merged commit a1836eb into moby:master Dec 1, 2025
278 of 281 checks passed
@thaJeztah thaJeztah deleted the fix_df_shared_usage branch December 2, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants