Skip to content

[ROB-0000] bumping holmes #2052

Merged
Avi-Robusta merged 1 commit intomasterfrom
bump-holmes
Apr 23, 2026
Merged

[ROB-0000] bumping holmes #2052
Avi-Robusta merged 1 commit intomasterfrom
bump-holmes

Conversation

@Avi-Robusta
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

Docker image ready for 949e0c8 (built in 2m 45s)

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

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:949e0c8

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

Walkthrough

The holmes Helm chart dependency version in helm/robusta/Chart.yaml was updated from 0.24.0 to 0.25.0. This change affects the resolved dependency version when enableHolmesGPT is enabled in the Robusta Helm chart configuration.

Changes

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

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • arikalon1
  • moshemorad
  • RoiGlinik
🚥 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 verify if it relates to the changeset. Add a description explaining the purpose and impact of bumping the holmes dependency version.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: bumping the holmes dependency version from 0.24.0 to 0.25.0 in the Helm chart.
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 bump-holmes

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

Copy link
Copy Markdown

@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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@helm/robusta/Chart.yaml`:
- Line 18: The Chart.yaml entry currently sets "version: 0.25.0" which isn't an
existing HolmesGPT release; update that value to a valid release tag (e.g.,
"0.24.3") or the correct intended release, and confirm the corresponding Helm
chart/tag exists in the Robusta Helm repository so the chart version and any
image/tag references remain consistent with the chosen Holmes release.
🪄 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: 099a2c55-5aea-44ab-a2d0-e85b30e0473e

📥 Commits

Reviewing files that changed from the base of the PR and between 7ec46d2 and 431766e.

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

Comment thread helm/robusta/Chart.yaml
@Avi-Robusta Avi-Robusta merged commit 410ef31 into master Apr 23, 2026
7 checks passed
@Avi-Robusta Avi-Robusta deleted the bump-holmes branch April 23, 2026 17:43
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