♻️⬆️ Maintenance/removing pydantic constraints

[matusdrobuliak66]

What do these changes do?

• ♻️ Removing pydantic constraints
• ⬆️ upgrade test & tools dependencies of pydantic

Highlights on updated libraries

• #packages before: 4
• #packages after : 1

#	name	before	after	upgrade	count	packages
1	importlib-metadata	6.0.0	🗑️ removed		4	agent🧪 autoscaling🧪 simcore-sdk🧪 storage🧪
2	pydantic	1.10.2	1.10.7		9	dask-task-models-library🧪 environment-setup🧪 models-library🧪 service-integration🧪 service-library🧪🧪 settings-library🧪 simcore-sdk🧪 swarm-deploy🧪
3	typing-extensions	4.5.0	🗑️ removed		1	simcore-sdk🧪
4	zipp	3.15.0	🗑️ removed		4	agent🧪 autoscaling🧪 simcore-sdk🧪 storage🧪

Related issue/s

• fixes Breaking change after pydantic version upgrade #3905
• Upgrade to python 3.10 osparc-issues#877
• Project Workbench is ignored in json schema validation #3992

How to test

[codeclimate]

The PR diff size of 12008 lines exceeds the maximum allowed for the inline comments feature.

[codecov]

Codecov Report

Merging #4065 (b3fce4c) into master (11cfff5) will increase coverage by 2.4%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #4065      +/-   ##
=========================================
+ Coverage    85.4%   87.9%    +2.4%     
=========================================
  Files         951     500     -451     
  Lines       41262   25863   -15399     
  Branches      957     288     -669     
=========================================
- Hits        35260   22741   -12519     
+ Misses       5783    3054    -2729     
+ Partials      219      68     -151     

Flag	Coverage Δ
integrationtests	66.6% <100.0%> (-0.1%)	⬇️
unittests	84.0% <100.0%> (+1.7%)	⬆️

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

Impacted Files	Coverage Δ
...mcore_service_webserver/projects/project_models.py	96.6% <100.0%> (+0.2%)	⬆️

... and 467 files with indirect coverage changes

[codeclimate]

The PR diff size of 12024 lines exceeds the maximum allowed for the inline comments feature.

[codeclimate]

The PR diff size of 12024 lines exceeds the maximum allowed for the inline comments feature.

[codeclimate]

The PR diff size of 12346 lines exceeds the maximum allowed for the inline comments feature.

[pcrespov]

Thx a lot for removing this constraint!

As agreed, please let #4066 get merge in master first.

[codeclimate]

The PR diff size of 12279 lines exceeds the maximum allowed for the inline comments feature.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[codeclimate]

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

View more on Code Climate.

[GitHK]

I don't understand why we strip the patternProperties and additionalProperties form some placed but add them elsewhere.

[matusdrobuliak66]

I don't understand why we strip the patternProperties and additionalProperties form some placed but add them elsewhere.

These are two different things:

1. why do we strip them?

• Project Workbench is ignored in json schema validation #3992
• so after we fix this, and refactor webserver we would add them back

2. why do we add them?

• Pydantic v 1.0.3 introduced additionalProperties when you create json schema/open api specs from Pydantic model

[GitHK]

👍