Skip to content

Improve handling and documentation of various transport reference corner cases#2767

Merged
mtrmac merged 6 commits intocontainers:mainfrom
mtrmac:transport-corner-cases
Mar 11, 2025
Merged

Improve handling and documentation of various transport reference corner cases#2767
mtrmac merged 6 commits intocontainers:mainfrom
mtrmac:transport-corner-cases

Conversation

@mtrmac
Copy link
Copy Markdown
Collaborator

@mtrmac mtrmac commented Mar 7, 2025

See individual commit messages for details.

(Minimally) fixes #2679 .

Copy link
Copy Markdown
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

mtrmac added 6 commits March 11, 2025 18:10
Should not change (test) behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
They can't be used anyway (destinations require a reference,
and sources panic), so fail cleanly.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Reject NewIndexReference with a "missing" index value of -1;
callers can use the other NewReference instead.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
…andled specially

Should not change behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Should not change behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
The same thing is already said a few lines below.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@mtrmac mtrmac force-pushed the transport-corner-cases branch from d61dcc0 to 67490ae Compare March 11, 2025 17:12
@mtrmac mtrmac merged commit 6f1abf0 into containers:main Mar 11, 2025
10 checks passed
@mtrmac mtrmac deleted the transport-corner-cases branch March 11, 2025 17:43
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.

oci/layout.NewReference (and others?) doesn’t document the image=="" special case

2 participants