cluster-api
cluster-api copied to clipboard
🌱 Added go directive test
fixes #10258
/retest
thanks for picking up this task! lgtm pending fixing the shell-check error
/retest
@sbueringer am not sure if my changes are breaking stuff could you confirm
/area ci
@AvineshTripathi thanks for working on this! /lgtm cc @sbueringer @killianmuldoon for a final pass (otherwise I will merge in a few days)
LGTM label has been added.
[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
- ~~OWNERS~~ [killianmuldoon]
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment
/lgtm
/hold cancel
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?
Hey sure I can take those up, can you explain bit more "manual cherry-pick this pr into.."
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