Conversation
Signed-off-by: Qingchuan Hao <qingchuan.hao@microsoft.com>
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.
Tip: disable this comment in your organization's Code Review settings.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThe project metadata now supports Python versions below 3.15 and upgrades LiteLLM from 1.89.0 to 1.93.0. Obsolete Python-version restriction comments were removed. ChangesRuntime and dependency compatibility
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. Comment |
✅ Deploy Preview for holmes-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@pyproject.toml`:
- Line 13: Update the pyproject.toml Python requirement to exclude Python 3.14,
keeping the supported range aligned with the Prometheus toolset compatibility
documented in CONTRIBUTING.md.
🪄 Autofix
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 Plus
Run ID: 4da05d67-2eaa-401d-9723-9b2c36753d29
⛔ Files ignored due to path filters (1)
poetry.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
pyproject.toml
Signed-off-by: Qingchuan Hao <qingchuan.hao@microsoft.com> # Conflicts: # poetry.lock
|
✅ Docker images ready for
Use these tags to pull the images for testing. 📋 Copy commandsgcloud auth configure-docker us-central1-docker.pkg.dev
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes:0cbc1d9d3
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes:0cbc1d9d3 me-west1-docker.pkg.dev/robusta-development/development/holmes-dev:0cbc1d9d3
docker push me-west1-docker.pkg.dev/robusta-development/development/holmes-dev:0cbc1d9d3
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes-operator:0cbc1d9d3
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes-operator:0cbc1d9d3 me-west1-docker.pkg.dev/robusta-development/development/holmes-operator-dev:0cbc1d9d3
docker push me-west1-docker.pkg.dev/robusta-development/development/holmes-operator-dev:0cbc1d9d3Patch Helm values in one line (choose the chart you use): HolmesGPT chart: helm upgrade --install holmesgpt ./helm/holmes \
--set registry=me-west1-docker.pkg.dev/robusta-development/development \
--set image=holmes-dev:0cbc1d9d3 \
--set operator.registry=me-west1-docker.pkg.dev/robusta-development/development \
--set operator.image=holmes-operator-dev:0cbc1d9d3Robusta wrapper chart: helm upgrade --install robusta robusta/robusta \
--reuse-values \
--set holmes.registry=me-west1-docker.pkg.dev/robusta-development/development \
--set holmes.image=holmes-dev:0cbc1d9d3 \
--set holmes.operator.registry=me-west1-docker.pkg.dev/robusta-development/development \
--set holmes.operator.image=holmes-operator-dev:0cbc1d9d3 |
Signed-off-by: Qingchuan Hao <qingchuan.hao@microsoft.com> # Conflicts: # poetry.lock
Summary
LiteLLM 1.93.0 has been available for more than two weeks.
Validation
poetry lock --no-updatepoetry check --lockSummary by CodeRabbit