Fix Time Drift Calculation

[max-muoto]

Note: Before submitting this pull request, please review our contributing
guidelines.

Description

Fix bug discussed in #8207 but doing the time drift calculation in events in UTC as opposed to local times.

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: +1.99 🎉

Comparison is base (4bcac7a) 85.08% compared to head (6f0b53f) 87.08%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8236      +/-   ##
==========================================
+ Coverage   85.08%   87.08%   +1.99%     
==========================================
  Files         148      148              
  Lines       18455    18470      +15     
  Branches     2428     2525      +97     
==========================================
+ Hits        15703    16085     +382     
+ Misses       2475     2106     -369     
- Partials      277      279       +2     

Flag	Coverage Δ
unittests	87.05% <100.00%> (+1.97%)	⬆️

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

Impacted Files	Coverage Δ
celery/events/event.py	100.00% <100.00%> (ø)
celery/events/receiver.py	94.66% <100.00%> (+0.07%)	⬆️
celery/events/state.py	97.22% <100.00%> (ø)

... and 30 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[max-muoto]

@auvipy Still working on resolving some things here, but will ping you once it's ready.

[auvipy]

I still can't see any fix except the introduction of a newly introduced variable

[max-muoto]

@auvipy Not totally done (also need to add test coverage), but the idea is that non_adjusted_timestamp is the timestamp except it never gets localized with utcoffset.

body['timestamp'] = adjust_timestamp(timestamp, offset) adjusts the timestamp with the provided utcoffset.

We use this new variable in the drift calculation to avoid the issues mentioned here: #8207

[max-muoto]

@auvipy Basic implementation is done, let me know if what I said makes sense. Will add unit tests tomorrow if you think it looks good.

[auvipy]

were you able to fix it?