-
Notifications
You must be signed in to change notification settings - Fork 35
PLTF-19: Automate updating OpenHands and runtime-api charts for enterprise release #262
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.
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
Merged
Merged
Changes from 64 commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
b0a3ae9
Add empty update-openhands-chart script
openhands-agent b744af0
rename to py
aivong-openhands 58257e7
Add PyGithub to fetch latest tag from OpenHands repo
openhands-agent 4f98c72
Filter tags to match semantic version format (x.y.z)
openhands-agent cd11761
Add functionality to update Chart.yaml appVersion with latest tag
openhands-agent 2978a17
Replace ruamel.yaml with PyYAML
openhands-agent f55479b
Change repo reference to OpenHands/OpenHands to prevent redirection
openhands-agent ef5a4bd
Use regex to update appVersion without reformatting YAML
openhands-agent bf5ac44
Use ruamel.yaml to preserve original YAML formatting
openhands-agent 8524dea
Add README with script usage instructions
openhands-agent ee17c9f
remove note
aivong-openhands 3de9479
Add patch version bump for chart version field
openhands-agent 463fe2c
Add runtime-api version update from OCI registry
openhands-agent 8035e1a
Add message when runtime-api version is already latest
openhands-agent 3b90cd5
Add unit tests for update-openhands-chart script
openhands-agent cfc66d2
Remove __pycache__ from version control
openhands-agent 91ff6e7
Add deploy repo tag fetching with GITHUB_TOKEN support
openhands-agent 13e13de
Fetch deploy config values from deploy.yaml workflow
openhands-agent fcdad69
Update enterprise-server image tag from deploy config
openhands-agent 2da7239
Update runtime-api image tag from deploy config
openhands-agent 94ad52b
Update runtime image tag from deploy config
openhands-agent 96b024f
Add unit tests for DeployConfig and update_values
openhands-agent 6034e28
Remove __pycache__ from version control
openhands-agent d4f67b6
Add get_short_sha helper function and unit tests
openhands-agent 52e739e
Add __pycache__ to .gitignore
openhands-agent 9d5e3cb
Add console output separation between fetched and updated values
openhands-agent 494ef14
Remove __pycache__ files
openhands-agent 7fbd647
Fix .gitignore format
openhands-agent b8d6cc3
Add console output distinction between Chart.yaml and values.yaml upd…
openhands-agent 7fc496e
Add format_sha_tag helper function and unit tests
openhands-agent 63bf050
Add --dry-run flag to show changes without modifying files
openhands-agent 432c2a6
Automate draft PR creation with branch update-openhands-chart-VERSION
openhands-agent 127a806
Fix PR to only include Chart.yaml and values.yaml changes
openhands-agent 0ba8894
update README
aivong-openhands db860a8
clean up README
aivong-openhands 7856924
Merge branch 'main' into aivong/openhands-chart-script
aivong-openhands 6310e5a
add instructions to use dry run mode
aivong-openhands 20ebe58
update readme
aivong-openhands f191da0
add instructions on running tests
aivong-openhands 171dc68
general title in readme
aivong-openhands ef072c3
add note on getting the github token
aivong-openhands d226b61
Check for uncommitted changes instead of using stash
openhands-agent 3059cc6
Add --deploy-tag optional argument to specify deploy tag
openhands-agent 15c9cd5
Bump runtime-api version to 0.1.21 and update warmRuntimes image
openhands-agent b091677
set charts to be publish sequentially due to runtime api dependency
aivong-openhands d32a9c3
Revert "Bump runtime-api version to 0.1.21 and update warmRuntimes im…
openhands-agent 38b8162
Add runtime-api chart version bump and warmRuntimes image update to s…
openhands-agent 3cf83e6
elaborate on help
aivong-openhands 07fcdc4
Merge branch 'main' into aivong/openhands-chart-script
aivong-openhands 03b3542
Remove commit and PR creation logic from update script
openhands-agent 09b9f30
Use semver tag containing OPENHANDS_SHA for appVersion
openhands-agent c57c2b0
Improve console output to distinguish openhands vs runtime-api chart …
openhands-agent 0d64d0b
add help command in readme
aivong-openhands 7139d3d
rename openhands specific functions
aivong-openhands 982abc6
Update runtime-api chart before openhands chart
openhands-agent 6efe680
specify var name instead of sha to reduce duplicated values
aivong-openhands 9acaa0a
Checkout main branch before getting semver tag containing commit
openhands-agent 54cda22
update readme
aivong-openhands a969ccb
Rename script to update_openhands_charts.py
openhands-agent 832316f
Merge branch 'main' into aivong/openhands-chart-script
aivong-openhands b9f8491
Move script files to scripts/update_openhands_charts/ folder
openhands-agent 38e0943
run PR checks sequentially
aivong-openhands f88ebab
Revert "run PR checks sequentially"
aivong-openhands d90f73d
Merge branch 'main' into aivong/openhands-chart-script
aivong-openhands 65cfcd9
Update runtime-api image tag with sha-SHORT_SHA from RUNTIME_API_SHA
openhands-agent 8d7cbc2
exit when github token envvar is not set
aivong-openhands File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,7 @@ | ||
| charts/**/charts | ||
|
|
||
| **/.DS_Store | ||
| __pycache__/ | ||
|
|
||
| # Files generated by JetBrains IDEs, e.g. IntelliJ IDEA | ||
| .idea/ | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| # Description | ||
|
|
||
| Updates the OpenHands and runtime-api helm charts to cut a new enterprise chart release. | ||
|
|
||
| ## Prerequisites | ||
|
|
||
| - [uv](https://docs.astral.sh/uv/) must be installed | ||
| - A GitHub token with read access to the OpenHands repository | ||
|
|
||
| ## Usage | ||
|
|
||
| 1. Set the `GITHUB_TOKEN` environment variable: | ||
|
|
||
| ```bash | ||
| export GITHUB_TOKEN=your_github_token | ||
|
aivong-openhands marked this conversation as resolved.
|
||
| ``` | ||
|
|
||
| > Try getting with: `gh auth status --show-token` | ||
|
|
||
| 2. Run the script: | ||
|
|
||
| ```bash | ||
| ./scripts/update_openhands_charts/update_openhands_charts.py | ||
| ``` | ||
|
|
||
| Or using uv directly: | ||
|
|
||
| ```bash | ||
| uv run scripts/update_openhands_charts/update_openhands_charts.py | ||
| ``` | ||
|
|
||
| > View help for available arguments: `uv run scripts/update_openhands_charts/update_openhands_charts.py --help` | ||
|
|
||
| ### DRY RUN mode | ||
|
|
||
| ```bash | ||
| ./scripts/update_openhands_charts/update_openhands_charts.py --dry-run | ||
| ``` | ||
|
|
||
| Or using uv directly: | ||
|
|
||
| ```bash | ||
| uv run scripts/update_openhands_charts/update_openhands_charts.py --dry-run | ||
| ``` | ||
|
|
||
| ## Tests | ||
|
|
||
| Run the tests: | ||
|
|
||
| ```bash | ||
| ./scripts/update_openhands_charts/test_update_openhands_charts.py | ||
| ``` | ||
|
|
||
| Or using uv directly: | ||
|
|
||
| ```bash | ||
| uv run scripts/update_openhands_charts/test_update_openhands_charts.py | ||
| ``` | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this PR needs to be merged before #264 so that the runtime-api chart publish can happen before the openhands chart publish
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good thinking.