Parsing-server-ip-from-Heat-output

Bug #1485750 reported by Devdatta Kulkarni
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Solum
Fix Released
High
Devdatta Kulkarni

Bug Description

Change the parsing of Heat stack-show command to correctly parse the server-ip.

The heat stack-show command generates a list of two entries for a stack's output. One entry is for the output_key "URL" and other is corresponding to the output_key "public_ip". We want to parse the IP address of the server by parsing the entry corresponding to the "public_ip".

Currently we are picking the first entry from the outputs. See below:

https://github.com/stackforge/solum/blob/master/solum/deployer/handlers/heat.py#L530

This is incorrect, as the order of entries in the output may not be fixed.

Changed in solum:
assignee: nobody → Devdatta (devdatta-kulkarni)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to solum (master)

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

Changed in solum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Reviewed: https://review.openstack.org/213940
Committed: https://git.openstack.org/cgit/stackforge/solum/commit/?id=59f5e5add58b9d30d25128f88bf8fc445453c0dc
Submitter: Jenkins
Branch: master

commit 59f5e5add58b9d30d25128f88bf8fc445453c0dc
Author: Devdatta Kulkarni <email address hidden>
Date: Mon Aug 17 15:12:35 2015 -0500

    Correctly parsing the server ip from the created heat stack.

    The heat stack-show command generates a list of two entries for a stack's
    output. One entry is for the output_key "URL" and other is corresponding to
    the output_key "public_ip". We want to parse the IP address of the server
    by parsing the entry corresponding to the "public_ip".
    Currently we are picking the first entry from the outputs.

    This is incorrect, as the order of entries in the output may not be fixed.

    Change-Id: Iaf09f670e87097c85f3363e03dddac84cf4db497
    Fixes-Bug: #1485750

Changed in solum:
status: In Progress → Fix Committed
tags: added: solum-deployer
Changed in solum:
milestone: none → 2015.1.2
Changed in solum:
milestone: 2015.1.2 → 4.0.0
importance: Undecided → High
Changed in solum:
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.