PLTF-19: Automate updating OpenHands and runtime-api charts for enterprise release#262
Conversation
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
…ates Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
The script now only updates the chart files locally. Users can use external tools like gh CLI or peter-evans/create-pull-request action to handle git operations and PR creation. Co-authored-by: openhands <openhands@all-hands.dev>
Instead of fetching the latest semver tag from OpenHands/OpenHands repo, the script now: 1. Gets the OPENHANDS_SHA from the deploy config 2. Runs git tag --contains on the local OpenHands repo 3. Uses the latest semver tag containing that commit as appVersion This ensures the appVersion matches the actual deployed commit. Co-authored-by: openhands <openhands@all-hands.dev>
…updates Co-authored-by: openhands <openhands@all-hands.dev>
- Update runtime-api chart files first and use the bumped version for the openhands chart dependency instead of fetching from OCI registry - Remove get_latest_helm_chart_version function (no longer needed) - Simplify flow by removing conditional deploy_config check (already required earlier in the function) Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
To reduce scope of this script, the commit and create PR logic has been removed in 03b3542 |
- Rename update-openhands-chart.py to update_openhands_charts.py - Rename test_update_openhands_chart.py to test_update_openhands_charts.py - Update README.md references - Fix typo: update_openhand_chart -> update_openhands_chart Co-authored-by: openhands <openhands@all-hands.dev>
- Move update_openhands_charts.py, test_update_openhands_charts.py, and README.md to their own folder under scripts/ - Update REPO_ROOT path to account for new folder structure - Update README.md with correct paths Co-authored-by: openhands <openhands@all-hands.dev>
This reverts commit 38e0943.
| path: charts/openhands | ||
|
|
||
| # because openhands depends on runtime-api | ||
| max-parallel: 1 |
There was a problem hiding this comment.
I believe this PR needs to be merged before #264 so that the runtime-api chart publish can happen before the openhands chart publish
- Add runtime_api_sha parameter to update_runtime_api_values function - Update image.tag in runtime-api values.yaml with sha-SHORT_SHA format - Add test for image tag update Co-authored-by: openhands <openhands@all-hands.dev>
|
One last thing. When I ran this before setting the env var, it didn't get output indicating the problem was that I needed to set the token. Approving, you're welcome to merge when you believe this is addressed. |
Updated in 8d7cbc2 User sees the following error if they have not set their token env var: |
Description
Adding a script that makes the relevant chart updates for an OpenHands enterprise release
Helm Chart Checklist
versionfield inChart.yamlfor each modified chartAdditional Notes
Script run generated the following changes: #264