cluster-api icon indicating copy to clipboard operation
cluster-api copied to clipboard

🌱 Added go directive test

Open AvineshTripathi opened this issue 11 months ago • 9 comments

fixes #10258

AvineshTripathi avatar Mar 12 '24 18:03 AvineshTripathi

/retest

AvineshTripathi avatar Mar 12 '24 18:03 AvineshTripathi

thanks for picking up this task! lgtm pending fixing the shell-check error

fabriziopandini avatar Mar 13 '24 09:03 fabriziopandini

/retest

AvineshTripathi avatar Mar 15 '24 18:03 AvineshTripathi

@sbueringer am not sure if my changes are breaking stuff could you confirm

AvineshTripathi avatar Mar 15 '24 18:03 AvineshTripathi

/test pull-cluster-api-test-main

This seems a flake (there is also a discussion thread in slack)

fabriziopandini avatar Mar 18 '24 13:03 fabriziopandini

/area ci

fabriziopandini avatar Mar 19 '24 10:03 fabriziopandini

@AvineshTripathi thanks for working on this! /lgtm cc @sbueringer @killianmuldoon for a final pass (otherwise I will merge in a few days)

fabriziopandini avatar Mar 19 '24 10:03 fabriziopandini

LGTM label has been added.

Git tree hash: fe36be89b3c3b477151bc76d50197697b088c8ed

k8s-ci-robot avatar Mar 19 '24 10:03 k8s-ci-robot

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: killianmuldoon

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Mar 20 '24 08:03 k8s-ci-robot

/lgtm

/hold cancel

chrischdi avatar Mar 26 '24 11:03 chrischdi

Would be great if we can manual cherry-pick this PR into 1.6 & 1.5 (in both cases we should use 1.20 as GO_DIRECTIVE_VERSION)

@AvineshTripathi Do you want to take this over?

sbueringer avatar Mar 26 '24 12:03 sbueringer

Hey sure I can take those up, can you explain bit more "manual cherry-pick this pr into.."

AvineshTripathi avatar Mar 26 '24 12:03 AvineshTripathi

Basically you create the same PR against the release-1.6 & release-1.5 branch (but using 1.20 as GO_DIRECTIVE_VERSION). You can either do it entirely manually or something like this:

git checkout release-1.6
git checkout -b release-1.6-go-directive
git cherry-pick <commit of this PR>
# Change GO_DIRECTIVE_VERSION  to 1.20
git commit --amend
...

Just saying "Manually" because usually we have a cherry-pick bot. But this won't be enough as we want to use 1.20

sbueringer avatar Mar 26 '24 12:03 sbueringer