BSN: Consistency hash should be empty, not 'False'

Bug #1325771 reported by Kevin Benton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Kevin Benton
Icehouse
Fix Released
Undecided
Unassigned

Bug Description

When the Big Switch plugin first starts, it doesn't have a consistency hash yet, so it is currently setting the HTTP header to 'False'. This requires special-casing on the backend to handle. It should just be blank.

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/97371

Changed in neutron:
assignee: nobody → Kevin Benton (kevinbenton)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/icehouse)

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

Changed in neutron:
importance: Undecided → Medium
milestone: none → juno-1
tags: added: bigswitch icehouse-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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

commit 7b9ed4edeef7d599c164c8682fbbb36066d98abb
Author: Kevin Benton <email address hidden>
Date: Sun Jun 1 22:43:02 2014 -0700

    BSN: Set hash header to empty instead of False

    Sets the consistency hash header to empty instead
    of False since 'False' is handled like a string on
    the backend and requires special-casing to detect.

    Closes-Bug: #1325771
    Change-Id: Iee1651574c01a32e78167a9bbed4e0433abbdec2

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/icehouse)

Reviewed: https://review.openstack.org/97373
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=315319c246e17575756139c20694bc37efdf5818
Submitter: Jenkins
Branch: stable/icehouse

commit 315319c246e17575756139c20694bc37efdf5818
Author: Kevin Benton <email address hidden>
Date: Sun Jun 1 22:43:02 2014 -0700

    BSN: Set hash header to empty instead of False

    Sets the consistency hash header to empty instead
    of False since 'False' is handled like a string on
    the backend and requires special-casing to detect.

    Conflicts:

     neutron/tests/unit/bigswitch/test_servermanager.py

    Change-Id: Iee1651574c01a32e78167a9bbed4e0433abbdec2
    Closes-Bug: #1325771
    (cherry-picked from 7b9ed4edeef7d599c164c8682fbbb36066d98abb)

tags: added: in-stable-icehouse
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.