← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1209410] Re: processutlis.execute not usable in windows

 

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

** Changed in: cinder
    Milestone: None => havana-rc1

-- 
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/1209410

Title:
  processutlis.execute not usable in windows

Status in Cinder:
  Fix Released
Status in OpenStack Compute (Nova):
  Fix Released
Status in Oslo - a Library of Common OpenStack Code:
  Fix Released

Bug description:
  In Havana time, in the execute method of processutils the condition
  (os.geteuid() != 0) was added in line 130

      if run_as_root and os.geteuid() != 0:
          if not root_helper:
              raise NoRootWrapSpecified(
                  message=('Command requested root, but did not specify a root '
                           'helper.'))
          cmd = shlex.split(root_helper) + list(cmd)

  That provokes the next error in Windows:

  
  2013-08-08 00:27:09.937 2148 TRACE cinder.openstack.common.rpc.amqp     (stdout, stderr) = processutils.execute(*cmd, **kwargs)
  2013-08-08 00:27:09.937 2148 TRACE cinder.openstack.common.rpc.amqp   File "C:\Users\Pedro\dev\cinder\cinder\openstack\common\processutils.py", line 130, in execute
  2013-08-08 00:27:09.937 2148 TRACE cinder.openstack.common.rpc.amqp     if run_as_root and os.geteuid() != 0:
  2013-08-08 00:27:09.937 2148 TRACE cinder.openstack.common.rpc.amqp AttributeError: 'module' object has no attribute 'geteuid'

  This should be removed in order to this code be used in nova and
  cinder windows-based drivers

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