Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update to go 1.23.3 #1005

Merged
merged 3 commits into from
Nov 20, 2024
Merged

Update to go 1.23.3 #1005

merged 3 commits into from
Nov 20, 2024

Conversation

hebelsan
Copy link
Contributor

@hebelsan hebelsan commented Nov 7, 2024

How to categorize this PR?

/area quality
/kind enhancement
/platform azure

What this PR does / why we need it:
Update to go 1.23.3

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Update go to version 1.23.3

@hebelsan hebelsan requested review from a team as code owners November 7, 2024 13:10
@gardener-robot gardener-robot added needs/review Needs review area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/enhancement Enhancement, improvement, extension platform/azure Microsoft Azure platform/infrastructure size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) labels Nov 7, 2024
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 7, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Nov 7, 2024
kon-angelo
kon-angelo previously approved these changes Nov 7, 2024
Copy link
Contributor

@kon-angelo kon-angelo left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/review Needs review labels Nov 7, 2024
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 7, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 7, 2024
go.mod Outdated
@@ -1,6 +1,6 @@
module github.com/gardener/gardener-extension-provider-azure

go 1.23.0
go 1.23.3
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure about this change 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

The go directive in the go.mod is the mininum go version needed to run the program. So, it should remain at 1.23.0 I believe

@kon-angelo kon-angelo self-requested a review November 11, 2024 15:00
@gardener-robot gardener-robot added the needs/review Needs review label Nov 11, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 11, 2024
@gardener-robot gardener-robot removed the needs/review Needs review label Nov 11, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 11, 2024
Dockerfile Outdated
@@ -1,5 +1,5 @@
############# builder
FROM golang:1.23.0 AS builder
FROM golang:1.23 AS builder
Copy link
Contributor

@kon-angelo kon-angelo Nov 11, 2024

Choose a reason for hiding this comment

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

Removing the patch version from the tests is great and it will make dependabot updates work out of the box for all specific go patch versions.

For the main images I think it's best to be very explicit

@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 11, 2024
@gardener-robot-ci-3 gardener-robot-ci-3 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Nov 11, 2024
@AndreasBurger AndreasBurger merged commit 37ddd7e into master Nov 20, 2024
10 checks passed
@AndreasBurger AndreasBurger deleted the upd-go branch November 20, 2024 08:38
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/enhancement Enhancement, improvement, extension needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) platform/azure Microsoft Azure platform/infrastructure reviewed/lgtm Has approval for merging size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) status/closed Issue is closed (either delivered or triaged)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants