NVP connection fails because port is a string

Bug #1255519 reported by Salvatore Orlando
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Salvatore Orlando
Havana
Fix Released
Medium
Salvatore Orlando

Bug Description

On a dev machine I've recently create I noticed failures at startup when Neutron is configured with the NVP plugin.
I root caused the failure to port being explicitly passed to HTTPSConnection constructor as a string rather than an integer.

This can be easily fixed ensuring port is always an integer.

I am not sure of the severity of this bug as it might strictly related to this specific dev env, but it might be worth applying and backporting it

Tags: nicira
tags: added: nicira
removed: nvp
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/58751

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/havana)

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/58758

tags: removed: grizzly-backport-potential
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-1 → icehouse-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/58751
Committed: http://github.com/openstack/neutron/commit/0cbcdcfc5094fcb52679550bc67db8136441adf6
Submitter: Jenkins
Branch: master

commit 0cbcdcfc5094fcb52679550bc67db8136441adf6
Author: Salvatore Orlando <email address hidden>
Date: Wed Nov 27 05:06:20 2013 -0800

    Ensure NVP API connection port is always an integer

    Convert the 'port' part of a NVP endpoint to int before passing
    it to the NVP API client.

    Change-Id: I15137d943ab950cb5cc80d7af2971bce3ac1a265
    Closes-Bug: #1255519

Changed in neutron:
status: In Progress → Fix Committed
Alan Pevec (apevec)
tags: removed: havana-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/havana)

Reviewed: https://review.openstack.org/58758
Committed: http://github.com/openstack/neutron/commit/6371d1b1d1d28ae9e30f902053e5527588dcf5f3
Submitter: Jenkins
Branch: stable/havana

commit 6371d1b1d1d28ae9e30f902053e5527588dcf5f3
Author: Salvatore Orlando <email address hidden>
Date: Wed Nov 27 05:13:51 2013 -0800

    Ensure NVP API connection port is always an integer

    Convert the 'port' part of a NVP endpoint to int before passing
    it to the NVP API client.

    Change-Id: I15137d943ab950cb5cc80d7af2971bce3ac1a265
    Closes-Bug: #1255519

Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-2 → 2014.1
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.