chore(codebase): enable copyloopvar

[ivankatliarchuk]

Description

Enable copyloopvar linter.

This was fixed in 1.21 and explained why is a bad idea in that go issue golang/go#56010

If copy is required, it should be clearly indicated, and most likely variable should be renamed.

extract from go issue

Changing the semantics is usually a no-op, and when it’s not, it fixes buggy code far more often than it breaks correct code
We built a toolchain with the change and tested a subset of Google’s Go tests and analyzed the resulting failures. The rate of new test failures was approximately 1 in 2,000, but nearly all were previously undiagnosed actual bugs. The rate of spurious test failures (correct code actually broken by the change) was 1 in 50,000.

somehow same linter cathches hidden special characters, that do cause some buggy logic

Checklist

• Unit tests updated
• End user documentation updated

[mloiseleur]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mloiseleur

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 [mloiseleur]

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