Skip to content

Bump modules and minimum go version#55514

Merged
istio-testing merged 1 commit intoistio:masterfrom
howardjohn:mod/bump-124
Mar 19, 2025
Merged

Bump modules and minimum go version#55514
istio-testing merged 1 commit intoistio:masterfrom
howardjohn:mod/bump-124

Conversation

@howardjohn
Copy link
Copy Markdown
Member

Start requiring Go 1.24.

We don't use any new features of it yet in this PR, but can in others
(generic type alias, etc).

RSA key size min was introduced, so workaround it in tests (we could
also change the key size, but it takes multiple seconds to generate
large RSA keys with -race)

@howardjohn howardjohn added the release-notes-none Indicates a PR that does not require release notes. label Mar 13, 2025
@howardjohn howardjohn requested a review from hzxuzhonghu as a code owner March 13, 2025 15:58
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Mar 13, 2025
@howardjohn
Copy link
Copy Markdown
Member Author

/retest

Start requiring Go 1.24.

We don't use any new features of it yet in this PR, but can in others
(generic type alias, etc).

RSA key size min was introduced, so workaround it in tests (we could
also change the key size, but it takes multiple seconds to generate
large RSA keys with -race)
@howardjohn
Copy link
Copy Markdown
Member Author

/retest

@istio-testing istio-testing merged commit 3e13f43 into istio:master Mar 19, 2025
29 checks passed
fjglira pushed a commit to fjglira/istio that referenced this pull request Sep 26, 2025
* upstream/master:
  Bump modules and minimum go version (istio#55514)
  update supported groups for TLS (istio#55589)
  Automator: update proxy@master in istio/istio@master (istio#55590)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes-none Indicates a PR that does not require release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants