Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: use a single source of truth for the go version (backport #15583) #15606

Merged
merged 2 commits into from
Feb 7, 2025

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 7, 2025

Motivation/summary

go version is defined in two places: go.mod and .go-version they often get out of sync and it's not clear which is used where. CI is using the version defined in go.mod which might be outdated compared to .go-version.

bump go version in go.mod and ensure scripts are using it consistently

remove old jenkins scripts

.go-version is kept around because it's still used in one place: the version bump automation. I plan to raise an issue to just create a workflow that runs make update-go-version since we already handle everything in a make task.

Checklist

For functional changes, consider:

  • Is it observable through the addition of either logging or metrics?
  • Is its use being published in telemetry to enable product improvement?
  • Have system tests been added to avoid regression?

How to test these changes

Related issues

Related to #15581


This is an automatic backport of pull request #15583 done by Mergify.

go version is defined in two places: go.mod and .go-version
they often get out of sync and it's not clear which is used where.
CI is using the version defined in go.mod which might be outdated compared
to .go-version.

bump go version in go.mod and ensure scripts are using it consistently

remove old jenkins scripts

(cherry picked from commit c24a1cb)
@mergify mergify bot requested a review from a team as a code owner February 7, 2025 17:09
@carsonip
Copy link
Member

carsonip commented Feb 7, 2025

@Mergifyio queue

Copy link
Contributor Author

mergify bot commented Feb 7, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 1353b25

@mergify mergify bot merged commit 1353b25 into 8.x Feb 7, 2025
13 checks passed
@mergify mergify bot deleted the mergify/bp/8.x/pr-15583 branch February 7, 2025 18:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants