Skip to content

Upgrade Go and dependencies to latest versions#1012

Draft
aravindtga wants to merge 1 commit into
kptdev:mainfrom
Nordix:upgrade-go-1-26-3-with-kpt
Draft

Upgrade Go and dependencies to latest versions#1012
aravindtga wants to merge 1 commit into
kptdev:mainfrom
Nordix:upgrade-go-1-26-3-with-kpt

Conversation

@aravindtga
Copy link
Copy Markdown
Contributor

@aravindtga aravindtga commented May 27, 2026 •
edited
Loading

Upgrade Go and dependencies to latest versions

Note

1225 of the 1258 changed files are in third_party/k8s.io/apiserver-v0.36.1/. the local apiserver fork updated from v0.34.1 to v0.36.1 (with the single-line requestTimeoutUpperBound patch preserved).

Only 33 files contain actual porch code changes.

What changed:

  • Go 1.25.7 → 1.26.3
  • Kubernetes client libraries 0.34.1 → 0.36.1
  • controller-runtime 0.22.4 → 0.24.1
  • kpt v1.0.0-beta.62.1 → v1.0.0-beta.64
  • krm-functions-sdk v1.0.2 → v1.0.3
  • Local apiserver fork updated from v0.34.1 to v0.36.1 (preserving the requestTimeoutUpperBound patch)
  • kind node image updated to v1.35.1
  • Disabled MutatingAdmissionPolicy admission plugin (not available as v1 until k8s 1.36)
  • Regenerated OpenAPI specs, deepcopy, and informer code for the new k8s version
  • Fixed mockery docker image Go version mismatch with GOTOOLCHAIN=auto
  • Fixed test code to use SDK's kptfileapi types instead of kpt's

Type of Change

  • Bug fix
  • Enhancement
  • New feature
  • Refactor
  • Documentation
  • Tests
  • Other: ________

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes
  • Tests added/updated
  • Documentation added/updated
  • All tests and gating checks pass

AI Disclosure

  • I have used AI in the creation of this PR.

If so, please describe how:

  • Kiro CLI to assist with dependency analysis, compatibility checks, and code fixes during the upgrade process.

Copilot AI review requested due to automatic review settings May 27, 2026 16:55
@aravindtga aravindtga self-assigned this May 27, 2026
Copilot AI reviewed May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch 2 times, most recently from af4cc28 to 991eec7 Compare May 27, 2026 17:35
Copilot AI review requested due to automatic review settings May 27, 2026 17:35
Copilot AI reviewed May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch from 991eec7 to 1708204 Compare May 27, 2026 17:44
Copilot AI review requested due to automatic review settings May 27, 2026 17:51
@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch from 1708204 to 8d0f2de Compare May 27, 2026 17:51
Copilot AI reviewed May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch from 8d0f2de to 5c99642 Compare May 27, 2026 18:12
Copilot AI review requested due to automatic review settings May 27, 2026 20:28
@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch from 5c99642 to 34d000a Compare May 27, 2026 20:28
Copilot AI reviewed May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch 3 times, most recently from 106702d to bfdea17 Compare May 27, 2026 21:31
Copilot AI review requested due to automatic review settings May 27, 2026 21:31
Copilot AI reviewed May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch 2 times, most recently from f9a7c89 to 2aa665a Compare May 27, 2026 22:17
Copilot AI review requested due to automatic review settings May 27, 2026 22:17
Copilot AI reviewed May 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch 2 times, most recently from 21ed04e to 051407b Compare May 28, 2026 05:47
Copilot AI review requested due to automatic review settings May 28, 2026 05:47
Copilot AI reviewed May 28, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 28, 2026 •
edited
Loading

✅ Deploy Preview for kpt-porch ready!

Name Link
🔨 Latest commit d5ec7f9
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a17daa728ae44000889b93b
😎 Deploy Preview https://deploy-preview-1012--kpt-porch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@aravindtga
Upgrade Go and dependencies to latest versions
d5ec7f9 
Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
@aravindtga aravindtga force-pushed the upgrade-go-1-26-3-with-kpt branch from 051407b to d5ec7f9 Compare May 28, 2026 06:03
@aravindtga aravindtga marked this pull request as ready for review May 28, 2026 06:19
Copilot AI review requested due to automatic review settings May 28, 2026 06:19
Copilot AI reviewed May 28, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@liamfallon
Copy link
Copy Markdown
Contributor

liamfallon commented May 28, 2026

@aravindtga we should not make any code changed in third_party/k8s.io/apiserver-v0.36.1/, but instead upgrade that entire clone to the latest version of the API server.

@aravindtga aravindtga marked this pull request as draft May 28, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@aravindtga aravindtga

Labels

do-not-merge/work-in-progress #ededed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aravindtga @liamfallon