Skip to content

[release/1.7 backport] reference: deprecate SplitObject#10397

Merged
AkihiroSuda merged 1 commit intocontainerd:release/1.7from
thaJeztah:1.7_backport_deprecate_reference_SplitObject
Jun 29, 2024
Merged

[release/1.7 backport] reference: deprecate SplitObject#10397
AkihiroSuda merged 1 commit intocontainerd:release/1.7from
thaJeztah:1.7_backport_deprecate_reference_SplitObject

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

pkg/reference: deprecate SplitObject

The behavior of this function is quite counter-intuitive, as it preserves
the delimiter in the result, and its use for external consumers would be
very limited.

Spec.Digest no longer uses this function, and it appears that BuildKit is
currently the only (publicly visible) external consumer of it.

This patch deprecates the function.

@k8s-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@thaJeztah thaJeztah changed the title [release/1.7 backport] pkg/reference: deprecate SplitObject [release/1.7 backport]reference: deprecate SplitObject Jun 27, 2024
@thaJeztah thaJeztah self-assigned this Jun 27, 2024
The behavior of this function is quite counter-intuitive, as it preserves
the delimiter in the result, and its use for external consumers would be
very limited.

Spec.Digest no longer uses this function, and it appears that BuildKit is
currently the only (publicly visible) external consumer of it.

This patch deprecates the function.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit fec33aa)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the 1.7_backport_deprecate_reference_SplitObject branch from 8380593 to dba5357 Compare June 28, 2024 19:14
@thaJeztah thaJeztah marked this pull request as ready for review June 28, 2024 19:15
@thaJeztah thaJeztah changed the title [release/1.7 backport]reference: deprecate SplitObject [release/1.7 backport] reference: deprecate SplitObject Jun 28, 2024
@thaJeztah thaJeztah closed this Jun 28, 2024
@thaJeztah thaJeztah reopened this Jun 28, 2024
@thaJeztah thaJeztah requested a review from samuelkarp June 28, 2024 23:36
@AkihiroSuda AkihiroSuda merged commit aea977f into containerd:release/1.7 Jun 29, 2024
@thaJeztah thaJeztah deleted the 1.7_backport_deprecate_reference_SplitObject branch June 29, 2024 13:25
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.

4 participants