Skip to content

Conversation

@pcrespov
Copy link
Member

@pcrespov pcrespov commented Jun 4, 2025

What do these changes do?

Create RPC namespaces associated to the docker service, not the image, otherwise the api-server can saturate the web-server

  • wb-api-server should only be communicated by the api-server
  • separate RPC namespace

Related issue/s

How to test

Dev-ops

@pcrespov pcrespov added this to the Bazinga! milestone Jun 4, 2025
@pcrespov pcrespov self-assigned this Jun 4, 2025
@pcrespov pcrespov added the release Preparation for pre-release/release label Jun 4, 2025
@codecov
Copy link

codecov bot commented Jun 4, 2025

Codecov Report

❌ Patch coverage is 91.66667% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.73%. Comparing base (20be01b) to head (f292ba9).
⚠️ Report is 401 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (20be01b) and HEAD (f292ba9). Click for more details.

HEAD has 27 uploads less than BASE
Flag BASE (20be01b) HEAD (f292ba9)
unittests 29 2
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7815       +/-   ##
===========================================
- Coverage   79.17%   58.73%   -20.45%     
===========================================
  Files        1797      315     -1482     
  Lines       69901    14425    -55476     
  Branches     1227      289      -938     
===========================================
- Hits        55346     8472    -46874     
+ Misses      14213     5864     -8349     
+ Partials      342       89      -253     
Flag Coverage Δ *Carryforward flag
integrationtests 53.06% <ø> (ø) Carriedforward from 20be01b
unittests 84.91% <91.66%> (+5.18%) ⬆️

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

Components Coverage Δ
pkg_aws_library ∅ <ø> (∅)
pkg_celery_library ∅ <ø> (∅)
pkg_dask_task_models_library ∅ <ø> (∅)
pkg_models_library ∅ <ø> (∅)
pkg_notifications_library ∅ <ø> (∅)
pkg_postgres_database ∅ <ø> (∅)
pkg_service_integration ∅ <ø> (∅)
pkg_service_library ∅ <ø> (∅)
pkg_settings_library 90.19% <ø> (-0.72%) ⬇️
pkg_simcore_sdk 77.13% <ø> (-7.96%) ⬇️
agent ∅ <ø> (∅)
api_server ∅ <ø> (∅)
autoscaling ∅ <ø> (∅)
catalog ∅ <ø> (∅)
clusters_keeper ∅ <ø> (∅)
dask_sidecar ∅ <ø> (∅)
datcore_adapter ∅ <ø> (∅)
director ∅ <ø> (∅)
director_v2 43.34% <ø> (-7.75%) ⬇️
dynamic_scheduler ∅ <ø> (∅)
dynamic_sidecar 88.34% <ø> (-1.74%) ⬇️
efs_guardian ∅ <ø> (∅)
invitations ∅ <ø> (∅)
payments ∅ <ø> (∅)
resource_usage_tracker ∅ <ø> (∅)
storage ∅ <ø> (∅)
webclient ∅ <ø> (∅)
webserver 4.67% <ø> (-72.10%) ⬇️

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 20be01b...f292ba9. 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.

@pcrespov pcrespov force-pushed the fix/web-server-rpc-namespace branch from 8b3ab57 to 20be01b Compare June 4, 2025 19:38
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 4, 2025

@pcrespov pcrespov removed the release Preparation for pre-release/release label Jun 5, 2025
pcrespov and others added 23 commits July 9, 2025 14:11
…ce decorator to generate opentelemtry profile span (ITISFoundation#8070)

Co-authored-by: Dustin Kaiser <[email protected]>
…` folder (ITISFoundation#8085)

Co-authored-by: Andrei Neagu <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@pcrespov pcrespov modified the milestones: Bazinga!, Cheops Sep 26, 2025
@sonarqubecloud
Copy link

@pcrespov
Copy link
Member Author

Bad PR. redoing changings

@pcrespov pcrespov closed this Sep 26, 2025
@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2025

🧪 CI Insights

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

❌ Job Failures

Pipeline Job Health on master Retries 🔍 CI Insights 📄 Logs
CI integration-tests Healthy 0 View View
system-tests Broken 0 View View
unit-tests Broken 0 View View
PR CI check OAS' are up to date Unknown 0 View View

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants