Project

General

Profile

Actions

Bug #17015

closed

crunch-dispatch-slurm picks up containers for wrong cluster

Added by Peter Amstutz about 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
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 1 (0 open1 closed)

Task #17028: Review 17015-cds-correct-containersResolvedPeter Amstutz11/19/2020Actions
Actions #1

Updated by Peter Amstutz about 4 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz about 4 years ago

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

Updated by Peter Amstutz about 4 years ago

  • Category set to Crunch
Actions #4

Updated by Peter Amstutz about 4 years ago

  • Assigned To set to Peter Amstutz
Actions #5

Updated by Peter Amstutz about 4 years ago

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

Updated by Peter Amstutz about 4 years ago

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

Updated by Peter Amstutz about 4 years ago

  • Story points set to 1.0
Actions #8

Updated by Peter Amstutz about 4 years 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.

developer-run-tests: #2187

Actions #9

Updated by Lucas Di Pentima about 4 years ago

This LGTM, please merge. Thanks!

Actions #10

Updated by Peter Amstutz about 4 years ago

  • Status changed from New to Resolved
Actions #11

Updated by Peter Amstutz almost 4 years ago

  • Release set to 37
Actions

Also available in: Atom PDF