← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1286187] Re: plugins/xenserver/networking/etc/xensource/scripts/novalib.py uses subprocess incorrectly

 

** Changed in: nova
       Status: Fix Committed => Fix Released

** Changed in: nova
    Milestone: None => icehouse-3

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1286187

Title:
  plugins/xenserver/networking/etc/xensource/scripts/novalib.py uses
  subprocess incorrectly

Status in OpenStack Compute (Nova):
  Fix Released

Bug description:
  Both execute_get_output() and execute() don't wait until the process
  is finished running.

  execute_get_output() probably hasn't caused a problem since it at
  least does one read and the commands it runs likely would finish (but
  this isn't guaranteed).

  execute() sets up a PIPE for the process stdout, but doesn't do any
  reads before returning to the caller. This could make the code execute
  multiple processes in parallel leading to a race condition that could
  cause commands to execute in the opposite order that is intended. It
  could potentially also cause the process to block on writes to the
  PIPE that isn't being read leading it to never finish executing.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1286187/+subscriptions


References