Functional test utilities confuse zero and None

Bug #1424824 reported by Ken Giusti
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.messaging
Fix Released
Medium
Ken Giusti

Bug Description

Client and server group code check input arguments against false, yet integer zero is a valid server index.

Ken Giusti (kgiusti)
Changed in oslo.messaging:
assignee: nobody → Ken Giusti (kgiusti)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.messaging (master)

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

Changed in oslo.messaging:
status: New → In Progress
Mehdi Abaakouk (sileht)
Changed in oslo.messaging:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.messaging (master)

Reviewed: https://review.openstack.org/158444
Committed: https://git.openstack.org/cgit/openstack/oslo.messaging/commit/?id=9f457b4f3e3424e5c4b3f7eb6904f0c33e348f4a
Submitter: Jenkins
Branch: master

commit 9f457b4f3e3424e5c4b3f7eb6904f0c33e348f4a
Author: Kenneth Giusti <email address hidden>
Date: Mon Feb 23 15:59:57 2015 -0500

    Properly distinguish between server index zero and no server

    Closes-Bug: #1424824
    Change-Id: I1a8c1fa8ad953d9437e35d9f0ec38b6c97020009

Changed in oslo.messaging:
status: In Progress → Fix Committed
Changed in oslo.messaging:
milestone: none → 1.8.1
status: Fix Committed → Fix Released
status: Fix Released → Fix Committed
milestone: 1.8.1 → next-liberty
Changed in oslo.messaging:
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.