Skip to content

ROB-3020 update holmes dependency version #1997

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
RoiGlinik merged 1 commit into from
Jan 15, 2026
Merged

ROB-3020 update holmes dependency version #1997

RoiGlinik merged 1 commit into from
Jan 15, 2026
+4 −4

Conversation

@RoiGlinik
Copy link
Contributor

@RoiGlinik RoiGlinik commented Jan 15, 2026

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Jan 15, 2026 •
edited
Loading

Walkthrough

Updates the holmes Helm chart dependency version from 0.18.3 to 0.18.4 in the Robusta Helm chart configuration.

Changes

Cohort / File(s) Summary
Holmes dependency version bump
helm/robusta/Chart.yaml		 Updated holmes chart dependency from version 0.18.3 to 0.18.4

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • arikalon1
  • moshemorad
  • nherment
🚥 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, so the check cannot verify if the description relates to the changeset. Add a pull request description that explains the purpose and context of updating the holmes dependency version.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating the holmes dependency version from 0.18.3 to 0.18.4 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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ROB-3020-release-0.31.4
📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3e94bc1 and 9de2880.

⛔ 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.18.3.tgz
  • helm/robusta/charts/holmes-0.18.4.tgz
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: run_tests
  • GitHub Check: build
🔇 Additional comments (1)
helm/robusta/Chart.yaml (1)

17-20: Dependency version bump verified. Holmes version 0.18.4 is published in the robusta-charts repository and correctly configured.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link

github-actions bot commented Jan 15, 2026 •
edited
Loading

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

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

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

@RoiGlinik RoiGlinik enabled auto-merge (squash) January 15, 2026 09:12
@RoiGlinik RoiGlinik merged commit 99038b0 into master Jan 15, 2026
5 checks passed
@RoiGlinik RoiGlinik deleted the ROB-3020-release-0.31.4 branch January 15, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arikalon1 arikalon1 arikalon1 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@RoiGlinik @arikalon1