Maintenance/ lgtm+CodeQL fixes

[pcrespov]

What do these changes do?

Does some fixes to keep up with issues found by LGTM which is now integrated in github's CodeQL and runs on our repo every Monday morning

• In order to keep the PR reasonably short and contained, we have done some fixes in the following:
  • packages/postgres-database
  • packages/service-library
  • services/catalog
  • services/dask-sidecar
  • services/datcore-adapter
  • storage
• See commits for description of changes.

Related issue/s

• Part of osparc-issues/issues/675

How to test

Checklist

[codecov]

Codecov Report

Merging #3363 (dc702f2) into master (ddecdef) will increase coverage by 0.0%.
The diff coverage is 93.7%.

@@          Coverage Diff           @@
##           master   #3363   +/-   ##
======================================
  Coverage    82.9%   82.9%           
======================================
  Files         807     807           
  Lines       34197   34187   -10     
  Branches     1358    1358           
======================================
- Hits        28376   28369    -7     
+ Misses       5639    5636    -3     
  Partials      182     182           

Flag	Coverage Δ
integrationtests	68.3% <100.0%> (+<0.1%)	⬆️
unittests	79.5% <93.7%> (-0.1%)	⬇️

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

Impacted Files	Coverage Δ
...se/src/simcore_postgres_database/sidecar_models.py	0.0% <0.0%> (ø)
...se/src/simcore_postgres_database/storage_models.py	100.0% <ø> (ø)
...tion/src/service_integration/compose_spec_model.py	100.0% <ø> (ø)
...rc/servicelib/fastapi/long_running_tasks/client.py	100.0% <ø> (ø)
...e/src/simcore_service_storage/db_file_meta_data.py	98.1% <ø> (ø)
...rvice_datcore_adapter/utils/requests_decorators.py	83.7% <66.6%> (-0.5%)	⬇️
...database/src/simcore_postgres_database/__init__.py	100.0% <100.0%> (ø)
...s-database/src/simcore_postgres_database/errors.py	100.0% <100.0%> (ø)
...e/src/simcore_postgres_database/models/__init__.py	100.0% <100.0%> (ø)
...e/src/simcore_postgres_database/utils_aiopg_orm.py	84.4% <100.0%> (ø)
... and 23 more

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[sanderegg]

not sure what lgtm does.. removes the from asyncio import... when there is already a import asyncio is that so?

[pcrespov]

not sure what lgtm does.. removes the from asyncio import... when there is already a import asyncio is that so?

@sanderegg it is a style rule that I think it's reasonable: https://lgtm.com/rules/1818040193/