Skip to content

chore(deps): bump golang from 1.24.5 to 1.24.6 and helm-values-schema-json plugin to 2.2.1#5126

Merged
moolen merged 4 commits intoexternal-secrets:mainfrom
jakobmoellerdev:bump-go-1.24.6
Aug 11, 2025
Merged

chore(deps): bump golang from 1.24.5 to 1.24.6 and helm-values-schema-json plugin to 2.2.1#5126
moolen merged 4 commits intoexternal-secrets:mainfrom
jakobmoellerdev:bump-go-1.24.6

Conversation

@jakobmoellerdev
Copy link
Copy Markdown
Contributor

Problem Statement

What is the problem you're trying to solve?

Bumping Go to remediate Trivy finding

Related Issue

Bumps go to 1.24.6 manually to resolve Trivy finding blocking CI https://github.com/external-secrets/external-secrets/actions/runs/16884393908/job/47828242767

Proposed Changes

How do you like to solve the issue and why?

Do the same as #5117 but also bump go runtime

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test
  • I ensured my PR is ready for review with make reviewable

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
@jakobmoellerdev jakobmoellerdev requested a review from a team as a code owner August 11, 2025 17:40
@jakobmoellerdev
Copy link
Copy Markdown
Contributor Author

Added a pin to the HELM schema plugin which introduced some breaking changes for flags and output reformatting (see previous failure in https://github.com/external-secrets/external-secrets/actions/runs/16887655454/job/47839413240)

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
@jakobmoellerdev
Copy link
Copy Markdown
Contributor Author

jakobmoellerdev commented Aug 11, 2025

have to go even further back on helm schema plugin. the current rollback I did failed but just didnt return a non-zero error code so CI passed as a false positive.

https://github.com/losisin/helm-values-schema-json/releases/tag/v2.2.1

Fixed
Exit with non-zero exit code on error by @applejag in 239

https://github.com/losisin/helm-values-schema-json/releases/tag/v2.0.0
Added

BREAKING: feat: kebab-case flags instead of camelCase, and --values/-f instead of --input/-i by @applejag in 178

thus we need to go to 1.9.2 or bump to v2.2.1

EDIT:
As per Slack Convo we bump to 2.2.1

properly bumps helm schema plugin to 2.2.1 and actually uses the correct flags that were previously failing

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
@applejag
Copy link
Copy Markdown

Sorry about the breaking changes!

Happy to see people are using the plugin though 😅

gusfcarvalho
gusfcarvalho previously approved these changes Aug 11, 2025
@jakobmoellerdev jakobmoellerdev changed the title chore(deps): bump golang from 1.24.5 to 1.24.6 chore(deps): bump golang from 1.24.5 to 1.24.6 and helm-values-schema-json plugin to 2.2.1 Aug 11, 2025
properly bumps helm schema plugin to 2.2.1 and actually uses the correct flags that were previously failing

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
@sonarqubecloud
Copy link
Copy Markdown

@jakobmoellerdev
Copy link
Copy Markdown
Contributor Author

well the CI gods haunt me, apparently the generator has some different new line settings than my pre commit formatter that caused the diff checker to break :D

@jakobmoellerdev
Copy link
Copy Markdown
Contributor Author

Aaaaaand now we have infra breaking on us. This PR is cursed 🤬 :D we will wait for the next run

Copy link
Copy Markdown
Member

@moolen moolen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉 thank you for fixing this ❤️

@moolen moolen merged commit a25afa4 into external-secrets:main Aug 11, 2025
32 of 35 checks passed
alliseeisgold pushed a commit to alliseeisgold/external-secrets that referenced this pull request Aug 25, 2025
…-json plugin to `2.2.1` (external-secrets#5126)

* chore(deps): bump golang from 1.24.5 to 1.24.6

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>

* chore(deps): pin helm schema plugin to v1.9.2 to avoid breaking changes

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>

* chore(deps): bump helm-schema to v2.2.1

properly bumps helm schema plugin to 2.2.1 and actually uses the correct flags that were previously failing

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>

* chore(deps): bump helm-schema to v2.2.1

properly bumps helm schema plugin to 2.2.1 and actually uses the correct flags that were previously failing

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>

---------

Signed-off-by: Jakob Möller <jakob.moeller@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants