fix: use UTC time in tests

[filariow]

No description provided.

[rajivnathan]

Was there something that prompted this change?

[filariow]

Was there something that prompted this change?

I was not able to understand the reason why, but these tests started failing on my local with both go 1.20.8 and 1.22.2.

[rajivnathan]

I was not able to understand the reason why, but these tests started failing on my local with both go 1.20.8 and 1.22.2.

That's interesting, I just tried without your changes and it's failing for me too. I verified your change resolves it but it would be good to understand why. It looks like it could be related to #432 cc @sbryzak

[filariow]

I think the main reason why we didn't notice this before merging #432 is that the CI instances are running with timezone set to UTC.

[rajivnathan]

Ok looks like this was probably just missed from the original PR. I think the fix is good 👍

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: filariow, rajivnathan

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

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.14%. Comparing base (2d147e4) to head (1cfcd94).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #444   +/-   ##
=======================================
  Coverage   78.14%   78.14%           
=======================================
  Files          41       41           
  Lines        2681     2681           
=======================================
  Hits         2095     2095           
  Misses        491      491           
  Partials       95       95           

Flag	Coverage Δ
unittests	78.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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