puppet-ceilometer

validate_re fails for non-string arguments under Puppet 3.7.x

Bug #1371235 reported by Mike Dorman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ceilometer
Fix Released
Undecided
Mike Dorman
puppet-ceilometer 5.0.0 "5.0.0"

Bug Description

The validate_re() call against $evaluation_interval in
manifests/alarm/evaluator.pp fails under Puppet 3.7.x when
$evaluation_interval is not strictly a string argument. validate_re()
fails saying it doens't match if it's an int, which is the default.

Mike Dorman (mdorman-m)
Changed in puppet-ceilometer:
assignee: nobody → Mike Dorman (mdorman-m)
OpenStack Infra (hudson-openstack)
Changed in puppet-ceilometer:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceilometer (master)

Reviewed: https://review.openstack.org/122490
Committed: https://git.openstack.org/cgit/stackforge/puppet-ceilometer/commit/?id=a4f20d069ef002fe22b51cd954fec196954caa89
Submitter: Jenkins
Branch: master

commit a4f20d069ef002fe22b51cd954fec196954caa89
Author: Mike Dorman <email address hidden>
Date: Thu Sep 18 12:18:30 2014 -0600

    Stringify $evaluation_interval for validate_re call

    Under Puppet 3.7.x, calls to validate_re fail if the parameter is not
    strictly a string. This change creates a string from
    $evaluation_interval when passing to validate_re, so the call will
    pass regardless of if the parameter to the class is specified as
    string or an integer.

    Change-Id: I17b82977f21483acf134837a66c5ecebe346cfdf
    Closes-Bug: 1371235

Changed in puppet-ceilometer:
status: In Progress → Fix Committed
Mathieu Gagné (mgagne)
Changed in puppet-ceilometer:
milestone: none → 5.0.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.