Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Cancel tasks for assigned to a node that re-registers #146

Merged
merged 4 commits into from
Oct 15, 2020
Merged

Cancel tasks for assigned to a node that re-registers #146

merged 4 commits into from
Oct 15, 2020

Conversation

bmc-msft
Copy link
Contributor

Summary of the Pull Request

There are a few cases where a node re-registers while it has work claimed. While those cases should get fixed, this PR fails each of the tasks in these cases.

PR Checklist

  • Applies to work item: #xxx
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@bmc-msft bmc-msft changed the title handle nodes re-registering with existing work Cancel tasks for assigned to a node that re-registers Oct 14, 2020
@bmc-msft bmc-msft merged commit 543dad3 into microsoft:main Oct 15, 2020
@bmc-msft bmc-msft deleted the hand-nodes-re-registering-with-existing-tasks branch October 15, 2020 14:21
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants