Skip to content

Conversation

@rata
Copy link
Contributor

@rata rata commented Aug 24, 2023

This PR upstream added the spec for the mountExtensions feature field:
opencontainers/runtime-spec#1219

This commit just implements that and updates the OCI max version implemented to the one currently used in the spec (an unreleased version).

It is not clear if in the future, the version of unreleased specs will be changed to something else:
opencontainers/runtime-spec#1221

But this is what is currently accepted.


Besides the tests here, I've also verified this against containerd integration tests.

cc @giuseppe

This PR upstream added the spec for the mountExtensions feature field:
	opencontainers/runtime-spec#1219

This commit just implements that and updates the OCI max version
implemented to the one currently used in the spec (an unreleased
version).

It is not clear if in the future, the version of unreleased specs will
be changed to something else:
	opencontainers/runtime-spec#1221

But this is what is currently accepted.

Signed-off-by: Rodrigo Campos <rodrigoca@microsoft.com>
Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

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

LGTM

@giuseppe giuseppe merged commit 6369207 into containers:main Aug 24, 2023
@rata rata deleted the rata/features-expose-idmap branch August 24, 2023 12:32
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.

2 participants