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.

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

versioned catalogd https interface implementation phase 1 #427

Closed
Tracked by #239
grokspawn opened this issue Oct 7, 2024 · 0 comments
Closed
Tracked by #239

versioned catalogd https interface implementation phase 1 #427

grokspawn opened this issue Oct 7, 2024 · 0 comments
Assignees
Labels
epic/catalogd-api-audit v1.0 Issues related to the initial stable release of OLMv1
Milestone

Comments

@grokspawn
Copy link
Contributor

grokspawn commented Oct 7, 2024

implements phase 1 of the CatalogD expandable interface.

change catalog-specific URL from full path to API endpoint ref

This phase just moves from status.contentURL to status.baseURL, and will be used to provide reference to the catalog-specific API instead of the full path to JSONLines-formatted content, plus tests and documentation.

@grokspawn grokspawn added this to OLM v1 Oct 7, 2024
@grokspawn grokspawn added epic/catalogd-api-audit v1.0 Issues related to the initial stable release of OLMv1 labels Oct 7, 2024
@grokspawn grokspawn added this to the v1.0.0 milestone Oct 7, 2024
@grokspawn grokspawn changed the title versioned catalogd https interface implementation versioned catalogd https interface implementation phase 1 Oct 10, 2024
grokspawn added a commit to grokspawn/catalogd that referenced this issue Oct 10, 2024
solves operator-framework#427 and implements phase 1 of the CatalogD expandable interface.
This phase just moves from `status.contentURL` to `status.baseURL`, and
will be used to provide reference to the catalog-specific API instead of
the full path to JSONLines-formatted content, plus tests and
documentation.

Signed-off-by: Jordan Keister <[email protected]>
grokspawn added a commit to grokspawn/catalogd that referenced this issue Oct 10, 2024
solves operator-framework#427 and implements phase 1 of the CatalogD expandable interface.
This phase just moves from `status.contentURL` to `status.baseURL`, and
will be used to provide reference to the catalog-specific API instead of
the full path to JSONLines-formatted content, plus tests and
documentation.

Signed-off-by: Jordan Keister <[email protected]>
grokspawn added a commit to grokspawn/catalogd that referenced this issue Oct 11, 2024
solves operator-framework#427 and implements phase 1 of the CatalogD expandable interface.
This phase just moves from `status.contentURL` to `status.baseURL`, and
will be used to provide reference to the catalog-specific API instead of
the full path to JSONLines-formatted content, plus tests and
documentation.

Signed-off-by: Jordan Keister <[email protected]>
grokspawn added a commit to grokspawn/catalogd that referenced this issue Oct 11, 2024
solves operator-framework#427 and implements phase 1 of the CatalogD expandable interface.
This phase just moves from `status.contentURL` to `status.baseURL`, and
will be used to provide reference to the catalog-specific API instead of
the full path to JSONLines-formatted content, plus tests and
documentation.

Signed-off-by: Jordan Keister <[email protected]>
@grokspawn grokspawn self-assigned this Oct 14, 2024
@grokspawn grokspawn moved this to Implementing in OLM v1 Oct 14, 2024
grokspawn added a commit to grokspawn/catalogd that referenced this issue Oct 17, 2024
solves operator-framework#427 and implements phase 1 of the CatalogD expandable interface.
This phase just moves from `status.contentURL` to `status.baseURL`, and
will be used to provide reference to the catalog-specific API instead of
the full path to JSONLines-formatted content, plus tests and
documentation.

Signed-off-by: Jordan Keister <[email protected]>
github-merge-queue bot pushed a commit that referenced this issue Oct 18, 2024
* change catalog-specific URL from full path to API endpoint ref

solves #427 and implements phase 1 of the CatalogD expandable interface.
This phase just moves from `status.contentURL` to `status.baseURL`, and
will be used to provide reference to the catalog-specific API instead of
the full path to JSONLines-formatted content, plus tests and
documentation.

Signed-off-by: Jordan Keister <[email protected]>

* sticking to catalog-centric reference, and updated type

Signed-off-by: Jordan Keister <[email protected]>

* remove some magic numbers

Signed-off-by: Jordan Keister <[email protected]>

* review reconciliation

Signed-off-by: Jordan Keister <[email protected]>

---------

Signed-off-by: Jordan Keister <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic/catalogd-api-audit v1.0 Issues related to the initial stable release of OLMv1
Projects
Status: Done
Development

No branches or pull requests

1 participant