Skip to content

Upgrade Holmes 0.24.0#2043

Merged
arikalon1 merged 1 commit intomasterfrom
upgrade_holmes_0.24
Apr 5, 2026
Merged

Upgrade Holmes 0.24.0#2043
arikalon1 merged 1 commit intomasterfrom
upgrade_holmes_0.24

Conversation

@moshemorad
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Docker image ready for 8233ed0 (built in 2m 56s)

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

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:8233ed0

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 5, 2026

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: c6ebfd32-4c27-4981-aeb7-6e1772100af6

📥 Commits

Reviewing files that changed from the base of the PR and between 79c5495 and c59acb6.

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

Walkthrough

The Helm chart dependency for holmes in helm/robusta/Chart.yaml was updated from version 0.22.0 to 0.24.0. No other properties of the dependency were modified.

Changes

Cohort / File(s) Summary
Helm Chart Dependency Update
helm/robusta/Chart.yaml
Updated holmes dependency version from 0.22.0 to 0.24.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • arikalon1
  • RoiGlinik
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, which makes it impossible to assess whether it relates to the changeset. Add a pull request description explaining the purpose of the upgrade, any breaking changes, testing performed, or relevant context.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and accurately summarizes the main change: upgrading the Holmes dependency from 0.22.0 to 0.24.0 in the Helm chart.
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
  • Commit unit tests in branch upgrade_holmes_0.24

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

@arikalon1 arikalon1 merged commit 7c8d299 into master Apr 5, 2026
7 checks passed
@arikalon1 arikalon1 deleted the upgrade_holmes_0.24 branch April 5, 2026 17:02
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