Add -race flag to CA UT script

[kukichek]

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

Adds race check to CA UT script runner

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

[k8s-ci-robot]

Hi @kukichek. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[kukichek]

/release-note-none

[adrianmoisey]

/area cluster-autoscaler

[mboersma]

/ok-to-test

[jackfrancis]

@kukichek could you add a superficial change to the CA go source code (e.g., a comment) to trigger a test run?

[kukichek]

@jackfrancis Added a commit with some comment reformatting. The tests are passing

[kgolab]

/approve

[kukichek]

/retitle Add -race flag to CA UT script

[k8s-ci-robot]

@kukichek: Re-titling can only be requested by trusted users, like repository collaborators.

Details

In response to this:

/retitle Add -race flag to CA UT script

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[jackfrancis]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jackfrancis, kgolab, kukichek

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• cluster-autoscaler/OWNERS [jackfrancis]
• hack/OWNERS [jackfrancis,kgolab]

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

[jackfrancis]

/cherry-pick cluster-autoscaler-release-1.32
/cherry-pick cluster-autoscaler-release-1.33
/cherry-pick cluster-autoscaler-release-1.34
/cherry-pick cluster-autoscaler-release-1.35

[k8s-infra-cherrypick-robot]

@jackfrancis: #9160 failed to apply on top of branch "cluster-autoscaler-release-1.32":

Applying: Add -race flag to ca test script
Using index info to reconstruct a base tree...
A	hack/go-unit-tests-ca.sh
Falling back to patching base and 3-way merge...
Auto-merging hack/install-verify-tools.sh
CONFLICT (content): Merge conflict in hack/install-verify-tools.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Add -race flag to ca test script

Details

In response to this:

/cherry-pick cluster-autoscaler-release-1.32
/cherry-pick cluster-autoscaler-release-1.33
/cherry-pick cluster-autoscaler-release-1.34
/cherry-pick cluster-autoscaler-release-1.35

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[jackfrancis]

/cherry-pick cluster-autoscaler-release-1.33

[k8s-infra-cherrypick-robot]

@jackfrancis: #9160 failed to apply on top of branch "cluster-autoscaler-release-1.33":

Applying: Add -race flag to ca test script
Using index info to reconstruct a base tree...
A	hack/go-unit-tests-ca.sh
Falling back to patching base and 3-way merge...
Auto-merging hack/install-verify-tools.sh
CONFLICT (content): Merge conflict in hack/install-verify-tools.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Add -race flag to ca test script

Details

In response to this:

/cherry-pick cluster-autoscaler-release-1.32
/cherry-pick cluster-autoscaler-release-1.33
/cherry-pick cluster-autoscaler-release-1.34
/cherry-pick cluster-autoscaler-release-1.35

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[k8s-infra-cherrypick-robot]

@jackfrancis: #9160 failed to apply on top of branch "cluster-autoscaler-release-1.33":

Applying: Add -race flag to ca test script
Using index info to reconstruct a base tree...
A	hack/go-unit-tests-ca.sh
Falling back to patching base and 3-way merge...
Auto-merging hack/install-verify-tools.sh
CONFLICT (content): Merge conflict in hack/install-verify-tools.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Add -race flag to ca test script

Details

In response to this:

/cherry-pick cluster-autoscaler-release-1.33

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[k8s-infra-cherrypick-robot]

@jackfrancis: #9160 failed to apply on top of branch "cluster-autoscaler-release-1.34":

Applying: Add -race flag to ca test script
Using index info to reconstruct a base tree...
A	hack/go-unit-tests-ca.sh
Falling back to patching base and 3-way merge...
Auto-merging hack/install-verify-tools.sh
CONFLICT (content): Merge conflict in hack/install-verify-tools.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Add -race flag to ca test script

Details

In response to this:

/cherry-pick cluster-autoscaler-release-1.32
/cherry-pick cluster-autoscaler-release-1.33
/cherry-pick cluster-autoscaler-release-1.34
/cherry-pick cluster-autoscaler-release-1.35

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[jackfrancis]

/cherry-pick cluster-autoscaler-release-1.35

[k8s-infra-cherrypick-robot]

@jackfrancis: new pull request created: #9225

Details

In response to this:

/cherry-pick cluster-autoscaler-release-1.32
/cherry-pick cluster-autoscaler-release-1.33
/cherry-pick cluster-autoscaler-release-1.34
/cherry-pick cluster-autoscaler-release-1.35

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[k8s-infra-cherrypick-robot]

@jackfrancis: new pull request could not be created: failed to create pull request against kubernetes/autoscaler#cluster-autoscaler-release-1.35 from head k8s-infra-cherrypick-robot:cherry-pick-9160-to-cluster-autoscaler-release-1.35: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for k8s-infra-cherrypick-robot:cherry-pick-9160-to-cluster-autoscaler-release-1.35."}],"documentation_url":"https://docs.github.com/rest/pulls/pulls#create-a-pull-request","status":"422"}

Details

In response to this:

/cherry-pick cluster-autoscaler-release-1.35

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.