-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat!: update task status #397
Conversation
f9f8b0b
to
478ab8b
Compare
42e9cdb
to
e21fbf5
Compare
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,frontend,camelyon |
End to end tests: ⚙️ RUNNING It'll just take a minute. |
e21fbf5
to
97527fe
Compare
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,frontend,camelyon |
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,camelyon |
Signed-off-by: SdgJlbl <[email protected]>
97527fe
to
64d40e3
Compare
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,camelyon |
End to end tests: ❌ FAILURE Better luck next time! |
Signed-off-by: Guilhem Barthés <[email protected]>
c08b8ba
to
360ae6a
Compare
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,camelyon |
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status,substra-tests=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,camelyon |
End to end tests: ✔️ SUCCESS |
…` in local worker Signed-off-by: Guilhem Barthés <[email protected]>
/e2e --refs substra-backend=feat/add-new-compute-tasks-status-linked-to-function-status,orchestrator=feat/add-new-compute-tasks-status-linked-to-function-status,substra-tests=feat/add-new-compute-tasks-status-linked-to-function-status --tests=sdk,mnist,camelyon |
End to end tests: ✔️ SUCCESS “To infinity and beyond!” ― Buzz Lightyear, Toy Story |
Signed-off-by: Guilhem Barthés <[email protected]>
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.
LGTM thanks :)
## Companion PR - Substra/substra-backend#823 - Substra/substra-tests#319 - Substra/substra#397 - Substra/substra-frontend#297 - Substra/substra-documentation#390 ## Description <!-- Please reference issue if any. --> <!-- Please include a summary of your changes. --> Fixes of FL-1397 ## How has this been tested? <!-- Please describe the tests that you ran to verify your changes. --> ## Checklist - [x] [changelog](../CHANGELOG.md) was updated with notable changes - [ ] documentation was updated --------- Signed-off-by: Guilhem Barthés <[email protected]>
Companion PRs
Part of FL-1397
Please check if the PR fulfills these requirements