Fn::Select no longer accepts integer selector values

Bug #1245863 reported by Roman Podoliaka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Critical
Roman Podoliaka
tripleo
Fix Released
Critical
Roman Podoliaka

Bug Description

Change https://review.openstack.org/#/c/54063/ makes Fn::Select accept only string selector values. At the same time, our doc states, that the selector can be either of type string or of type integer (http://docs.openstack.org/developer/heat/template_guide/functions.html#fn-select)

If this is considered to be the correct behavior, we must update Heat docs and fix templates we are using in TripleO devtest story.

Changed in heat:
assignee: nobody → Roman Podoliaka (rpodolyaka)
Changed in tripleo:
status: New → Triaged
importance: Undecided → Critical
assignee: nobody → Roman Podoliaka (rpodolyaka)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Changed in heat:
status: New → In Progress
Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

Alternative fix to tripleo-heat-templates is here https://review.openstack.org/#/c/54322/

Revision history for this message
Steven Hardy (shardy) wrote :

https://review.openstack.org/#/c/54063/ should not have changed this behavior IMO, so this seems to be a regression which we missed at the review :(

Changed in heat:
importance: Undecided → High
milestone: none → icehouse-1
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Agreed Steven. Marking Critical, this will break a lot of templates.

Let's make sure we follow up with some tempest tests.

Changed in heat:
importance: High → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/54321
Committed: http://github.com/openstack/heat/commit/11ac4dd3f899dc0f28d356bad89ec559e9d1e81f
Submitter: Jenkins
Branch: master

commit 11ac4dd3f899dc0f28d356bad89ec559e9d1e81f
Author: Roman Podoliaka <email address hidden>
Date: Tue Oct 29 14:18:04 2013 +0200

    Make Fn::Select accept an integer selector again

    Closes-Bug: #1245863

    Change-Id: I364da88f1c581b406d51e1b584778c7dd4e2564f

Changed in heat:
status: In Progress → Fix Committed
Changed in tripleo:
status: Triaged → Fix Released
Thierry Carrez (ttx)
Changed in heat:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: icehouse-1 → 2014.1
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.