Feature #15345

[arvados-dispatch-cloud] kill container (management API)

Added by Tom Clegg over 1 year ago. Updated 10 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Tom Clegg
Category:
-
Target version:
2019-07-03 Sprint
Start date:
06/19/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
1.0
Release:
Arvados 2.0 - Early 2020
Release relationship:
Auto

Description

Described at Dispatching containers to cloud VMs:

POST /arvados/v1/dispatch/containers/kill?container_uuid=X terminates a container immediately.
  • a single attempt is made to send SIGTERM to the container
  • container state/priority fields are not affected
  • assuming SIGTERM works, the container record will end up with state "Cancelled"

Currently (without this API) an admin can use the controller API to update the container's priority to 0, with similar results. This API is a bit more direct: it works even if the dispatcher isn't syncing with the controller properly/quickly, and it bypasses the TimeoutSignal config that rate-limits SIGTERM attempts per container.

Subtasks

Task #15384: Review 15345-kill-container-apiResolvedEric Biagiotti

Related issues

Blocks Arvados - Story #13908: [Epic] Replace SLURM for cloud job scheduling/dispatchingResolved

Associated revisions

Revision 1519b9e7
Added by Tom Clegg over 1 year ago

Merge branch '15345-kill-container-api'

closes #15345

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

History

#1 Updated by Tom Clegg over 1 year ago

  • Blocks Story #13908: [Epic] Replace SLURM for cloud job scheduling/dispatching added

#2 Updated by Tom Morris over 1 year ago

  • Target version changed from To Be Groomed to Arvados Future Sprints
  • Story points set to 1.0

#3 Updated by Tom Clegg over 1 year ago

  • Status changed from New to In Progress
  • Assigned To set to Tom Clegg

#4 Updated by Tom Clegg over 1 year ago

  • Description updated (diff)

#6 Updated by Tom Morris over 1 year ago

  • Target version changed from Arvados Future Sprints to 2019-07-03 Sprint

#7 Updated by Eric Biagiotti over 1 year ago

Tom Clegg wrote:

15345-kill-container-api @ 49717fb59156c2b276ccc2fde0b9f2de71e812a6 -- https://ci.curoverse.com/view/Developer/job/developer-run-tests/1326/

This LGTM, if the tests are rerun and they pass. Thanks!

#9 Updated by Tom Clegg over 1 year ago

  • Status changed from In Progress to Resolved

#10 Updated by Peter Amstutz 10 months ago

  • Release set to 22

Also available in: Atom PDF