repo build broken due to pywbem 0.8.1

Bug #1559380 reported by Logan V
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
High
Kevin Carter
Kilo
Fix Released
High
Kevin Carter
Liberty
Fix Released
High
Kevin Carter
Trunk
Fix Released
High
Kevin Carter

Bug Description

pywbem release this afternoon appears to be breaking the repo build on master/liberty.

master gate is broken:
http://logs.openstack.org/99/277199/16/check/gate-openstack-ansible-dsvm-commit/51de785/console.html#_2016-03-18_21_09_37_014

liberty gate broken:
http://logs.openstack.org/65/294765/1/check/gate-openstack-ansible-dsvm-commit/03b96b6/console.html#_2016-03-18_19_50_33_660

My local liberty build is broken:
http://cdn.pasteraw.com/f3su2r6wanlc9feocqaf95btf370c33

I dropped a pin for pywbem<0.8.1 in requirements.txt locally and that fixed it for me.

Revision history for this message
Logan V (loganv) wrote :
Revision history for this message
Andreas Maier (maiera) wrote :

Hi. I am the maintainer of pywbem. The reason for the OSA build break is the new dependency to the "six" module in the pywbem setup script. It needs that to execute, so it is a prerequisite that needs to be satisfied upfront before the script even starts.
Apparently, six is not available everywhere.

Can you somehow establish this prerequisite, or do you prefer if we rework the setup script to avoid the dependency on six?

Revision history for this message
Andreas Maier (maiera) wrote :

On the build break in your local liberty build: pywbem 0.8.1 introduces a dependency to M2Crypto, which is stated in the "install_requires" argument of the setup script. I see that you have M2Crypto at the end of the list of dependencies.

Can you please find out whether moving it before pywbem helps?

Revision history for this message
Andreas Maier (maiera) wrote :

Update on the six dependency: The dependency was easy to eliminate, the cost is 2 x 3 lines of type definitions that would otherwise come from six. I assume this will not be the only case where the pre-dependency on six hurts, so I'll go for eliminating it.

Revision history for this message
Andreas Maier (maiera) wrote :

I released 0.8.2 to PyPI. That version eliminates the dependency on the "six" package it had during its own installation.

Please try it out to see whether it resolves the issue in the OpenStack Ansible gate checks.

Please also let me know whether you can move the M2Crypto installation before the pywbem installation in your local build, and if so, whether that helps. Note that pywbem does state its dependency on M2Crypto in the 'install_requires' argument of its setup script.

information type: Public → Public Security
information type: Public Security → Public
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible (liberty)

Fix proposed to branch: liberty
Review: https://review.openstack.org/295557

Revision history for this message
Jesse Pretorius (jesse-pretorius) wrote :

As this package wasn't actually required directly for anything in OSA, it has been removed:

Master: https://review.openstack.org/294932 / https://review.openstack.org/294933
Liberty: https://review.openstack.org/294934
Kilo: https://review.openstack.org/294935

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: liberty
Review: https://review.openstack.org/297060

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible (liberty)

Reviewed: https://review.openstack.org/297060
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=a59b2d2ba4dc091318a3525788aea29c13d7bb2a
Submitter: Jenkins
Branch: liberty

commit a59b2d2ba4dc091318a3525788aea29c13d7bb2a
Author: Jesse Pretorius <email address hidden>
Date: Thu Mar 24 12:00:38 2016 +0000

    Add release note for pywbem issue

    Change-Id: I148e5f4ef71fcfd0387f03ed53c47cd8c9582c58
    Closes-Bug: 1559380

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.9

This issue was fixed in the openstack/openstack-ansible 12.0.9 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on openstack-ansible (liberty)

Change abandoned by Jesse Pretorius (odyssey4me) (<email address hidden>) on branch: liberty
Review: https://review.openstack.org/295557
Reason: Abandoned as this patch is not required and the author has not responded to the previous comment.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 12.0.11

This issue was fixed in the openstack/openstack-ansible 12.0.11 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.