-
Notifications
You must be signed in to change notification settings - Fork 27
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
1848 add permission rights to async jobs #7262
1848 add permission rights to async jobs #7262
Conversation
…low generates latest version
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7262 +/- ##
==========================================
- Coverage 87.11% 86.68% -0.43%
==========================================
Files 1689 1527 -162
Lines 65523 62450 -3073
Branches 1119 898 -221
==========================================
- Hits 57081 54136 -2945
+ Misses 8124 8054 -70
+ Partials 318 260 -58
Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx!
packages/models-library/src/models_library/api_schemas_rpc_async_jobs/async_jobs.py
Outdated
Show resolved
Hide resolved
packages/models-library/src/models_library/api_schemas_rpc_async_jobs/async_jobs.py
Outdated
Show resolved
Hide resolved
services/storage/src/simcore_service_storage/api/rpc/_data_export.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good! thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
…:bisgaard-itis/osparc-simcore into 1848-add-permission-rights-to-async-jobs
|
What do these changes do?
storage
before submitting job to workerAsyncJobId
fromuuid.UUID
tostr
in order to support "embedding" metadata into task nameRelated issue/s
How to test
Dev-ops checklist