chore: Bump version for automation-flavors#351#1887
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe Helm chart adds an OpenShift cluster manager API token Secret, injects it into destroy workflow containers, and updates ChangesOpenShift token wiring
Chart metadata
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
A single node development cluster (infra-pr-1887) was allocated in production infra for this PR. CI will attempt to deploy 🔌 You can connect to this cluster with: 🛠️ And pull infractl from the deployed dev infra-server with: 🔓 You must go to to export an . Your token from the prod infra instance will not work with dev environments. 🚲 You can then use the dev infra instance e.g.: Further Development☕ If you make changes, you can commit and push and CI will take care of updating the development cluster. 🚀 If you only modify configuration (chart/infra-server/configuration) or templates (chart/infra-server/{static,templates}), you can get a faster update with: LogsLogs for the development infra depending on your @redhat.com authuser: Or: |
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 `@chart/infra-server/static/workflow-openshift-4.yaml`:
- Around line 121-125: Add OPENSHIFT_CLUSTER_MANAGER_API_TOKEN to the main
workflow’s destroy container environment, using the same secretKeyRef name
openshift-cluster-manager-token and key OPENSHIFT_CLUSTER_MANAGER_API_TOKEN
already used by create. Keep the existing destroy environment entries unchanged.
🪄 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: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 7c879648-b5d4-4a46-8290-bc2a27dfbbff
📒 Files selected for processing (3)
chart/infra-server/static/janitor-delete-openshift-4.yamlchart/infra-server/static/workflow-openshift-4.yamlchart/infra-server/templates/openshift-4/secrets.yaml
9da3234 to
4278739
Compare
Auto-generated: Bump automation-flavors version for https://github.com/stackrox/automation-flavors/pull/351.