Incorrect numeric comparisons

Bug #1794068 reported by Lucian Petrut
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
os-win
Fix Released
Undecided
Unassigned

Bug Description

In a few places, we're using "is" or "is not", when comparing numeric variables.

This is incorrect and will give unexpected results when the types do not match (for example, if one of the variables is a long int): http://paste.openstack.org/raw/730618/

This issue only affects some metric related functions.

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

Reviewed: https://review.openstack.org/604742
Committed: https://git.openstack.org/cgit/openstack/os-win/commit/?id=1f8200ef7909b13adf501f2805554160fa934f3a
Submitter: Zuul
Branch: master

commit 1f8200ef7909b13adf501f2805554160fa934f3a
Author: Lucian Petrut <email address hidden>
Date: Mon Sep 24 14:00:03 2018 +0300

    Fix incorrect numeric comparisons

    In a few places, we're using "is" or "is not", when comparing
    numeric variables.

    This is incorrect and will give unexpected results when the types
    do not match (for example, if one of the variables is a long int):
    http://paste.openstack.org/raw/730618/

    This change fixes those comparisons.

    Closes-Bug: #1794068

    Change-Id: Ibf28778879050cfdfa82ef4f9b587170dde6203f

Changed in os-win:
status: New → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-win (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/607547

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-win (stable/rocky)

Reviewed: https://review.openstack.org/607547
Committed: https://git.openstack.org/cgit/openstack/os-win/commit/?id=59e1117506067f42cec30e0eae6c7e92ffb0b6bc
Submitter: Zuul
Branch: stable/rocky

commit 59e1117506067f42cec30e0eae6c7e92ffb0b6bc
Author: Lucian Petrut <email address hidden>
Date: Mon Sep 24 14:00:03 2018 +0300

    Fix incorrect numeric comparisons

    In a few places, we're using "is" or "is not", when comparing
    numeric variables.

    This is incorrect and will give unexpected results when the types
    do not match (for example, if one of the variables is a long int):
    http://paste.openstack.org/raw/730618/

    This change fixes those comparisons.

    Closes-Bug: #1794068

    Change-Id: Ibf28778879050cfdfa82ef4f9b587170dde6203f
    (cherry picked from commit 1f8200ef7909b13adf501f2805554160fa934f3a)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-win (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/607867

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-win (stable/queens)

Reviewed: https://review.openstack.org/607867
Committed: https://git.openstack.org/cgit/openstack/os-win/commit/?id=b4fa3ce9582c7c87174cf61bd501571da0eae968
Submitter: Zuul
Branch: stable/queens

commit b4fa3ce9582c7c87174cf61bd501571da0eae968
Author: Lucian Petrut <email address hidden>
Date: Mon Sep 24 14:00:03 2018 +0300

    Fix incorrect numeric comparisons

    In a few places, we're using "is" or "is not", when comparing
    numeric variables.

    This is incorrect and will give unexpected results when the types
    do not match (for example, if one of the variables is a long int):
    http://paste.openstack.org/raw/730618/

    This change fixes those comparisons.

    Closes-Bug: #1794068

    Change-Id: Ibf28778879050cfdfa82ef4f9b587170dde6203f
    (cherry picked from commit 1f8200ef7909b13adf501f2805554160fa934f3a)
    (cherry picked from commit 59e1117506067f42cec30e0eae6c7e92ffb0b6bc)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-win 4.1.0

This issue was fixed in the openstack/os-win 4.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-win queens-eol

This issue was fixed in the openstack/os-win queens-eol release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-win rocky-eol

This issue was fixed in the openstack/os-win rocky-eol release.

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.