This pull request fixes 2 alerts when merging 359bb27 into ebeb4a4 - view on LGTM.com

fixed alerts:

• 2 for Wrong name for an argument in a call

Are there instructions on how to run the tests locally?

Sorry for the delay. I still haven't fixed my local setup to work with the GitHub password changes, so I couldn't push changes and just ended up doing it through the web UI, so hopefully the tests will pass now.

Codecov Report

Merging #6898 (574e0ca) into master (ebeb4a4) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #6898      +/-   ##
==========================================
- Coverage   89.23%   89.20%   -0.03%     
==========================================
  Files         138      138              
  Lines       16626    16654      +28     
  Branches     2099     2106       +7     
==========================================
+ Hits        14836    14857      +21     
- Misses       1571     1577       +6     
- Partials      219      220       +1     

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

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ebeb4a4...574e0ca. Read the comment docs.

This pull request fixes 2 alerts when merging 574e0ca into 2ac3310 - view on LGTM.com

fixed alerts:

• 2 for Wrong name for an argument in a call

hey @daveisfera is there a way to get rid of args n kwargs from getting logged ?

Should be possible by customizing the logger config. I haven't done that myself (because I wanted the args and kwargs to be logged), but it should be possible

@daveisfera @thedrow @auvipy (and all) - apologies if this is the wrong place to post this. In celery 4.x the "received" logs will show the task's ETA if available. Example:
INFO Received task: exampleTaskName[8b68c9be-d3c3-4352-b3bc-8749ed5a5c0c] ETA:[2022-10-11 21:09:20.919008+00:00]. However 5.x is no longer there. I'm seeing that you have to add eta: req.eta to the context dict, and also overwrite LOG_RECEIVED in my app. Is there another way? Should this be a request in the celery repo. Thanks for the help!

@daveisfera @thedrow @auvipy (and all) - apologies if this is the wrong place to post this. In celery 4.x the "received" logs will show the task's ETA if available. Example: INFO Received task: exampleTaskName[8b68c9be-d3c3-4352-b3bc-8749ed5a5c0c] ETA:[2022-10-11 21:09:20.919008+00:00]. However 5.x is no longer there. I'm seeing that you have to add eta: req.eta to the context dict, and also overwrite LOG_RECEIVED in my app. Is there another way? Should this be a request in the celery repo. Thanks for the help!

we got a new PR here #8301 please check