Signed-off-by: Seongmin Lee git@boltless.me
lexicons,appview,spindle: add workflow cancel #889
STACK 4
TANGLED_PIPELINE_ID
expand 1 commit
hide 1 commit
Signed-off-by: Seongmin Lee <git@boltless.me>
expand 1 commit
hide 1 commit
Signed-off-by: Seongmin Lee <git@boltless.me>
can we allow owners to also cancel pipelines?
sorry, i mean collaborators.
@oppi.li I'm not sure how should I implement that. We don't have repo:pipeline action defined for repo owners and contributors.
For example, setting access is controlled by repo:settings.
Should I just check for both repo:owner and repo:contributor as a dirty fix?
can we allow collaborators to also cancel pipelines?
@oppi.li I think we can merge this for now. Spindle's collaborator ingesting logic is currently broken so allowing them to cancel pipelines will reveal the bug. I'll make it possible on upcoming PR I'm working on sl/spindle-rewrite branch.
reviews:
- here, we should use
config.Core.Dev - here, don't set expiry, the default is already 60s into the future
- there is no auth filter for the cancel button on the appview side, the button should only be visible to repo owners + collaborators, and the
/workflow/cancelendpoint should return 403 for non-collaborators and non-owners. - here, the error message needs to be "Failed to cancel workflow", not "Failed to add secret"
the rest of the code is brilliant, will give this a test locally as well!
expand 1 commit
hide 1 commit
Signed-off-by: Seongmin Lee <git@boltless.me>
expand 1 commit
hide 1 commit
Signed-off-by: Seongmin Lee <git@boltless.me>
expand 1 commit
hide 1 commit
Signed-off-by: Seongmin Lee <git@boltless.me>
expand 1 commit
hide 1 commit
Signed-off-by: Seongmin Lee <git@boltless.me>
None yet.
None yet.
requesting a review @oppi.li