Skip to content

Use proxy from container libs#2816

Merged
mtrmac merged 2 commits intocontainers:mainfrom
giuseppe:use-proxy-from-container-libs
Mar 19, 2026
Merged

Use proxy from container libs#2816
mtrmac merged 2 commits intocontainers:mainfrom
giuseppe:use-proxy-from-container-libs

Conversation

@giuseppe
Copy link
Copy Markdown
Member

@giuseppe giuseppe commented Mar 3, 2026

skopeo changes for containers/container-libs#677

@giuseppe giuseppe marked this pull request as draft March 3, 2026 09:54
@packit-as-a-service
Copy link
Copy Markdown

Ephemeral COPR build failed. @containers/packit-build please check.

Copy link
Copy Markdown
Collaborator

@cgwalters cgwalters left a comment

Choose a reason for hiding this comment

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

Thanks for working on this!!

Comment thread cmd/skopeo/proxy.go.backup Outdated
@giuseppe giuseppe force-pushed the use-proxy-from-container-libs branch from c31b9a2 to f7722ac Compare March 3, 2026 13:51
@giuseppe giuseppe changed the title [RFC] Use proxy from container libs Use proxy from container libs Mar 3, 2026
@giuseppe giuseppe force-pushed the use-proxy-from-container-libs branch 2 times, most recently from d97421a to eba4f23 Compare March 3, 2026 20:33
@TomSweeneyRedHat
Copy link
Copy Markdown
Member

That's some serious code consolidation. The change LGTM in general.

@giuseppe giuseppe force-pushed the use-proxy-from-container-libs branch 2 times, most recently from 62155ea to 150d4af Compare March 3, 2026 21:08
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the use-proxy-from-container-libs branch from 150d4af to 7f09762 Compare March 19, 2026 16:39
@giuseppe giuseppe marked this pull request as ready for review March 19, 2026 16:39
@giuseppe giuseppe requested a review from cgwalters March 19, 2026 16:40
Copy link
Copy Markdown
Contributor

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

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

ACK

Comment thread cmd/skopeo/proxy_windows.go Outdated
@giuseppe giuseppe force-pushed the use-proxy-from-container-libs branch from 7f09762 to 3d580eb Compare March 19, 2026 19:11
Comment thread cmd/skopeo/proxy_windows.go
Comment thread cmd/skopeo/proxy_windows.go Outdated
Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the use-proxy-from-container-libs branch from 3d580eb to 2eb170a Compare March 19, 2026 19:27
Copy link
Copy Markdown
Contributor

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

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

Thanks!

@mtrmac mtrmac merged commit 529952e into containers:main Mar 19, 2026
39 checks passed
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