Skip to content

Bump Holmes + KRR versions - #2160

Merged
moshemorad merged 1 commit into
masterfrom
rob-1229-release-new-holmes-runner
Aug 26, 2026
Merged

Bump Holmes + KRR versions#2160
moshemorad merged 1 commit into
masterfrom
rob-1229-release-new-holmes-runner

Conversation

@moshemorad

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Aug 26, 2026

Copy link
Copy Markdown

Docker image ready for cf91ac5 (built in 4m 41s)

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

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:cf91ac5

@coderabbitai

coderabbitai Bot commented Aug 26, 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: d087ebd0-18c2-4328-bebc-c528c772898a

📥 Commits

Reviewing files that changed from the base of the PR and between d5f32a2 and 9e120b6.

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

Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

The Helm chart updates the Holmes dependency to 0.40.0. The KRR playbook updates its default container image to v1.30.0. The KRR_IMAGE_OVERRIDE behavior remains unchanged.

Changes

Version updates

Layer / File(s) Summary
Holmes dependency update
helm/robusta/Chart.yaml
The Holmes Helm dependency version changes from 0.39.0 to 0.40.0.
KRR image update
playbooks/robusta_playbooks/krr.py
The default KRR image tag changes from v1.29.0 to v1.30.0. The environment-variable override remains supported.

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

Merge Risk: ⚪ Minimal · up to 9e120

This PR updates Holmes and KRR versions and their packaged chart dependency; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: avi-robusta

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so the description does not give meaningful context about the version updates. Add a brief description that states the Holmes dependency changed from 0.39.0 to 0.40.0 and the default KRR image changed from v1.29.0 to v1.30.0.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the two version bumps made in the changeset: Holmes and KRR.
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 1…
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.
Full details: Docstring Coverage

Explanation

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 1 files. (1 skipped: 1 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rob-1229-release-new-holmes-runner

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

@moshemorad
moshemorad merged commit 22e6dc1 into master Aug 26, 2026
7 checks passed
@moshemorad
moshemorad deleted the rob-1229-release-new-holmes-runner branch August 26, 2026 09:17
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