🌱Upgrade golang version (1.19.6 -> 1.20.3)

[Prajyot-Parab]

What this PR does / why we need it:

• Upgrade golang version (1.19.6 -> 1.20.3)
• Fixing golangci-lint error: rand.Seed has been deprecated since Go 1.20.
  Seeding is added by default from 1.20 onwards. More info here

Part of #8459

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

[Prajyot-Parab]

/hold

[sbueringer]

/retest

/test pull-cluster-api-e2e-full-main
/test pull-cluster-api-e2e-workload-upgrade-1-27-latest-main

[sbueringer]

/test pull-cluster-api-apidiff-main
/test pull-cluster-api-build-main
/test pull-cluster-api-verify-main

[furkatgofurov7]

/test pull-cluster-api-apidiff-main

This looks good to me, thanks!
@sbueringer do we take care of the go mod bump in a separate PR (as we usual did in the past)?

[furkatgofurov7]

test pull-cluster-api-apidiff-main

hmm, seems I can't retrigger the test, might be that only maintainers have the rights to do so, but let's try these too since it was triggered but there was a force push after

/test pull-cluster-api-e2e-full-main
/test pull-cluster-api-e2e-workload-upgrade-1-27-latest-main

[sbueringer]

/retest

[sbueringer]

hmm, seems I can't retrigger the test, might be that only maintainers have the rights to do so, but let's try these too since it was triggered but there was a force push after

There are no limitations regarding triggering jobs (at least if you're an org member)

[sbueringer]

/test pull-cluster-api-build-main

[furkatgofurov7]

/retest

[Prajyot-Parab]

/unhold

[killianmuldoon]

/lgtm

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: 64f8b693e0aec3b02d54d5c1c26bfa849e147479

[sbueringer]

@Prajyot-Parab please add "Part of #8459" to the PR description

[sbueringer]

Let's please also bump:

• go.mod files

In a follow-up:

• cloudbuild.yaml, cloudbuild-nightly.yaml: there is currently no gcb-docker-gcloud image with Go 1.20 => I think that was in test-infra and we can open a PR to bump Go in that image

[Prajyot-Parab]

Noted.

[sbueringer]

@sbueringer do we take care of the go mod bump in a separate PR (as we usual did in the past)?

I would just do it in this PR. Just a bit simpler. I think there is no need to split it into a separate PR.

[furkatgofurov7]

@sbueringer do we take care of the go mod bump in a separate PR (as we usual did in the past)?

I would just do it in this PR. Just a bit simpler. I think there is no need to split it into a separate PR.

That is what I always thought but was not aware the reasoning behind the splitting, thanks for confirming.

/cc @Prajyot-Parab (to confirm the answer ^ to your question offline)

[Prajyot-Parab]

/hold

[Prajyot-Parab]

/retest

[sbueringer]

/lgtm
/approve

/hold
Feel free to remove the hold once all currently running tests are green

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: f9413ef7e4542c6f54804b896e27e740a0e1200f

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sbueringer

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:

• OWNERS [sbueringer]

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

[sbueringer]

/retest

known flake

[sbueringer]

/check-cla

[sbueringer]

@Prajyot-Parab Can you check if there's anything you have to do / can do regarding the CLA?

[sbueringer]

/hold cancel

tests are fine

[furkatgofurov7]

/easycla

[furkatgofurov7]

@Prajyot-Parab Can you check if there's anything you have to do / can do regarding the CLA?

@sbueringer fixed

[johannesfrey]

/area dependency