Project

General

Profile

Actions
Copy link

Idea #13501

closed

[arvados-cwl-runner] [arv-keepdocker] Use subprocess32 instead of subprocess

Added by Thomas Hickman almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Lucas Di Pentima
Category:
-
Target version:
2018-07-18 Sprint
Start date:
07/09/2018
Due date:
Story points:
0.5
Release:
Arvados 1.2
Release relationship:
Auto

Description

Google has created a backport of python3.3's subprocess module to python2, which includes includes "many important reliability bug fixes relevant on POSIX platforms" (see https://github.com/google/python-subprocess32) -- including making it safer to use with multi-threaded programs. It would be good to use this instead of the native python2 subprocess module.
  • cwltool already uses subprocess32
  • arvados-cwl-runner uses native subprocess
  • arv-keepdocker uses native subprocess
  • nodemanager uses native subprocess

Subtasks 1 (0 open — 1 closed)

Task #13725: Review 13501-subprocess32ResolvedPeter Amstutz07/09/2018Actions
Actions
Copy link

Also available in: Atom PDF