Skip to content

Conversation

@Avi-Robusta
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jan 27, 2026

Docker image ready for cf60256 (built in 4m 12s)

⚠️ Warning: does not support ARM (ARM images are built on release only - not on every PR)

Use this tag to pull the image for testing.

📋 Copy commands

⚠️ Temporary images are deleted after 30 days. Copy to a permanent registry before using them:

gcloud auth configure-docker us-central1-docker.pkg.dev
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:cf60256
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:cf60256 me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:cf60256
docker push me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:cf60256

Patch 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:cf60256

@coderabbitai
Copy link

coderabbitai bot commented Jan 27, 2026

Walkthrough

The holmes Helm chart dependency version in the Robusta Helm configuration is updated from 0.18.5 to 0.19.0 under the enableHolmesGPT condition, with no other changes to the chart structure.

Changes

Cohort / File(s) Change Summary
Dependency Version Update
helm/robusta/Chart.yaml
Holmes chart dependency version bumped from 0.18.5 to 0.19.0 under the enableHolmesGPT conditional dependency block

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • arikalon1
  • RoiGlinik
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: bumping the holmes dependency version to 0.19.0 in the helm chart configuration.
Description check ✅ Passed No pull request description was provided by the author; however, the title is sufficiently descriptive about the changeset and the referenced issue number provides context.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bump-holmes-sp

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Avi-Robusta Avi-Robusta enabled auto-merge (squash) January 27, 2026 12:42
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@helm/robusta/Chart.yaml`:
- Line 18: Chart.yaml currently pins holmes to an unavailable stable release
"0.19.0"; update the version declaration in Chart.yaml to either a published
stable holmes release (e.g., a known released version) or explicitly pin to the
pre-release "0.19.0-alpha" to match published artifacts and ensure Helm
values/CRDs compatibility; edit the line containing the version field (version:
0.19.0) to the chosen valid version string and verify related values/CRDs and
charts reference the same pinned version.

@Avi-Robusta Avi-Robusta merged commit 7c38202 into master Jan 27, 2026
7 checks passed
@Avi-Robusta Avi-Robusta deleted the bump-holmes-sp branch January 27, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants