Skip to content

DeleteImage(): always delete mapped top layers#1209

Merged
rhatdan merged 1 commit intocontainers:mainfrom
nalind:always-delete-mapped-layers
Apr 14, 2022
Merged

DeleteImage(): always delete mapped top layers#1209
rhatdan merged 1 commit intocontainers:mainfrom
nalind:always-delete-mapped-layers

Conversation

@nalind
Copy link
Copy Markdown
Member

@nalind nalind commented Apr 14, 2022

When deleting an image, always delete an image's mapped top layers, even if when the unmapped top layer is shared with other images and must therefore not be deleted.

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Apr 14, 2022

Your desciption is a little contradictory?

@nalind
Copy link
Copy Markdown
Member Author

nalind commented Apr 14, 2022

The main distinction is mapped vs unmapped?

When deleting an image, always delete an image's mapped top layers, even
when the unmapped top layer is shared with other images and must
therefore not be deleted.  Mapped top layers aren't shared with other
images, so we never need to preserve them.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
@nalind nalind force-pushed the always-delete-mapped-layers branch from 39e9a25 to ff62c5b Compare April 14, 2022 15:10
@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Apr 14, 2022

LGTM

1 similar comment
@baude
Copy link
Copy Markdown
Member

baude commented Apr 14, 2022

LGTM

Copy link
Copy Markdown
Member

@jwhonce jwhonce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

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.

4 participants