Skip to content
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

allow pending job proposals to be revoked #8657

Merged
merged 2 commits into from
Mar 15, 2023
Merged

Conversation

eutopian
Copy link
Contributor

@eutopian eutopian commented Mar 8, 2023

This change will allow FMS to send revoke job requests to the node operators. Before a job has been approved by a node operator, FMS can send a revoke request. The revoked job proposal will no longer be approvable, but it can be reproposed with a new pending job spec.

@eutopian eutopian force-pushed the FMS-48/revoke-jobs branch from 5f53dba to 77409e4 Compare March 8, 2023 21:47
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

I see that you haven't updated any CHANGELOG files. Would it make sense to do so?

@eutopian eutopian force-pushed the FMS-48/revoke-jobs branch 2 times, most recently from e204067 to facb152 Compare March 8, 2023 21:54
@eutopian eutopian marked this pull request as ready for review March 8, 2023 22:01
@eutopian eutopian force-pushed the FMS-48/revoke-jobs branch 5 times, most recently from 6cee084 to b3d7eb2 Compare March 9, 2023 14:01
@eutopian eutopian force-pushed the FMS-48/revoke-jobs branch from b3d7eb2 to 71f163e Compare March 9, 2023 14:23
@cl-sonarqube-production
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 89.9% 89.9% Coverage on New Code (is less than 90%)

See analysis details on SonarQube

@eutopian eutopian merged commit 0df25a4 into develop Mar 15, 2023
@eutopian eutopian deleted the FMS-48/revoke-jobs branch March 15, 2023 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants