OpenStack Compute (nova)

2014.1.4 Nova introduces new dependency for oslo.utils in stable/icehouse

Bug #1432685 reported by Corey Bryant
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
Undecided
Unassigned
Icehouse
Fix Released
Undecided
Unassigned
OpenStack Compute (nova) 2014.1.5
Juno
Invalid
Undecided
Unassigned

Bug Description

In the latest release of 2014.1.4 nova, commit 4b46a86f8a2af096e399df8518f8269f825684e0 introduces a new dependency for oslo.utils in nova/compute/api.py.

I think it's policy to not allow new dependencies in stable releases so this may have slipped by.

See original description
Corey Bryant (corey.bryant)
description: updated
Davanum Srinivas (DIMS) (dims-v)
summary: - 2014.1.4 introduces new dependency for oslo.utils
+ 2014.1.4 introduces new dependency for oslo.utils in stable/icehouse
Alan Pevec (apevec)
summary: - 2014.1.4 introduces new dependency for oslo.utils in stable/icehouse
+ 2014.1.4 Nova introduces new dependency for oslo.utils in
+ stable/icehouse
Revision history for this message
Alan Pevec (apevec) wrote :

oslo.utils is indeed not in Icehouse Nova requirements.txt but it is in Icehouse global-requirements:
https://git.openstack.org/cgit/openstack/requirements/tree/global-requirements.txt?h=stable/icehouse#n53

BTW we have the same situation with oslo.utils in Nova stable/juno branch: not in requirements.txt but used in this backport and few more places, commits 33e0813b and 1e4088f4 included in 2014.2.2

That mean gate jobs are not enforcing project's requirements.txt ?

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/icehouse)

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/164763

Revision history for this message
Ihar Hrachyshka (ihar-hrachyshka) wrote :

Another reason to actually push for a hacking check: https://review.openstack.org/157894

Alan Pevec (apevec)
Changed in nova:
status: New → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/icehouse)

Reviewed: https://review.openstack.org/164763
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=3501ec2cb773417f05004d8b5567d5755213dabe
Submitter: Jenkins
Branch: stable/icehouse

commit 3501ec2cb773417f05004d8b5567d5755213dabe
Author: Corey Bryant <email address hidden>
Date: Mon Mar 16 12:19:05 2015 -0400

    Drop use of oslo.utils in nova

    Use units from nova.openstack.common instead of using units from
    oslo.utils.

    Fixes bug: 1432685

    Change-Id: I5d66c5c10728024e4d3dd3ae5362217f76164ed1

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

There's a cap now:

http://git.openstack.org/cgit/openstack/requirements/tree/global-requirements.txt?h=stable/juno#n68

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

Other bug subscribers

Remote bug watches

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