Bug #15296

[a-c-r] Handle case of canceling container that is already cancelled.

Added by Peter Amstutz 4 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
06/18/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

When the user hits control-C on a running instance of a-c-r, the shutdown process cancels the container request by updating the request to set priority to 0. If there is a race condition and container request was already final (but a-c-r didn't notice yet), this will raise an exception trying to update a finalized container. This exception should be caught and suppressed (if the request is already finalized it doesn't need to be cancelled).


Subtasks

Task #15327: Review 15296-cwl-cancel-procsResolvedEric Biagiotti

Associated revisions

Revision 5acd6856
Added by Peter Amstutz 3 months ago

Merge branch '15296-cwl-cancel-procs' closes #15296

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

History

#1 Updated by Peter Amstutz 4 months ago

  • Status changed from New to In Progress

#2 Updated by Peter Amstutz 4 months ago

  • Description updated (diff)

#3 Updated by Peter Amstutz 4 months ago

  • Status changed from In Progress to New

#4 Updated by Peter Amstutz 3 months ago

  • Target version changed from 2019-06-05 Sprint to 2019-06-19 Sprint

#5 Updated by Peter Amstutz 3 months ago

  • Status changed from New to In Progress

#6 Updated by Peter Amstutz 3 months ago

  • Assigned To set to Peter Amstutz
  • Status changed from In Progress to New

#7 Updated by Ward Vandewege 3 months ago

  • Release set to 22

#8 Updated by Peter Amstutz 3 months ago

15296-cwl-cancel-procs @ ecc5385565685bb5a4c4b2e1af3afb3ffea30937

  • Cancel all managed processes not just the "runner" process
  • Catch exceptions when setting container priority to 0

#9 Updated by Peter Amstutz 3 months ago

  • Status changed from New to In Progress

#10 Updated by Peter Amstutz 3 months ago

  • Target version changed from 2019-06-19 Sprint to 2019-07-03 Sprint

#11 Updated by Peter Amstutz 3 months ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved

Also available in: Atom PDF