Grafana Dashboad creation fails when using alternate overcloud hostnames

Bug #1612787 reported by Matt Wisch
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-browbeat
Fix Released
High
Matt Wisch

Bug Description

When using alternate overcloud hostnames via a hostname mapping template (like ips-from-pool.yaml) the hosts_to_dictionary function will fail if the server index is not preceded by a hyphen, returning the error: "ValueError: invalid literal for int() with base 10: 'controller01'"

I am proposing a change that will allow both the default server naming index, ie. <cloud>-<function>-##, and alternate naming schemes where the server name ends with a number, but a hyphen does not directly precede the index (or the name does not contain a hyphen at all), ie. <cloud>-<function>### or <function>##

Matt Wisch (matt-wisch)
Changed in openstack-browbeat:
assignee: nobody → Matt Wisch (matt-wisch)
status: New → Fix Committed
Joe Talerico (jtaleric)
Changed in openstack-browbeat:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to browbeat (master)

Reviewed: https://review.openstack.org/355055
Committed: https://git.openstack.org/cgit/openstack/browbeat/commit/?id=b339831ca53ef5f8e6ea88d10f022bb5f8f3c5fb
Submitter: Jenkins
Branch: master

commit b339831ca53ef5f8e6ea88d10f022bb5f8f3c5fb
Author: Matt Wisch <email address hidden>
Date: Fri Aug 12 15:51:12 2016 -0400

    Updated method to index hosts for Grafana Dashboard creation

    Updated the hosts_to_dictionary function to check to see if the last
    digit of the overcloud host name is a digit. If so, it uses a regex
    to isolate the all of the digits at the end of the hostname, and proceeds
    to use that as the dictionary index.

    This change allows for alternate overcloud host naming schemes that do not
    have a hyphen preceding the server number.

    Change-Id: Ief10b2100b5d7c55a940ab4186ce2e31c109420c
    Closes-Bug: #1612787

Changed in openstack-browbeat:
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.