Bug #17015

crunch-dispatch-slurm picks up containers for wrong cluster

Added by Peter Amstutz 6 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
11/19/2020
Due date:
% Done:

100%

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

Description

If a user has multiple arvados clusters using the same backend slurm cluster, crunch-dispatch-slurm will attempt to manage containers submitted by a different dispatcher. Even if it doesn't succeed it will create noisy errors in the log.


Subtasks

Task #17028: Review 17015-cds-correct-containersResolvedPeter Amstutz

Associated revisions

Revision f8b5741b
Added by Peter Amstutz 5 months ago

Merge branch '17015-cds-correct-containers' refs #17015

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

History

#1 Updated by Peter Amstutz 6 months ago

  • Description updated (diff)

#2 Updated by Peter Amstutz 6 months ago

  • Target version changed from 2020-11-18 to 2020-11-04 Sprint

#3 Updated by Peter Amstutz 6 months ago

  • Category set to Crunch

#4 Updated by Peter Amstutz 6 months ago

  • Assigned To set to Peter Amstutz

#5 Updated by Peter Amstutz 6 months ago

  • Target version changed from 2020-11-04 Sprint to 2020-11-18

#6 Updated by Peter Amstutz 5 months ago

  • Target version changed from 2020-11-18 to 2020-12-02 Sprint

#7 Updated by Peter Amstutz 5 months ago

  • Story points set to 1.0

#8 Updated by Peter Amstutz 5 months ago

17015-cds-correct-containers @ a555df85fe5647cfd83ee3f8d1b254f97fb77a18

Ignore jobs that don't have our UUID prefix.

I tried to write a test for this but after spending an hour fiddling with it the amount of work in refactoring and mocking required to confirm a simple 1 line fix didn't seem like a good use of time.

https://ci.arvados.org/view/Developer/job/developer-run-tests/2187/

#9 Updated by Lucas Di Pentima 5 months ago

This LGTM, please merge. Thanks!

#10 Updated by Peter Amstutz 5 months ago

  • Status changed from New to Resolved

#11 Updated by Peter Amstutz 5 months ago

  • Release set to 37

Also available in: Atom PDF