opencti icon indicating copy to clipboard operation
opencti copied to clipboard

[backend] Adding info logs to task manager(#8720).

Open aHenryJard opened this issue 1 year ago • 2 comments

Proposed changes

  • Add some info log to task manager for troubleshooting.

Related issues

  • closes #8720

Checklist

  • [ ] I consider the submitted work as finished
  • [ ] I tested the code for its functionality
  • [ ] I wrote test cases for the relevant uses case (coverage and e2e)
  • [ ] I added/update the relevant documentation (either on github or on notion)
  • [ ] Where necessary I refactored code to improve the overall quality

Further comments

aHenryJard avatar Oct 21 '24 15:10 aHenryJard

Codecov Report

Attention: Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 66.30%. Comparing base (c7a29d4) to head (10f6b5a). Report is 27 commits behind head on master.

Files with missing lines Patch % Lines
...latform/opencti-graphql/src/manager/taskManager.js 70.00% 3 Missing :warning:
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8725   +/-   ##
=======================================
  Coverage   66.29%   66.30%           
=======================================
  Files         597      597           
  Lines       60949    60957    +8     
  Branches     6258     6262    +4     
=======================================
+ Hits        40407    40417   +10     
+ Misses      20542    20540    -2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 21 '24 15:10 codecov[bot]

Is it supposed to be temporary ? not too verbose ?

labo-flg avatar Oct 21 '24 16:10 labo-flg

Is it supposed to be temporary ? not too verbose ?

Yes it can be temporary, it's not that verbose since usually in production log level is set to error, and it's move to info for a limited time.

aHenryJard avatar Oct 22 '24 06:10 aHenryJard