PLTF-2890: Release 1.37.2#679
Conversation
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
|
|
✅ Review complete. This review was performed through OpenHands Cloud Automation. You can log in and view the conversation here. |
all-hands-bot
left a comment
There was a problem hiding this comment.
Release 1.37.1 — Review
This is a standard release update PR with changes generated by scripts/update_openhands_charts/update_openhands_charts.py. The diff is clean and internally consistent. Here is a breakdown of the review:
Summary of Changes
| Component | Previous Tag/Version | New Tag/Version |
|---|---|---|
enterprise-server (openhands app) |
cloud-1.34.0 |
cloud-1.37.1 |
runtime-api |
sha-3603c38 |
sha-f74dce0 |
automation |
sha-c58faa1 |
sha-3a35409 |
agent-server (runtime) |
1.22.1-python / 1.21.1-python |
1.23.1-python (unified) |
Consistency Checks ✅
- Helm chart versions correctly bumped in each chart's
Chart.yaml(automation: 0.1.10→0.1.11, image-loader: 0.1.6→0.1.7, openhands: 0.7.36→0.7.37, runtime-api: 0.3.5→0.3.6). - Dependency pinning in
charts/openhands/Chart.yamlcorrectly references the bumped sub-chart versions (runtime-api: 0.3.6, automation: 0.1.11). agent-serverreferences are now uniformly1.23.1-pythonacross all files:charts/image-loader/values.yaml,charts/openhands/values.yaml,charts/runtime-api/values.yaml,replicated/openhands.yaml(all four occurrences), andreplicated/config.yaml.
Notable Observations
Previously stale replicated tags (now fixed): replicated/config.yaml and replicated/openhands.yaml previously held 1.21.1-python as the hardcoded fallback/default, while charts/runtime-api/values.yaml was already on 1.22.1-python. This two-version drift has been brought into alignment in this PR — all references now point to 1.23.1-python. This was already called out and addressed in earlier review iterations (commit 82ba38f).
Version jump: The openhands appVersion jumps from cloud-1.34.0 to cloud-1.37.1, skipping 1.35.x and 1.36.x. This is expected if those tags correspond to intermediate or hotfix releases that were deployed independently without a chart update cycle.
SHA-based image tags: runtime-api and automation continue to use SHA-based image tags rather than semantic version strings. This is a consistent pattern, though it means the commit message/PR description is the only human-readable reference to what changed in those images.
Resolved Threads
Both pre-existing review threads (automation chart description fix, stale agent-server image references) are resolved.
Verdict
✅ The PR is mechanically sound. All image tags and chart versions are consistent across every file in the diff. Ready to merge.
This review was generated by an AI agent (OpenHands) on behalf of the user through OpenHands Automation. View conversation
|
@OpenHands update the chart lock file |
|
I'm on it! aivong-openhands can track my progress at all-hands.dev |
Co-authored-by: openhands <openhands@all-hands.dev>
|
Updated the chart lock file in d3e46d2 (charts/openhands/Chart.lock now locks runtime-api 0.3.6 and automation 0.1.11).\n\nThis comment was created by an AI agent (OpenHands) on behalf of the user. |
|
Since the last summary, there were no additional code changes. Checklist:
The changes are concise: only the chart lock file was modified to address the reviewer’s request. |
|
✅ Review complete. This review was performed through OpenHands Cloud Automation. You can log in and view the conversation here. |
all-hands-bot
left a comment
There was a problem hiding this comment.
PLTF-2890: Release 1.37.1 — Code Review
This is a routine release-bump PR generated by the update script. All 11 changed files are chart version increments and image tag updates. No application logic is modified.
What Changed
| Component | Old | New |
|---|---|---|
| appVersion | ||
| image | ||
| image | / | (unified) |
| image | ||
| image | ||
| chart | ||
| chart | ||
| chart | ||
| chart |
Verification
- ✅ dependency pins match bumped sub-chart versions (, )
- ✅ digest and timestamp refreshed consistently
- ✅ All image references are now uniformly (previously split across and — that drift was addressed and fixed in earlier commits on this PR)
- ✅ Previous review feedback (automation chart description, stale image references) has been resolved
No issues found. The diff is clean and internally consistent.
This review was generated by an AI agent (OpenHands) on behalf of the user through OpenHands Automation. View conversation
Looks like content is missing 🤔 |
|
✅ Review complete. This review was performed through OpenHands Cloud Automation. You can log in and view the conversation here. |
all-hands-bot
left a comment
There was a problem hiding this comment.
PLTF-2890: Release 1.37.1 — Review
This is a standard automated release bump PR. The diff is clean, internally consistent, and the main observations have already been noted in prior reviews. Confirming the analysis below.
Summary
All 11 changed files are chart version increments and image tag updates generated by scripts/update_openhands_charts/update_openhands_charts.py. No application logic is modified.
| Chart / Component | Before | After |
|---|---|---|
openhands appVersion |
cloud-1.34.0 |
cloud-1.37.1 |
openhands chart version |
0.7.36 |
0.7.37 |
runtime-api chart version |
0.3.5 |
0.3.6 |
runtime-api image |
sha-3603c38 |
sha-f74dce0 |
automation chart version |
0.1.10 |
0.1.11 |
automation image |
sha-c58faa1 |
sha-3a35409 |
image-loader chart version |
0.1.6 |
0.1.7 |
agent-server image (all locations) |
1.21.1-python / 1.22.1-python |
1.23.1-python (unified) |
Checks ✅
charts/openhands/Chart.yamldependency pins correctly reference the bumped sub-chart versions (runtime-api: 0.3.6,automation: 0.1.11).charts/openhands/Chart.lockdigest and timestamp are refreshed.- All
agent-serverimage references are now uniformly1.23.1-pythonacrosscharts/image-loader/values.yaml,charts/openhands/values.yaml,charts/runtime-api/values.yaml, and all four occurrences inreplicated/openhands.yaml— the previously observed drift (1.21.1-pythonvs1.22.1-python) is fully resolved. - The
replicated/config.yamldefault and help-text example forcustom_sandbox_image_tagare updated consistently to1.23.1-python.
Observation: Unchecked Checklist Items
Two items in the Helm Chart Checklist remain unchecked:
- "I have tested the chart upgrade path from the previous version" — Given the
appVersionjumps fromcloud-1.34.0tocloud-1.37.1(skipping 1.35.x and 1.36.x), it is worth confirming the upgrade path fromcloud-1.34.0has been validated or that this jump is intentional/safe (e.g., intermediate releases were deployed via a different mechanism without a chart update cycle). - "I have updated the chart's README.md if there are any breaking changes" — Can be left unchecked if no breaking changes are introduced in this release.
No other issues found.
This review was generated by an AI agent (OpenHands) on behalf of the user through OpenHands Automation. View conversation
OpenHands/OpenHands@82744a0 was merged |
has cherry picked cascade delete fix
Description
Helm Chart Checklist
versionfield inChart.yamlfor each modified chartAdditional Notes