fix linter errors

[nawazkh]

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Gist: #5466 (comment)

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 #

Special notes for your reviewer:

TODOs:

• squashed commits
• includes documentation
• adds unit tests
• cherry-pick candidate

Release note:

NONE

[nawazkh]

I wonder why the verify job did not catch this in #5452 ?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.56%. Comparing base (057907a) to head (7fdbf76).
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5466   +/-   ##
=======================================
  Coverage   52.55%   52.56%           
=======================================
  Files         272      272           
  Lines       29423    29423           
=======================================
+ Hits        15463    15465    +2     
+ Misses      13153    13151    -2     
  Partials      807      807           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nawazkh]

I wonder why the verify job did not catch this in #5452 ?

Looks like none of the GitHub Actions ran on #5452. Strange. Is it because the PR was from a non-maintainer ? Probing..

[nawazkh]

I wonder why the verify job did not catch this in #5452 ?

Looks like none of the GitHub Actions ran on #5452. Strange. Is it because the PR was from a non-maintainer ? Probing..

Looks like the setting "Approval for running fork pull request workflows from contributors" was set to "needs approval for all external contributors". I changed it to "need approval for first timers contributing to this repo" so that we don't miss out on running the GitHub Actions from external contributors who have already committed to this repository.

[willie-yao]

/lgtm
/approve

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: c5e1a4866b5f067bba6eba5b48e60c599f9b63bc

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: willie-yao

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 [willie-yao]

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