Fix CI image mismatch and remove stale docs#774
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 41 minutes and 59 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThis PR bumps version references from 0.5.1/0.0.0 to 0.6.0 across deployment manifests, OLM artifacts, documentation, build scripts, and CI workflows, pinning container image tags to Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Signed-off-by: Jason Madigan <jason@jasonmadigan.com>
9133505 to
b523de4
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@bundle/manifests/mcp-gateway.clusterserviceversion.yaml`:
- Around line 8-15: The bundle manifests are outdated (bundle output doesn't
match generated artifacts); run the project bundling step and commit the
regenerated files: run `make bundle` to regenerate the bundle (which will update
fields such as containerImage, name/version and CRDs in the bundle manifests),
review the changed files under bundle/manifests (e.g.,
mcp-gateway.clusterserviceversion.yaml) to ensure the containerImage and
name/version entries are correct, then add and commit those updated bundle files
to the branch so CI validation passes.
In `@config/deploy/olm/catalogsource.yaml`:
- Line 7: The catalog image is pinned to v0.6.0; on main we should track the
moving target. Update the catalog image reference (the YAML key `image` value
`ghcr.io/kuadrant/mcp-controller-catalog:v0.6.0`) to use the `:latest` tag
instead of the semver tag so the OLM catalog on main follows current main
builds.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: c280fc0c-a962-44b4-96f2-79cf6fb62e07
📒 Files selected for processing (16)
.github/workflows/conformance.yamlMakefilebundle/manifests/mcp-gateway.clusterserviceversion.yamlcharts/sample_local_helm_setup.shconfig/deploy/olm/catalogsource.yamlconfig/manifests/bases/mcp-gateway.clusterserviceversion.yamlconfig/mcp-gateway/components/controller/deployment-controller.yamlconfig/mcp-gateway/overlays/ci/kustomization.yamlconfig/mcp-system/deployment-broker.yamlconfig/mcp-system/deployment-controller.yamlconfig/openshift/deploy_openshift.shdocs/guides/how-to-install-and-configure.mddocs/guides/isolated-gateway-deployment.mddocs/guides/olm-install.mddocs/guides/quick-start.mdscripts/quick-start.sh
…S.md Signed-off-by: Jason Madigan <jason@jasonmadigan.com>
|
CC: @maleck13 removed those dead header refs. |
Summary