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

runservice: mark not running tasks as skipped when run marked to stop #219

Conversation

sgotti
Copy link
Member

@sgotti sgotti commented Feb 26, 2020

Currently when a run is marked to stop we are going to stop currently running
tasks and then their childs will be marked as skipped.

But tasks not depending on a stopped task (root task or childs with a finished
parent) that are just waiting for an executor slot, will be scheduled when
there will be a free slot also if the run is marked to stop (and then the
scheduler will stop them after some seconds).

This patch will mark all not started tasks as skipped when the run is marked to
stop.

Currently when a run is marked to stop we are going to stop currently running
tasks and then their childs will be marked as skipped.

But tasks not depending on a stopped task (root task or childs with a finished
parent) that are just waiting for an executor slot, will be scheduled when
there will be a free slot also if the run is marked to stop (and then the
scheduler will stop them after some seconds).

This patch will mark all not started tasks as skipped when the run is marked to
stop.
@sgotti sgotti merged commit e8fdb8d into agola-io:master Feb 27, 2020
tulliobotti64 pushed a commit to tulliobotti64/agola that referenced this pull request Oct 19, 2022
…ng_tasks_skipped_when_run_stop

runservice: mark not running tasks as skipped when run marked to stop
tulliobotti64 pushed a commit to tulliobotti64/agola that referenced this pull request Oct 19, 2022
…ng_tasks_skipped_when_run_stop

runservice: mark not running tasks as skipped when run marked to stop
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.

1 participant