neutron does not have a nova_api_insecure conf setting

Bug #1306822 reported by Paul Ward
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Paul Ward

Bug Description

In setting the nova_api_insecure flag to True in neutron.conf, I've found that neutron/notifiers/nova.py does not honor this flag (actually I found neutron doesn't even have this flag defined). In fact, it does not even pass any insecure parm at all on the instantiation of the novaclient.

https://github.com/openstack/neutron/blob/master/neutron/notifiers/nova.py#L45

Paul Ward (wpward)
Changed in neutron:
assignee: nobody → Paul Ward (wpward)
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/87085

Changed in neutron:
status: New → In Progress
Revision history for this message
Paul Ward (wpward) wrote : Re: nova notifier does not honor nova_api_insecure conf setting

I've uploaded a patch for this: https://review.openstack.org/#/c/87085/

Paul Ward (wpward)
summary: - nova notifier does not honor nova_api_insecure conf setting
+ neutron does not have a nova_api_insecure conf setting
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit a3e49885e7b7a69277a338f5d31e565e195b033c
Author: Paul Ward <email address hidden>
Date: Sat Apr 12 10:18:18 2014 -0500

    Add nova_api_insecure flag to neutron

    Neutron did not have any way to allow SSL validation problems in the
    novaclient similar to the way nova has the neutron_api_insecure flag.
    This change adds the nova_api_insecure flag to the neutron config and
    also passes it in the novaclient instantiation.

    Change-Id: Ifed581df253c01fab9dc1f22c8187876271b378e
    Closes-Bug: #1306822

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Brad Pokorny (bpokorny) wrote :

We've got users hitting this issue in Icehouse, so I tagged this for backport.

tags: added: icehouse-backport-potential
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to neutron (stable/icehouse)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (stable/icehouse)

Change abandoned by Brad Pokorny (<email address hidden>) on branch: stable/icehouse
Review: https://review.openstack.org/88437
Reason: The consensus on not backporting this to stable/icehouse is reasonable to me.

Thierry Carrez (ttx)
Changed in neutron:
milestone: none → juno-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-1 → 2014.2
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.