⬆️🐛Maintenance/fix e2e flakyness

[sanderegg]

What do these changes do?

• removes e2e tests from CI (puppeteer)
• tests are migrated into:
  • tests/e2e-playwright/tests/platform_CI_tests/test_after_logging_in.py (FI @odeimaiz )
  • tests/e2e-playwright/tests/platform_CI_tests/test_metadata.py (FI @odeimaiz )
• inverted startup phase of up-prod, up-devel, to first start the ops stack and then the simcore stack to improve slightly the performance (as the simcore stack depends on the ops stack)
• refactored the wait_for_services utility function to look nicer and produce more useful logs
• fixed warnings during installation of playwright
• upgraded simcore Traefik to 3.6.1, which is compatible with docker>29.X (FI @YuryHrytsuk @mrnicegyu11 )
• increased the start period of the migration service as it is slow in the CI it gets killed half way.

For instance setup in the e2e-playwright test now looks like this:
As a consequence, I guess we should pre-pull postgres/redis/rabbit before starting the stack...

Related issue/s

How to test

Dev-ops

[mergify]

🧪 CI Insights

Here's what we observed from your CI run for 58cee3b.

✅ Passed Jobs With Interesting Signals

Pipeline	Job	Signal	Health on master	Retries	🔍 CI Insights	📄 Logs
CI	integration-tests	Base branch is broken, but the job passed. Looks like this might be a real fix 💪		0	View	View
	unit-tests	Base branch is broken, but the job passed. Looks like this might be a real fix 💪		0	View	View

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.52%. Comparing base (daf0b87) to head (58cee3b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #8605   +/-   ##
=======================================
  Coverage   87.51%   87.52%           
=======================================
  Files        2010     2006    -4     
  Lines       78979    78895   -84     
  Branches     1378     1378           
=======================================
- Hits        69122    69053   -69     
+ Misses       9453     9438   -15     
  Partials      404      404           

Flag	Coverage Δ
integrationtests	63.88% <ø> (-0.07%)	⬇️
unittests	86.36% <ø> (+0.01%)	⬆️

Components	Coverage Δ
pkg_aws_library	95.30% <ø> (ø)
pkg_celery_library	83.43% <ø> (ø)
pkg_dask_task_models_library	79.37% <ø> (ø)
pkg_models_library	92.83% <ø> (ø)
pkg_notifications_library	85.20% <ø> (ø)
pkg_postgres_database	88.03% <ø> (ø)
pkg_service_integration	72.76% <ø> (ø)
pkg_service_library	71.30% <ø> (ø)
pkg_settings_library	90.21% <ø> (ø)
pkg_simcore_sdk	84.58% <ø> (ø)
agent	93.44% <ø> (ø)
api_server	91.37% <ø> (ø)
autoscaling	95.83% <ø> (ø)
catalog	92.06% <ø> (ø)
clusters_keeper	99.14% <ø> (ø)
dask_sidecar	91.72% <ø> (ø)
datcore_adapter	97.95% <ø> (ø)
director	75.72% <ø> (ø)
director_v2	91.21% <ø> (-0.09%)	⬇️
dynamic_scheduler	96.66% <ø> (ø)
dynamic_sidecar	90.72% <ø> (-0.11%)	⬇️
efs_guardian	89.83% <ø> (ø)
invitations	90.90% <ø> (ø)
payments	92.70% <ø> (ø)
resource_usage_tracker	92.27% <ø> (+0.26%)	⬆️
storage	86.62% <ø> (+0.16%)	⬆️
webclient	∅ <ø> (∅)
webserver	86.69% <ø> (+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 daf0b87...58cee3b. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sanderegg]

@mergify queue

[mergify]

queue

🛑 Configuration not compatible with a branch protection setting

The branch protection setting Require branches to be up to date before merging is not compatible with draft PR checks. To keep this branch protection enabled, update your Mergify configuration to enable in-place checks: set merge_queue.max_parallel_checks: 1, set every queue rule batch_size: 1, and avoid two-step CI (make merge_conditions identical to queue_conditions). Otherwise, disable this branch protection.

[pcrespov]

thx

[GitHK]

Just some questions

[odeimaiz]

thanks

[mrnicegyu11]

I'll need to consider the traefik upgrade briefly before approving, that could break things, jfyi

[mrnicegyu11]

tldr, but checked the traefik upgrade, this is fine w.r.t. the migration guide from the traefik-developers. 🪨 🚀 thanks a lot

[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