NE-2780: Add v1.2.3 bundle to all FBC catalogs#508
Conversation
📝 WalkthroughWalkthroughThis PR updates OLM catalog templates and generated catalog YAMLs for Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Add `v1.2.3` bundle to all FBC `catalog-template.yaml` files (v4.12–v4.22) in `stable-v1` and `stable-v1.2` channels using `registry.redhat.io`. Regenerate `catalog.yaml` files using `make generate-catalog`. Co-Authored-By: Claude
external-dns-operator.v1.2.3 stage bundle to all FBC catalogsv1.2.3 bundle to all FBC catalogs
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@catalog/v4.22/catalog-template.yaml`:
- Around line 144-146: The new bundle record is not aligned with the surrounding
entries list, so update the catalog template entry under entries by indenting
the external-dns-operator.v1.2.3 bundle fields consistently with the other list
items. Use the existing YAML structure in catalog-template.yaml as the reference
and ensure the image, name, and schema fields stay grouped beneath the same
entries item so the list remains valid.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 7780d7d7-d82c-4215-9e25-287f839611e8
📒 Files selected for processing (22)
catalog/v4.12/catalog-template.yamlcatalog/v4.12/catalog.yamlcatalog/v4.13/catalog-template.yamlcatalog/v4.13/catalog.yamlcatalog/v4.14/catalog-template.yamlcatalog/v4.14/catalog.yamlcatalog/v4.15/catalog-template.yamlcatalog/v4.15/catalog.yamlcatalog/v4.16/catalog-template.yamlcatalog/v4.16/catalog.yamlcatalog/v4.17/catalog-template.yamlcatalog/v4.17/catalog.yamlcatalog/v4.18/catalog-template.yamlcatalog/v4.18/catalog.yamlcatalog/v4.19/catalog-template.yamlcatalog/v4.19/catalog.yamlcatalog/v4.20/catalog-template.yamlcatalog/v4.20/catalog.yamlcatalog/v4.21/catalog-template.yamlcatalog/v4.21/catalog.yamlcatalog/v4.22/catalog-template.yamlcatalog/v4.22/catalog.yaml
🚧 Files skipped from review as they are similar to previous changes (19)
- catalog/v4.14/catalog-template.yaml
- catalog/v4.21/catalog-template.yaml
- catalog/v4.16/catalog-template.yaml
- catalog/v4.17/catalog-template.yaml
- catalog/v4.18/catalog-template.yaml
- catalog/v4.19/catalog-template.yaml
- catalog/v4.13/catalog-template.yaml
- catalog/v4.15/catalog.yaml
- catalog/v4.15/catalog-template.yaml
- catalog/v4.14/catalog.yaml
- catalog/v4.20/catalog-template.yaml
- catalog/v4.16/catalog.yaml
- catalog/v4.13/catalog.yaml
- catalog/v4.12/catalog.yaml
- catalog/v4.19/catalog.yaml
- catalog/v4.18/catalog.yaml
- catalog/v4.17/catalog.yaml
- catalog/v4.21/catalog.yaml
- catalog/v4.20/catalog.yaml
v1.2.3 bundle to all FBC catalogsv1.2.3 bundle to all FBC catalogs
|
@alebedev87: This pull request references NE-2780 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
must-gather hiccup, e2e passed. /override ci/prow/e2e-azure-operator |
|
/assign @davidesalerno |
|
@alebedev87: Overrode contexts on behalf of alebedev87: ci/prow/e2e-azure-operator DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@alebedev87: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidesalerno The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/lgtm |
Summary
Add
external-dns-operator.v1.2.3bundle to all FBCcatalog-template.yamlfiles (v4.12–v4.22) instable-v1andstable-v1.2channels usingregistry.redhat.io. Regeneratecatalog.yamlfiles usingmake generate-catalog.