Skip to content

Upgrade holmes to version 0.32.0#2100

Merged
Sheeproid merged 1 commit into
masterfrom
update_holmes
Jun 11, 2026
Merged

Upgrade holmes to version 0.32.0#2100
Sheeproid merged 1 commit into
masterfrom
update_holmes

Conversation

@moshemorad

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5cf40176-723c-46bd-bda4-a05bcb89dcba

📥 Commits

Reviewing files that changed from the base of the PR and between bbb7481 and 642d57b.

⛔ 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.31.0.tgz
  • helm/robusta/charts/holmes-0.32.0.tgz

Walkthrough

The Helm chart for Robusta updates its dependency on the holmes chart to use version 0.32.0 instead of 0.31.0. This is a straightforward dependency version increment with no other configuration changes.

Changes

Helm Dependency Update

Layer / File(s) Summary
Holmes dependency version update
helm/robusta/Chart.yaml
The holmes chart dependency version is incremented from 0.31.0 to 0.32.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a pull request description explaining the purpose and impact of the holmes version upgrade.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title clearly and concisely describes the main change: upgrading the holmes dependency from version 0.31.0 to 0.32.0, which matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update_holmes

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

@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown

Docker image ready for 38cfd5e (built in 3m 3s)

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

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:38cfd5e

@Sheeproid Sheeproid enabled auto-merge (squash) June 11, 2026 07:02
@Sheeproid Sheeproid merged commit 737f5aa into master Jun 11, 2026
7 checks passed
@Sheeproid Sheeproid deleted the update_holmes branch June 11, 2026 07:10
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