Integer type class does not recognize zero for min/max

Bug #1489688 reported by John L. Villalovos
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.config
Fix Released
Undecided
John L. Villalovos

Bug Description

The Integer type class does not recognize zero as a valid value for the min or max value.

Changed in oslo.config:
assignee: nobody → John L. Villalovos (happycamp)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.config (master)

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

Changed in oslo.config:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.config (master)

Reviewed: https://review.openstack.org/218076
Committed: https://git.openstack.org/cgit/openstack/oslo.config/commit/?id=85eee67b996a102b4318676bd03ee8624b9660b0
Submitter: Jenkins
Branch: master

commit 85eee67b996a102b4318676bd03ee8624b9660b0
Author: John L. Villalovos <email address hidden>
Date: Thu Aug 27 20:21:36 2015 -0700

    Make Integer type class honor zero for min/max

    The Integer type class has an option for setting a min or max value for the
    integer. But the class would not recognize zero as a valid value and not
    check it.

    Make the Integer class honor zero as a valid value for min and/or max.

    Add test cases for using zero for min and/or max.

    Change-Id: I395dea133d2a92e5ca4fab913aa17483a38f7ba1
    Closes-Bug: #1489688

Changed in oslo.config:
status: In Progress → Fix Committed
Changed in oslo.config:
milestone: none → 2.4.0
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/oslo.config 2.4.0

This issue was fixed in the openstack/oslo.config 2.4.0 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.