Skip to content

[release/1.7] Update metadata snapshotter to lease on already exists#10198

Merged
mxpv merged 2 commits intocontainerd:release/1.7from
dmcgowan:1.7-metadata-add-lease-exists
May 11, 2024
Merged

[release/1.7] Update metadata snapshotter to lease on already exists#10198
mxpv merged 2 commits intocontainerd:release/1.7from
dmcgowan:1.7-metadata-add-lease-exists

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan commented May 9, 2024

Currently the metadata snapshotter is not consistently adding keys to a
lease when already exists is returned. When a lease is provided, any
already exists errors should add the relevant key to the lease. It is
not expected that clients must explicitly lease a key after calling
Prepare/Commit.

Backport of #10187

dmcgowan added 2 commits May 9, 2024 14:21
Currently the metadata snapshotter is not consistently adding keys to a
lease when already exists is returned. When a lease is provided, any
already exists errors should add the relevant key to the lease. It is
not expected that clients must explicitly lease a key after calling
Prepare/Commit.

Signed-off-by: Derek McGowan <[email protected]>
(cherry picked from commit c7fb8a9)
Signed-off-by: Derek McGowan <[email protected]>
Signed-off-by: Derek McGowan <[email protected]>
(cherry picked from commit 8c6183d)
Signed-off-by: Derek McGowan <[email protected]>
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@mxpv mxpv merged commit 91c65aa into containerd:release/1.7 May 11, 2024
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.

6 participants