Skip to content

Update Go to 1.25#7326

Open
Sovietaced wants to merge 6 commits into
flyteorg:mainfrom
Sovietaced:go-1.25
Open

Update Go to 1.25#7326
Sovietaced wants to merge 6 commits into
flyteorg:mainfrom
Sovietaced:go-1.25

Conversation

@Sovietaced
Copy link
Copy Markdown
Member

@Sovietaced Sovietaced commented Apr 30, 2026

Why are the changes needed?

This pull request updates Flyte 2.0 to Go 1.25. It also includes...

  • Updates golangci-lint from 2.4.0 to 2.6.2. I chose to use the golangci lint file from executor to upgrade old v1 config files.
  • Runs make lint-fix on executor, flyteplugins, flytestdlib, and runs

This should unblock a number of dependency updates / security fixes.

What changes were proposed in this pull request?

How was this patch tested?

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.
  • changed: For changes in existing functionality.
  • deprecated: For soon-to-be-removed features.
  • removed: For features being removed.
  • fixed: For any bug fixed.
  • security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Stack

If you do use git town to manage PR Stacks, the stack relevant to this PR
will show below. Otherwise, you can ignore this section.

Docs link

uses: actions/setup-go@v5
with:
go-version: "1.24"
go-version: "1.25"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we go 1.26?

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

🐳 Docker CI Image Built

The CI Docker image has been built and pushed for this PR!

Image: ghcr.io/flyteorg/flyte/ci:pr-7326

This image will be automatically used by CI workflows in this PR.

To test locally:

make gen DOCKER_CI_IMAGE=ghcr.io/flyteorg/flyte/ci:pr-7326

Signed-off-by: Jason Parraga <sovietaced@gmail.com>
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
@Sovietaced Sovietaced marked this pull request as ready for review May 15, 2026 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants