RandomString doesn't fail-fast when validating

Bug #1392754 reported by Steven Hardy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
Steven Hardy
Juno
Fix Released
Undecided
Unassigned

Bug Description

RandomString has some class-specific validation, which overrides the base-class validation, such that if you e.g do this:

  HeatTemplateFormatVersion: '2012-12-12'
  Resources:
    secret:
      Type: OS::Heat::RandomString
      Properties:
        length: 513

We don't fail validation before attempting to create the stack, instead failing during stack creation.

Steven Hardy (shardy)
Changed in heat:
assignee: nobody → Steven Hardy (shardy)
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/134558

Changed in heat:
status: New → In Progress
Steven Hardy (shardy)
Changed in heat:
milestone: none → kilo-1
importance: Undecided → Medium
Steven Hardy (shardy)
tags: added: juno-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/134558
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=02c4b2441e12ed12fa58ddd9858b5d91314668ee
Submitter: Jenkins
Branch: master

commit 02c4b2441e12ed12fa58ddd9858b5d91314668ee
Author: Steven Hardy <email address hidden>
Date: Fri Nov 14 15:10:57 2014 +0000

    RandomString don't skip superclass validation

    Ensure the superclass validation happens as well as the class-local
    additional stuff.

    Change-Id: I3fdc6e5d6a4ca741cdd12ca0ed6ceb908c03ae9f
    Closes-Bug: #1392754

Changed in heat:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/juno)

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/135666

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/juno)

Reviewed: https://review.openstack.org/135666
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=1d6dc24f0dd55666bfa77dbc7c34c038d779082d
Submitter: Jenkins
Branch: stable/juno

commit 1d6dc24f0dd55666bfa77dbc7c34c038d779082d
Author: Steven Hardy <email address hidden>
Date: Fri Nov 14 15:10:57 2014 +0000

    RandomString don't skip superclass validation

    Ensure the superclass validation happens as well as the class-local
    additional stuff.

    Change-Id: I3fdc6e5d6a4ca741cdd12ca0ed6ceb908c03ae9f
    Closes-Bug: #1392754
    (cherry picked from commit 02c4b2441e12ed12fa58ddd9858b5d91314668ee)

tags: added: in-stable-juno
Thierry Carrez (ttx)
Changed in heat:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: kilo-1 → 2015.1.0
Zane Bitter (zaneb)
tags: removed: juno-backport-potential
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.