Skip to content

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

Merged
jessesanford merged 2 commits intocnoe-io:mainfrom
adsk-pset-de-hackathon:path-based-oci-registry
Sep 16, 2024
Merged

Fixes #327: OCI Registry for use with path-based-routing#313
jessesanford merged 2 commits intocnoe-io:mainfrom
adsk-pset-de-hackathon:path-based-oci-registry

Conversation

@jessesanford
Copy link
Copy Markdown
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 a team, nabuskey and nimakaviani June 19, 2024 05:43
@jessesanford
Copy link
Copy Markdown
Contributor Author

jessesanford commented Jun 19, 2024

@csantanapr 🍻

@cmoulliard
Copy link
Copy Markdown
Contributor

cmoulliard commented Jun 19, 2024

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

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

jessesanford commented Jun 25, 2024

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
Fixes cnoe-io#327 OCI Registry working with path-base-routing
d101929 
Signed-off-by: Jesse Sanford <108698+jessesanford@users.noreply.github.com>
@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 <108698+jessesanford@users.noreply.github.com>
@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
Copy Markdown
Contributor

cmoulliard commented Sep 16, 2024

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
@nabuskey nabuskey mentioned this pull request Oct 15, 2024
Closed
@jessesanford jessesanford deleted the path-based-oci-registry branch September 25, 2025 07:14
@jessesanford jessesanford restored the path-based-oci-registry branch September 25, 2025 15:07
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