Skip to content

🎨 Require parent info headers when running function #7900

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bisgaard-itis
Copy link
Contributor

What do these changes do?

  • Require that parent info headers are specified when running function

Related issue/s

How to test

Dev-ops

@bisgaard-itis bisgaard-itis requested a review from pcrespov as a code owner June 13, 2025 14:39
@bisgaard-itis bisgaard-itis marked this pull request as draft June 13, 2025 14:39
@bisgaard-itis bisgaard-itis self-assigned this Jun 13, 2025
@bisgaard-itis bisgaard-itis added this to the Engage milestone Jun 13, 2025
Copy link

codecov bot commented Jun 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.62%. Comparing base (9705882) to head (9abfff6).
Report is 5 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (9705882) and HEAD (9abfff6). Click for more details.

HEAD has 31 uploads less than BASE
Flag BASE (9705882) HEAD (9abfff6)
unittests 32 1
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7900       +/-   ##
===========================================
- Coverage   87.98%   69.62%   -18.37%     
===========================================
  Files        1844      795     -1049     
  Lines       71048    36421    -34627     
  Branches     1220      176     -1044     
===========================================
- Hits        62514    25357    -37157     
- Misses       8182    11006     +2824     
+ Partials      352       58      -294     
Flag Coverage Δ
integrationtests 64.31% <ø> (-0.01%) ⬇️
unittests 92.15% <100.00%> (+5.58%) ⬆️
Components Coverage Δ
api ∅ <ø> (∅)
pkg_aws_library ∅ <ø> (∅)
pkg_dask_task_models_library ∅ <ø> (∅)
pkg_models_library ∅ <ø> (∅)
pkg_notifications_library ∅ <ø> (∅)
pkg_postgres_database ∅ <ø> (∅)
pkg_service_integration ∅ <ø> (∅)
pkg_service_library ∅ <ø> (∅)
pkg_settings_library ∅ <ø> (∅)
pkg_simcore_sdk 77.16% <ø> (-7.89%) ⬇️
agent ∅ <ø> (∅)
api_server 92.15% <100.00%> (+0.39%) ⬆️
autoscaling ∅ <ø> (∅)
catalog ∅ <ø> (∅)
clusters_keeper ∅ <ø> (∅)
dask_sidecar ∅ <ø> (∅)
datcore_adapter ∅ <ø> (∅)
director ∅ <ø> (∅)
director_v2 77.69% <ø> (-13.41%) ⬇️
dynamic_scheduler ∅ <ø> (∅)
dynamic_sidecar 88.33% <ø> (-1.77%) ⬇️
efs_guardian ∅ <ø> (∅)
invitations ∅ <ø> (∅)
payments ∅ <ø> (∅)
resource_usage_tracker ∅ <ø> (∅)
storage ∅ <ø> (∅)
webclient ∅ <ø> (∅)
webserver 59.00% <ø> (-28.63%) ⬇️

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 9705882...9abfff6. 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.

Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wire parent information through function api
2 participants