ROB-310 Upgrade Holmes GPT dependency to version 0.33.0#2101
Conversation
Bump the holmes chart dependency from 0.32.0 to 0.33.0 in helm/robusta/Chart.yaml and rebuild the chart lock file. https://claude.ai/code/session_01PvMeJxmNLjUCpJAxNBHFED
WalkthroughThe ChangesHolmes Helm Dependency Bump
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
✅ Docker image ready for
Use this tag to pull the image for testing. 📋 Copy commandsgcloud auth configure-docker us-central1-docker.pkg.dev
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:f1a7220
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:f1a7220 me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:f1a7220
docker push me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:f1a7220Patch Helm values in one line: helm upgrade --install robusta robusta/robusta \
--reuse-values \
--set runner.image=me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:f1a7220 |
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 `@helm/robusta/Chart.yaml`:
- Line 18: The holmes dependency version is set to 0.33.0 in the Chart.yaml
file, but this version does not exist in the robusta-charts repository and will
cause Helm dependency resolution to fail. Change the holmes dependency version
from 0.33.0 to 0.32.0, which is the most recent documented release available. If
0.32.0 is not compatible with your requirements, use an earlier version such as
0.31.0 or 0.31.1.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: cd56d6f5-0b97-4fd6-a36d-808b5032aaea
⛔ Files ignored due to path filters (1)
helm/robusta/Chart.lockis excluded by!**/*.lock
📒 Files selected for processing (3)
helm/robusta/Chart.yamlhelm/robusta/charts/holmes-0.32.0.tgzhelm/robusta/charts/holmes-0.33.0.tgz
Summary
This pull request updates the Holmes GPT Helm chart dependency from version 0.32.0 to 0.33.0.
Changes
Chart.yamlChart.lock) with new dependency version and checksumsNotes
This is a dependency version bump that brings in the latest Holmes GPT improvements and fixes. The change is controlled by the existing
enableHolmesGPTcondition in the Helm values.https://claude.ai/code/session_01PvMeJxmNLjUCpJAxNBHFED