🐛 fix dynamic-sidecar preferences save and before shutdown event ordering

[GitHK]

What do these changes do?

User services rely on the before_shutdown hook to move the preferences in the correct folder. Saving them AFTER the hook runs makes more sense.

Related issue/s

How to test

Dev Checklist

• No ENV changes or I properly updated ENV (read the instruction)

DevOps Checklist

[mguidon]

Thanks

[sanderegg]

👍

[codeclimate]

Code Climate has analyzed commit c4b59ce and detected 0 issues on this pull request.

View more on Code Climate.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.1% Duplication

[codecov]

Codecov Report

Merging #5156 (c4b59ce) into master (df8953b) will increase coverage by 26.9%.
The diff coverage is 66.6%.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5156      +/-   ##
=========================================
+ Coverage    60.9%   87.8%   +26.9%     
=========================================
  Files         204      56     -148     
  Lines       11026    2760    -8266     
  Branches      198       0     -198     
=========================================
- Hits         6720    2425    -4295     
+ Misses       4256     335    -3921     
+ Partials       50       0      -50     

Flag	Coverage Δ
integrationtests	?
unittests	87.8% <66.6%> (?)

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

Files	Coverage Δ
...vice_dynamic_sidecar/modules/long_running_tasks.py	95.2% <66.6%> (+60.7%)	⬆️

... and 186 files with indirect coverage changes