Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixes #327: OCI Registry for use with path-based-routing #313

Merged
merged 2 commits into from
Sep 16, 2024
Merged

Fixes #327: OCI Registry for use with path-based-routing #313

merged 2 commits into from
Sep 16, 2024

Conversation

jessesanford
Copy link
Contributor

@jessesanford jessesanford commented Jun 19, 2024 •
edited
Loading

Fixes #327 by setting up nginx rewrite rules on the ingress so that the generic OCI registry discovery urls can be found when using path based routing.

@jessesanford jessesanford added the enhancement New feature or request label Jun 19, 2024
@jessesanford jessesanford requested review from nimakaviani, nabuskey and a team June 19, 2024 05:43
@jessesanford
Copy link
Contributor Author

@csantanapr 🍻

@cmoulliard
Copy link
Contributor

Can you add a description to this PR plus an issue as this is not clear for the reviewers to figure out what is the purpose of this PR ?

Remark: PR should be rejected if they don't include an issue's link and/or description

@jessesanford jessesanford force-pushed the path-based-oci-registry branch from 3eabbb8 to 3395b08 Compare June 21, 2024 20:28
@jessesanford
Copy link
Contributor Author

Can you add a description to this PR plus an issue as this is not clear for the reviewers to figure out what is the purpose of this PR ?

Remark: PR should be rejected if they don't include an issue's link and/or description

I agree, this is intentionally a quick draft PR to help with a need that @csantanapr had. I will create an issue and appropriate documentation before submitting the actual PR.

@jessesanford jessesanford force-pushed the path-based-oci-registry branch from 3395b08 to d622140 Compare September 13, 2024 23:06
@jessesanford jessesanford force-pushed the path-based-oci-registry branch from d622140 to d101929 Compare September 13, 2024 23:12
@jessesanford jessesanford changed the title working login, push and pull from workstation w/ use-path-routing=true Fix OCI Registry for use with path-based-routing Sep 13, 2024
@jessesanford jessesanford force-pushed the path-based-oci-registry branch from a0432df to d101929 Compare September 14, 2024 04:10
@jessesanford
use larger proxy-body-size
17ff7c2 
Signed-off-by: Jesse Sanford <[email protected]>
@jessesanford jessesanford force-pushed the path-based-oci-registry branch from 4def81f to 17ff7c2 Compare September 14, 2024 04:17
@jessesanford jessesanford changed the title Fix OCI Registry for use with path-based-routing Fixes #327: OCI Registry for use with path-based-routing Sep 14, 2024
@jessesanford jessesanford marked this pull request as ready for review September 14, 2024 04:19
@cmoulliard
Copy link
Contributor

I will create an issue and appropriate documentation before submitting the actual PR.

Don't forget please to add the ticket and description

@jessesanford jessesanford merged commit 5c47cf7 into cnoe-io:main Sep 16, 2024
7 checks passed
@nabuskey nabuskey mentioned this pull request Oct 15, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blakeromano blakeromano blakeromano approved these changes

@nimakaviani nimakaviani Awaiting requested review from nimakaviani

@nabuskey nabuskey Awaiting requested review from nabuskey

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature: OCI Registry with path based routing
4 participants
@jessesanford @cmoulliard @nabuskey @blakeromano