Skip to content

WIP: ✨ Is7092/list study jobs #7739

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 7 commits into
base: master
Choose a base branch
from

Conversation

pcrespov
Copy link
Member

What do these changes do?

Extend StudyService with functions that communicate with web-server.

  • create_job
  • get_study_job
  • delete_study_job

Related issue/s

How to test

Dev-ops

@pcrespov pcrespov self-assigned this May 22, 2025
@pcrespov pcrespov added the a:apiserver api-server service label May 22, 2025
Copy link

codecov bot commented May 22, 2025

Codecov Report

Attention: Patch coverage is 94.11765% with 2 lines in your changes missing coverage. Please review.

Project coverage is 67.94%. Comparing base (a0d9481) to head (2c7857c).

❗ There is a different number of reports uploaded between BASE (a0d9481) and HEAD (2c7857c). Click for more details.

HEAD has 31 uploads less than BASE
Flag BASE (a0d9481) HEAD (2c7857c)
unittests 32 1
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7739       +/-   ##
===========================================
- Coverage   86.81%   67.94%   -18.87%     
===========================================
  Files        1841      804     -1037     
  Lines       71465    37048    -34417     
  Branches     1214      174     -1040     
===========================================
- Hits        62044    25174    -36870     
- Misses       9078    11816     +2738     
+ Partials      343       58      -285     
Flag Coverage Δ
integrationtests 64.37% <ø> (+0.01%) ⬆️
unittests 91.89% <94.11%> (+5.31%) ⬆️
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 76.83% <ø> (-8.06%) ⬇️
agent ∅ <ø> (∅)
api_server 91.89% <94.11%> (+0.09%) ⬆️
autoscaling ∅ <ø> (∅)
catalog ∅ <ø> (∅)
clusters_keeper ∅ <ø> (∅)
dask_sidecar ∅ <ø> (∅)
datcore_adapter ∅ <ø> (∅)
director ∅ <ø> (∅)
director_v2 77.67% <ø> (-13.40%) ⬇️
dynamic_scheduler ∅ <ø> (∅)
dynamic_sidecar 88.42% <ø> (-1.75%) ⬇️
efs_guardian ∅ <ø> (∅)
invitations ∅ <ø> (∅)
payments ∅ <ø> (∅)
resource_usage_tracker ∅ <ø> (∅)
storage ∅ <ø> (∅)
webclient ∅ <ø> (∅)
webserver 56.63% <ø> (-27.32%) ⬇️

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 a0d9481...2c7857c. 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 added this to the Bazinga! milestone May 23, 2025
@pcrespov pcrespov force-pushed the is7092/list-study-jobs branch from e9f6427 to 2c7857c Compare May 26, 2025 17:28
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:apiserver api-server service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GET /v0/studies/{study_id}/jobs/{job_id} GET /v0/studies/{study_id}/jobs
1 participant