Nova notification introduces a hard dependency on novaclient

Bug #1321352 reported by Terry Wilson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Terry Wilson

Bug Description

The nova notification patch introduces a hard dependency on novaclient when it is a runtime-configurable dependency. The import from novaclient should be conditional on the appropriate nova notification options being enabled in the config.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
assignee: nobody → Terry Wilson (otherwiseguy)
status: New → In Progress
Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Low
milestone: none → juno-1
Revision history for this message
Assaf Muller (amuller) wrote :

Some more context: oVirt uses Neutron in a standalone manner - IE: Install Neutron API + agents + Keystone on a machine, then integrate it with oVirt to provide advanced networking services. Introducing more dependencies (Like Nova) is an issue for such a use case. I'm not saying from Neutron's point of view this is a high priority deal or anything, just providing some context.

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

Reviewed: https://review.openstack.org/94416
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=0f877f2594d415513856af3c528275fce2228ac1
Submitter: Jenkins
Branch: master

commit 0f877f2594d415513856af3c528275fce2228ac1
Author: Terry Wilson <email address hidden>
Date: Tue May 20 10:58:32 2014 -0500

    Remove hard dependency on novaclient

    The nova notification patch introduces a hard dependency on
    novaclient when it is a runtime-configurable dependency. The
    import from novaclient should be conditional on the
    appropriate nova notification options being enabled in the
    config.

    Change-Id: I2ef4bfa4d53afc7e8c800ad8e2a8737e117af238
    Closes-Bug: #1321352

Changed in neutron:
status: In Progress → Fix Committed
Assaf Muller (amuller)
tags: added: icehouse-backport-potential
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-1 → 2014.2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/332425
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=b0aa3a9664ce9adc57006287457d5d0a5fd2db14
Submitter: Jenkins
Branch: master

commit b0aa3a9664ce9adc57006287457d5d0a5fd2db14
Author: Assaf Muller <email address hidden>
Date: Tue Jun 21 18:47:08 2016 +0000

    Revert "move import to top and rename to make more readable"

    This reverts commit 4ae2c246dd1cc16ebffb6f9d6a64f6ba7db2c05a.

    Closes-Bug: #1321352
    Change-Id: I7094dfc90b08bd9da6ac31130f0ebafc547fa45a

tags: added: neutron-proactive-backport-potential
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/neutron 9.0.0.0b2

This issue was fixed in the openstack/neutron 9.0.0.0b2 development milestone.

tags: removed: neutron-proactive-backport-potential
tags: removed: icehouse-backport-potential
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.