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

WIP TESTING #1540

Closed
Closed

Conversation

camilamacedo86
Copy link
Contributor

@camilamacedo86 camilamacedo86 commented Jan 2, 2025

No description provided.

joelanford and others added 30 commits June 27, 2023 16:45
…ork#109)

Ideally, we would enforce restricted. However, some catalog
images may not be compatible with restriced enforcement.

This is another motivation for us to treat catalog images
differently from runnable images. PSA compatibility of
catalog images should never be a consideration because we
only need to extract static files from them. Actually
running them should never be necessary.

Signed-off-by: Joe Lanford <[email protected]>
…#112)

to ensure we only list resources belonging to the 
`Catalog` resource currently being reconciled.
Signed-off-by: Bryce Palmer <[email protected]>
Fix link and misspelling

The operator-controller docs link was 404ing, now it links to the
correct folder.

Signed-off-by: Justin Kulikauskas <[email protected]>
Signed-off-by: Andy Goldstein <[email protected]>
We do not want to fail the job if codecov fails
to upload the report due to rate limiting.

Signed-off-by: Mikalai Radchuk <[email protected]>
Signed-off-by: Andy Goldstein <[email protected]>
Signed-off-by: Andy Goldstein <[email protected]>
Marking the CatalogMetadataAPI feature gate as Deprecated before introducing the HTTP server as a default mechanism to serve catalog contents

Signed-off-by: Rashmi Gottipati <[email protected]>
* add metrics to catalogd http server

that can be used for calculating the Apdex Score
and assess the health of the http server that is
serving catalog contents to clients

Signed-off-by: Bryce Palmer <[email protected]>

* quick fixes from review comments

Signed-off-by: Bryce Palmer <[email protected]>

* rename package from server --> metrics

Signed-off-by: Bryce Palmer <[email protected]>

* rename package from server --> metrics

Signed-off-by: Bryce Palmer <[email protected]>

---------

Signed-off-by: Bryce Palmer <[email protected]>
…work#166)

* (docs): Add docs on fetching contents via HTTP server

Signed-off-by: Bryce Palmer <[email protected]>

* add curl examples

Signed-off-by: Bryce Palmer <[email protected]>

---------

Signed-off-by: Bryce Palmer <[email protected]>
Signed-off-by: Bryce Palmer <[email protected]>
…erator-framework#169)

* (refactor): Remove deprecated CatalogMetadata and associated code

Signed-off-by: Bryce Palmer <[email protected]>

* go mod tidy

Signed-off-by: Bryce Palmer <[email protected]>

---------

Signed-off-by: Bryce Palmer <[email protected]>
Add release guide

Signed-off-by: Catherine Chan-Tse <[email protected]>
@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 2, 2025
… adapt them for catalogd

- Rename the workflows to have the catalogd prefix
- Ensure that go.mod is obtained from catalogd
- Ensure that workdir is the catalogd path
- Fix the paths to use it from catalogd
- Fix the directory path
The check fails because the CRD does not exist in the new repo.
We are improving the CI and makefile target to skip when the orginal
does not exist.
@camilamacedo86 camilamacedo86 force-pushed the add-catalogd branch 2 times, most recently from ec9814c to ba38627 Compare January 2, 2025 18:13
@camilamacedo86 camilamacedo86 changed the title ⚠️ Creating Monorepo : Move catalogd to operator-controller repository WIP (Doing next steps / follow ups )⚠️ Creating Monorepo : Move catalogd to operator-controller repository Jan 2, 2025
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 2, 2025
- We are finding and replace the catalogd imports
- We removed the go.mod and sum.mod from catalogd dir
- Also, we remove the specific GitHub action that verify the go version from catalogd since it is done for operator-controller
- By last we updated all catalogd GitHub actions to use the go.mod from ROOT DIR
Update the operator-controller Makefile's crd-ref-docs task to generate catalogd docs from the repo
@camilamacedo86 camilamacedo86 force-pushed the add-catalogd branch 2 times, most recently from 8785f6e to f755727 Compare January 3, 2025 09:42
@camilamacedo86 camilamacedo86 marked this pull request as draft January 3, 2025 11:19
@camilamacedo86 camilamacedo86 changed the title WIP (Doing next steps / follow ups )⚠️ Creating Monorepo : Move catalogd to operator-controller repository WIP TESTING Jan 3, 2025
@camilamacedo86
Copy link
Contributor Author

Close in favor of #1542

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.