xattr and pysendfile dependencies broke Hyper-V integration

Bug #1031396 reported by Pedro Navarro Pérez on 2012-07-31
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Medium
Jay Pipes
glance (Ubuntu)
Undecided
Chuck Short

Bug Description

xattr and pysendfile dependencies broke Hyper-V integration.

Jay Pipes state that these packages are not mandatory, Monty Taylor recommends that it should moved to test-requires.

Jay Pipes (jaypipes) on 2012-07-31
Changed in glance:
assignee: nobody → Jay Pipes (jaypipes)
importance: Undecided → Medium
status: New → Confirmed
description: updated
Changed in glance (Ubuntu):
status: New → Confirmed

By running the following on a clean Windows VM with python 2.7 x86:

pip install glance

I get greenlet downloaded as a dep:

Downloading/unpacking greenlet>=0.3.1 (from glance)
  Downloading greenlet-0.4.0.zip (72Kb): 72Kb downloaded
  Running setup.py egg_info for package greenlet

Building greenlet fails:

building 'greenlet' extension

error: Unable to find vcvarsall.bat

----------------------------------------
Command C:\Python27\python.exe -c "import setuptools;__file__='C:\\Python27\\Scr
ipts\\build\\greenlet\\setup.py';exec(compile(open(__file__).read().replace('\r\
n', '\n'), __file__, 'exec'))" install --single-version-externally-managed --rec
ord c:\users\anorex\appdata\local\temp\pip-ujinwf-record\install-record.txt fail
ed with error code 1 in C:\Python27\Scripts\build\greenlet
Storing complete log in C:\Users\anorex\AppData\Roaming\pip\pip.log

Update:

by installing manually greenlet-win32:

http://pypi.python.org/packages/2.7/g/greenlet/greenlet-0.4.0.win-amd64-py2.7.exe#md5=3f8e62c2ee68e26301105fe9caba93c3

and running again "pip install glance"

I get errors at compiling xattr:

running build_ext

building 'xattr._xattr' extension

error: Unable to find vcvarsall.bat

----------------------------------------
Command C:\Python27\python.exe -c "import setuptools;__file__='C:\\Python27\\Scr
ipts\\build\\xattr\\setup.py';exec(compile(open(__file__).read().replace('\r\n',
 '\n'), __file__, 'exec'))" install --single-version-externally-managed --record
 c:\users\anorex\appdata\local\temp\pip-dgskkc-record\install-record.txt failed
with error code 1 in C:\Python27\Scripts\build\xattr
Storing complete log in C:\Users\anorex\AppData\Roaming\pip\pip.log

Chuck Short (zulcss) on 2012-07-31
Changed in glance (Ubuntu):
assignee: nobody → Chuck Short (zulcss)

As a workaround, I removed xattr and pysendfile from the requirements:

Lib\site-packages\glance-2012.2-py2.7.egg-info\requires.txt

I had also to install win32 ports of:

greenlet from: http://pypi.python.org/packages/2.7/g/greenlet/greenlet-0.4.0.win-amd64-py2.7.exe#md5=3f8e62c2ee68e26301105fe9caba93c3

pycrypto from: http://www.voidspace.org.uk/python/modules.shtml#pycrypto

lxml from: http://www.lfd.uci.edu/~gohlke/pythonlibs/#lxml

The rest of the setup went smooth.

Jay Pipes (jaypipes) on 2012-07-31
tags: added: windows

Fix proposed to branch: master
Review: https://review.openstack.org/10619

Changed in glance:
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/10619
Committed: http://github.com/openstack/glance/commit/db33171ead3a4e6d9e5f880d53c375611d441805
Submitter: Jenkins
Branch: master

commit db33171ead3a4e6d9e5f880d53c375611d441805
Author: Jay Pipes <email address hidden>
Date: Tue Jul 31 15:47:39 2012 -0400

    Move optional dependencies from pip-requires to test-requires

    python-sendfile and python-xattr are both optional library
    dependencies in Glance now. Some systems, like Windows, do not
    have these libraries and so we should move them from pip-requires
    to test-requires. AFAIK, the pip-requires is what is used to
    build the Windows installer.

    Fixes LP 1031396

    Change-Id: Ib47244e51ae03991e6171ab8d47238d955743567

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2012-08-15
Changed in glance:
milestone: none → folsom-3
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package glance - 2012.2~f3-0ubuntu1

---------------
glance (2012.2~f3-0ubuntu1) quantal; urgency=low

  [ Adam Gandleman ]
  * debian/patches/sql_conn.patch: Also set default sqlite path for
    in glance-api.conf. (LP: #1028711)
  * debian/patches/fix-docs-build.patch: Fix docs build

  [ Chuck Short ]
  * New upstream version.
  * debian/control: python-xattr is no longer a required depends.
    (LP: #1031396)
  * debian/control: Move python-jsonschema to glance.
    (LP: #1030152)
  * debian/control: Start the slow transition to python-glanceclient.
 -- Chuck Short <email address hidden> Thu, 16 Aug 2012 13:58:32 -0500

Changed in glance (Ubuntu):
status: Confirmed → Fix Released
Thierry Carrez (ttx) on 2012-09-27
Changed in glance:
milestone: folsom-3 → 2012.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers