-
Notifications
You must be signed in to change notification settings - Fork 60
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(dal,sdf): Close and Put On hold actions v2 endpoints #3764
feat(dal,sdf): Close and Put On hold actions v2 endpoints #3764
Conversation
vbustamante
commented
May 10, 2024
•
edited
edited
- Close actions by deleting them from the graph
- put actions on hold by updating the status
- send all actions to frontend instead of just enqueued ones
- fix some issues in the actions_v2 tests
lib/dal/src/schema/variant.rs
Outdated
workspace_snapshot.get_node_weight(prototype_idx).await? | ||
else { | ||
continue; | ||
}; |
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.
This was breaking component creation when they had both new and old creation actions
| ActionState::Failed | ||
| ActionState::OnHold => return Err(ActionError::InvalidOnHoldTransition(request.id)), | ||
ActionState::Queued => {} | ||
} |
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.
Likewise for these ones - are these conditions correct?
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.
I think putting a Failed action on hold makes sense. It would effectively be re-queueing it immediately into the On Hold state.
9a3e7e7
to
7493a90
Compare
Signed-off-by: Victor Bustamante <victor@systeminit.com>
99db381
to
41659d9
Compare
41659d9
to
8f698e5
Compare
982e67e
to
e62f7fe
Compare
bors try |
tryBuild failed: |
e62f7fe
to
4f8127d
Compare
…ng Actions in rebaser
bors try |
tryBuild failed: |
bors merge |
🕐 Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set. |
Build succeeded: |