-
Notifications
You must be signed in to change notification settings - Fork 140
Update toolhive images to v0.5.2 #2335
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
Conversation
6d14675 to
bc27866
Compare
bc27866 to
c6ca8fb
Compare
|
@claude Can you please bump the latest chart version with a minor bump. make sure you run pre-commit to update the docs |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
|
Claude finished @ChrisJBurns's task —— View job Chart version bump completedTasks:
SummarySuccessfully bumped the operator Helm chart version from Changes made:
Note: The README.md was manually updated since helm-docs and pre-commit tools were not available in the execution environment. The CI pipeline will validate the documentation generation when this PR is pushed. --- | |
Updates the operator Helm chart version from 0.3.6 to 0.4.0 (minor bump) to reflect the image updates to v0.5.2. Co-authored-by: Chris Burns <[email protected]>
|
The PR lints fail, can you make sure to run the pre-commit command, this will update the readme docs. Also I've changed my mind, can you bump the bug fix version of the chart instead of the minor version. |
|
@claude The PR lints fail, can you make sure to run the pre-commit command, this will update the readme docs. Also I've changed my mind, can you bump the bug fix version of the chart instead of the minor version. |
|
Claude finished @ChrisJBurns's task —— View job Chart version corrected to bugfix bumpTasks:
SummarySuccessfully corrected the chart version bump from a minor bump (0.4.0) to a bugfix/patch bump (0.3.7) as requested. Changes made:
Note: While pre-commit and helm-docs tools were not available in the execution environment, I've manually updated the README.md version badge to match. The CI pipeline will run pre-commit hooks when this PR is pushed, which will validate and regenerate the documentation if needed. |
Co-authored-by: Chris Burns <[email protected]>
|
@claude You haven't updated the openshift values file, and you need to run the pre-commit command in the base of this repo |
|
Claude finished @ChrisJBurns's task —— View job OpenShift values updated, pre-commit tools unavailableTasks:
SummaryI've successfully updated the OpenShift values file and pushed the changes to the branch. Changes Made:
Pre-commit Issue: go install github.com/norwoodj/helm-docs/cmd/helm-docs@latest
pre-commit run --all-filesHowever, I don't have approval to run Next Steps:
The OpenShift values are now correctly updated to v0.5.2 to match the main values file. |
Updates operator and proxyrunner images in values-openshift.yaml to match the versions in the main values.yaml file. Co-authored-by: Chris Burns <[email protected]>
|
@claude You should have the ability to run precommit now because the binaries should be available on the runner. So please try running the pre commit flag again please |
|
@claude Using the helm bump skill, please ensure the chart is bumped by a patch version |
|
Claude finished @ChrisJBurns's task —— View job Helm Chart Patch Version Bump - Environment Setup IssueTasks:
Issue SummaryI've attempted multiple approaches to run the required tools, but the GitHub Actions runner environment doesn't have the necessary binaries installed or in the PATH: Tools Checked:
Attempted Solutions:
Root CauseThe workflow file ( - name: Setup helm-docs
run: go install github.com/norwoodj/helm-docs/cmd/helm-docs@latestHowever, this step either:
Current State (All Changes Already Applied)The good news is that all the actual version bump work is already complete from previous Claude interactions: ✅ Chart.yaml: Version Recommended ActionsOption 1 - Fix Workflow (Recommended) - name: Add Go bin to PATH
run: echo "$HOME/go/bin" >> $GITHUB_PATHOption 2 - Verify Locally pre-commit run --all-files
# or
go install github.com/norwoodj/helm-docs/cmd/helm-docs@latest
helm-docs --chart-search-root=deploy/charts --template-files=./_templates.gotmpl --template-files=README.md.gotmplOption 3 - Grant Installation Permission VerificationDespite not being able to run helm-docs, the current state appears correct:
The only uncertainty is whether helm-docs would make any other changes to the README beyond what's already been updated. |
|
@claude Using the helm bump skill, please ensure the chart is bumped by a patch version. go should be setup properly now and you should have no issues finding the |
|
Claude finished @ChrisJBurns's task —— View job Helm Chart Patch Version Bump CompleteTasks:
SummarySuccessfully bumped the operator Helm chart with a patch version update as requested. Changes Applied:
Files Modified:
The chart version now properly reflects a patch bump and the appVersion matches the v0.5.2 images being deployed by this PR. All documentation has been regenerated using the helm-docs tool with the correct arguments from |
Updates the Helm chart version with a patch bump to align with the image updates to v0.5.2. Also updates appVersion to match the operator image version. Co-authored-by: Chris Burns <[email protected]>
|
@claude Thank you, although you have bumped the patch version by 2. Can you change it to |
|
Claude finished @ChrisJBurns's task —— View job Chart Version Corrected to 0.3.7Tasks:
SummarySuccessfully corrected the Helm chart version from 0.3.8 to 0.3.7 as requested. Changes Applied:
The chart now shows the correct patch version 0.3.7, matching the appVersion 0.5.2 for this image update PR. |
Co-authored-by: Chris Burns <[email protected]>
This PR contains the following updates:
v0.5.0->v0.5.2v0.5.0->v0.5.2Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
stacklok/toolhive (ghcr.io/stacklok/toolhive/operator)
v0.5.2Compare Source
What's Changed
8a1c437by @renovate[bot] in #2369-hide allto test-coverage Taskfile target by @JAORMX in #2414application/jsonby @blkt in #2422New Contributors
Full Changelog: v0.5.1...v0.5.2
v0.5.1Compare Source
What's Changed
GetToolConfigForMCPServerinto separate package for easier testing and reducing code in controllers layer by @ChrisJBurns in #2344f4d737aby @renovate[bot] in #2346hostby @blkt in #2353SampleRatefloat by @ChrisJBurns in #2367task lint-fixby @JAORMX in #2370Full Changelog: v0.5.0...v0.5.1
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.