♻️ Maintenance: removing/adding EFS env vars ⚠️

[matusdrobuliak66]

What do these changes do?

• removing EFS_ONLY_ENABLED_FOR_USERIDS which is not used anymore
• adding EFS_DEFAULT_USER_SERVICE_SIZE_BYTES to the docker-compose (that was manually added in the production deployment)

Related issue/s

• ⚠️ BREAKING CHANGE: https://git.speag.com/oSparc/osparc-ops-deployment-configuration/-/merge_requests/1115 MERGE AFTER this PR is in production

How to test

Dev-ops checklist

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.41%. Comparing base (dc35757) to head (884d35f).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6837      +/-   ##
==========================================
+ Coverage   88.38%   88.41%   +0.02%     
==========================================
  Files        1561     1554       -7     
  Lines       62396    62209     -187     
  Branches     2108     2164      +56     
==========================================
- Hits        55151    54999     -152     
+ Misses       6914     6880      -34     
+ Partials      331      330       -1     

Flag	Coverage Δ		*Carryforward flag
integrationtests	64.71% <ø> (-3.63%)	⬇️	Carriedforward from 6fc9da1
unittests	86.43% <ø> (+0.10%)	⬆️

*This pull request uses carry forward flags. Click here to find out more.

Components	Coverage Δ
api	∅ <ø> (∅)
pkg_aws_library	93.49% <ø> (ø)
pkg_dask_task_models_library	97.09% <ø> (ø)
pkg_models_library	91.25% <ø> (ø)
pkg_notifications_library	84.57% <ø> (ø)
pkg_postgres_database	87.76% <ø> (ø)
pkg_service_integration	70.00% <ø> (ø)
pkg_service_library	76.13% <ø> (+0.06%)	⬆️
pkg_settings_library	90.60% <ø> (-0.02%)	⬇️
pkg_simcore_sdk	85.36% <ø> (-0.03%)	⬇️
agent	97.00% <ø> (ø)
api_server	89.72% <ø> (ø)
autoscaling	95.21% <ø> (ø)
catalog	90.57% <ø> (ø)
clusters_keeper	98.72% <ø> (ø)
dask_sidecar	91.26% <ø> (ø)
datcore_adapter	93.17% <ø> (ø)
director	76.40% <ø> (ø)
director_v2	90.67% <ø> (-0.07%)	⬇️
dynamic_scheduler	96.59% <ø> (ø)
dynamic_sidecar	89.75% <ø> (ø)
efs_guardian	90.12% <ø> (ø)
invitations	93.44% <ø> (ø)
osparc_gateway_server	85.49% <ø> (ø)
payments	92.77% <ø> (ø)
resource_usage_tracker	90.71% <ø> (+0.07%)	⬆️
storage	89.66% <ø> (ø)
webclient	∅ <ø> (∅)
webserver	88.65% <ø> (-0.03%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc35757...884d35f. Read the comment docs.

[sanderegg]

don't forget corresponding MRs

[YuryHrytsuk]

Q:

Why we should merge config PR after this MR is in PROD and not before? This is usually the other way around

@matusdrobuliak66

[matusdrobuliak66]

Q:

Why we should merge config PR after this MR is in PROD and not before? This is usually the other way around

@matusdrobuliak66

Because it is a breaking change if we merge it now. This is the easiest way how to get rid of those vars and do not break anything.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud