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

Set image create #441

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Set image create #441

wants to merge 2 commits into from

Conversation

cpuguy83
Copy link
Member

@cpuguy83 cpuguy83 commented Nov 23, 2024

Set image config created time

Before this was just using whatever created time was set on the base
image config which was incorrect.

Example using the docker buildx bake runc-jammy-container command:

Before:

docker image ls | grep runc
runc                                          jammy             8d1d42e074f3   25 minutes ago   228MB

After:

docker image ls | grep runc
runc                                          jammy             03c24620c8fa   5 seconds ago    228MB

Note: how the created time before was 25mins ago (b/c apparently the base
jammy image was recently updated, I've seen this be 2 weeks) compared to
after its 5 seconds ago.

This just had the wrong logic so it was outputing the container rootfs
to the filesystem instead of to the image store.

Signed-off-by: Brian Goff <[email protected]>
Before this was just using whatever created time was set on the base
image config which was incorrect.

Example using the `docker buildx bake runc-jammy-container` command:

Before:

docker image ls | grep runc
runc                                          jammy             8d1d42e074f3   25 minutes ago   228MB

After:

  docker image ls | grep runc
  runc                                          jammy             03c24620c8fa   5 seconds ago    228MB

**Note**: how the created time before was 25mins ago (b/c apparently the base
jammy image was recently updated, I've seen this be 2 weeks) compared to
after its 5 seconds ago.

Signed-off-by: Brian Goff <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant