Skip to content

Go 1.26 and K8s 1.36 client-go updates#4703

Open
marosset wants to merge 11 commits intoray-project:masterfrom
marosset:go-client-go-updates
Open

Go 1.26 and K8s 1.36 client-go updates#4703
marosset wants to merge 11 commits intoray-project:masterfrom
marosset:go-client-go-updates

Conversation

@marosset
Copy link
Copy Markdown
Contributor

@marosset marosset commented Apr 13, 2026

Why are these changes needed?

These changes update the client-go packages to the K8s v1.36.0-beta.0 versions (which require updating golang to v1.26) in order to use the new scheduling APIs needed for native workload aware scheduling

x-ref: #4344

Most of the changes were made by running golangci-lint or the new code-generator for K8s client-go v1.36

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

@marosset marosset added the go Pull requests that update Go code label Apr 13, 2026
@marosset
Copy link
Copy Markdown
Contributor Author

I can make another PR to update to the GA v1.36.0 client-go packages once they are released.

….26)

fixup! Updating client-go to v1.36.0-beta.0 (and updating golang to 1.26)
fixup! Bump golangci-lint to v2.11.4 and fix lint issues for Go 1.26
fixup! bump go to 1.26 in ray-operator docker file
fixup! bump go to 1.26 in remaining files
fixup! suppress pre-existing gosec false positive in job_test.go
fixup! fix lint issues across kubectl-plugin, apiserver, and ray-operator
fixup! remove comment for deleted intPointer function
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 81774cf. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants