wrong import of contrib module from novaclient

Bug #1418024 reported by Andriy Kurilin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-openstackclient
Fix Released
Undecided
Andriy Kurilin

Bug Description

Module openstackclient.compute.client has a direct import of novaclient.v1_1.contrib.list_extensions[*]. This import produces a blocker for novaclient.

[*] - https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/client.py#L19

Revision history for this message
Terry Howe (thowe-g) wrote :

What do you propose as an alternative?

Revision history for this message
Andriy Kurilin (andreykurilin) wrote :

I will do a patch in near future(30-60 Minutes) :)

Changed in python-openstackclient:
assignee: nobody → Andrey Kurilin (akurilin)
status: New → In Progress
Revision history for this message
Andriy Kurilin (andreykurilin) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-openstackclient (master)

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

Revision history for this message
Dean Troyer (dtroyer) wrote :

I proposed https://review.openstack.org/#/c/160473/ as an alternative fix for this bug. It handles all three occurrences of direct imports into novaclient and relies on novaclient to do the version to class mapping.

Changed in python-openstackclient:
milestone: none → m8
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-openstackclient (master)

Reviewed: https://review.openstack.org/160473
Committed: https://git.openstack.org/cgit/openstack/python-openstackclient/commit/?id=5d92fc0d0a7c606fdcf7a3565801ae361389d9ab
Submitter: Jenkins
Branch: master

commit 5d92fc0d0a7c606fdcf7a3565801ae361389d9ab
Author: Dean Troyer <email address hidden>
Date: Mon Mar 2 13:49:40 2015 -0600

    Handle novaclient >2.20.0

    As of 2.21.0 novaclient moved all of the v1_1 classes to v2 with a
    deprecation warning. The version-non-specific interfaces provided in
    novaclient.client are insufficient to support a few specific commands in
    OSC so we need to conditionally import directly from the correct classes.

    Closes-Bug: #1418024
    Change-Id: I864b1908737803069dc1419c9cbca391b985c932

Changed in python-openstackclient:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-openstackclient (master)

Change abandoned by Andrey Kurilin (<email address hidden>) on branch: master
Review: https://review.openstack.org/152919
Reason: already fixed by https://review.openstack.org/#/c/160473/

Dean Troyer (dtroyer)
Changed in python-openstackclient:
status: Fix Committed → Fix Released
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.