Skip to content

ROB-1283 Bump Holmes chart from 0.40.0 to 0.41.0 - #2170

Merged
Avi-Robusta merged 1 commit into
masterfrom
holmes-0.41.0-alpha
Sep 8, 2026
Merged

ROB-1283 Bump Holmes chart from 0.40.0 to 0.41.0#2170
Avi-Robusta merged 1 commit into
masterfrom
holmes-0.41.0-alpha

Conversation

@Avi-Robusta

Copy link
Copy Markdown
Contributor

Bumps the Holmes chart dependency from 0.40.0 to the released 0.41.0.

What changed

  • Chart.yaml / Chart.lock: holmes dependency 0.40.0 -> 0.41.0
  • charts/holmes-0.41.0.tgz pulled via helm dependency update, stale holmes-0.40.0.tgz removed

Testing

Deployed to avi-test-cluster2 across three releases (robusta/default, robusta-2/monitoring-2, robusta-3/monitoring) — all runner and robustadev/holmes pods came up Running.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Docker image ready for 06f9f8e (built in 4m 17s)

⚠️ 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:06f9f8e
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:06f9f8e me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:06f9f8e
docker push me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:06f9f8e

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:06f9f8e

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 363d640e-14db-4409-9403-ead8ddcaa2e5

📥 Commits

Reviewing files that changed from the base of the PR and between 483a5d2 and faa2465.

⛔ Files ignored due to path filters (1)
  • helm/robusta/Chart.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • helm/robusta/Chart.yaml
  • helm/robusta/charts/holmes-0.40.0.tgz
  • helm/robusta/charts/holmes-0.41.0.tgz

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.


Walkthrough

The Helm chart updates the holmes dependency from version 0.40.0 to 0.41.0.

Changes

Helm dependency update

Layer / File(s) Summary
Update Holmes dependency version
helm/robusta/Chart.yaml
The chart declares holmes version 0.41.0 instead of 0.40.0.

Priority: ⬇️ Low — Defer this narrow Holmes chart dependency bump because it changes only the Helm version and has already been validated in a test cluster.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to faa24

This updates the bundled Holmes chart to version 0.41.0. The dependency declaration and lock resolution are aligned, with no current merge-blocking risk identified.

Suggested reviewers: moshemorad

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description clearly explains the Holmes chart dependency update, the related chart archive changes, and the deployment testing.
Title check ✅ Passed The title clearly and concisely identifies the Holmes chart dependency bump from 0.40.0 to 0.41.0.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch holmes-0.41.0-alpha

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

@Avi-Robusta
Avi-Robusta merged commit 8df33a3 into master Sep 8, 2026
5 of 6 checks passed
@Avi-Robusta
Avi-Robusta deleted the holmes-0.41.0-alpha branch September 8, 2026 12:55
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.

2 participants