⬆️ repo-wide upgrade to aiohttp==3.8.5

[pcrespov]

What do these changes do?

Master deploy presents sporadic errors in the web-server logged as

Task exception was never retrieved
future: <Task finished name='Task-262869' coro=<RequestHandler.start() done, defined at /home/scu/.venv/lib/python3.10/site-packages/aiohttp/web_protocol.py:462> exception=AttributeError("'NoneType' object has no attribute 'get_extra_info'")>
Traceback (most recent call last):
  File "/home/scu/.venv/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 505, in start
    request = self._request_factory(message, payload, self, writer, handler)
  File "/home/scu/.venv/lib/python3.10/site-packages/aiohttp/web_app.py", line 446, in _make_request
    return _cls(
  File "/home/scu/.venv/lib/python3.10/site-packages/aiohttp/web_request.py", line 811, in __init__
    super().__init__(*args, **kwargs)
  File "/home/scu/.venv/lib/python3.10/site-packages/aiohttp/web_request.py", line 189, in __init__
    self._transport_sslcontext = transport.get_extra_info("sslcontext")
AttributeError: 'NoneType' object has no attribute 'get_extra_info'

• A reference published in Request.transport is sometimes None aio-libs/aiohttp#3355 mentions a fix in latest release.
• This might also be the origin of the 502 error observed in the e2e. Note that 502 indicates that the server, while acting as a gateway or proxy, received an invalid response from the upstream server.
  • When this PR merges, we should follow up observing the logs in combination with Activate access-logs in traffik in master osparc-ops-environments#290
• NOTE: aiohttp was upgraded in all services except the dynamic-sidecar to avoid conflicts with ongoing replace docker-compose 1.29.1 with docker cli and plugin #4507

Highlights on updated libraries (only updated libraries are included)

• #packages before: 7
• #packages after : 7

#	name	before	after	upgrade	count	packages
1	aiohttp	3.8.4, 3.8.3	3.8.5		13	agent⬆️🧪 api-server⬆️ autoscaling⬆️ catalog⬆️🧪 datcore-adapter⬆️ invitations⬆️ resource-usage-tracker⬆️ storage⬆️🧪 web⬆️🧪
2	click	8.1.3	8.1.6		2	osparc-gateway-server🧪🧪
3	fsspec	2023.5.0	2023.6.0	minor	1	osparc-gateway-server🧪
4	importlib-metadata	6.6.0	6.8.0	minor	1	osparc-gateway-server🧪
5	markupsafe	2.1.2	2.1.3		1	osparc-gateway-server🧪
6	tblib	1.7.0	2.0.0	MAJOR	1	osparc-gateway-server🧪
7	zipp	3.15.0	3.16.2	minor	1	osparc-gateway-server🧪

Legend:

• ⬆️ base dependency (only services because packages are floating)
• 🧪 test dependency
• 🔧 tool dependency

Related issue/s

How to test

DevOps Checklist

[codeclimate]

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

View more on Code Climate.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[codecov]

Codecov Report

Merging #4544 (c914390) into master (dbcc0f4) will decrease coverage by 18.8%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #4544      +/-   ##
=========================================
- Coverage    88.4%   69.6%   -18.8%     
=========================================
  Files         517     265     -252     
  Lines       25472   13294   -12178     
  Branches      530       0     -530     
=========================================
- Hits        22519    9260   -13259     
- Misses       2823    4034    +1211     
+ Partials      130       0     -130     

Flag	Coverage Δ
integrationtests	62.8% <ø> (-5.6%)	⬇️
unittests	64.9% <ø> (-21.5%)	⬇️

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

see 381 files with indirect coverage changes

[matusdrobuliak66]

👍