[api] heatSWConfigInstance doesn't allow string s-w configs

Bug #1358046 reported by Steve McLellan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Fix Released
Medium
Steve McLellan

Bug Description

The contract for heat SW config sections is for a dictionary, which is fine for structured configs but not for e.g. scripts:

  heat-walrus-config:
    properties:
      config: |
          #!/bin/bash

        echo "I am the walrus"

      group: script
    type: OS::Heat::SoftwareConfig

Fix is to relax the restriction (though it might be neater to refactor the function).

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

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

Changed in murano:
status: New → In Progress
Changed in murano:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano (master)

Reviewed: https://review.openstack.org/114860
Committed: https://git.openstack.org/cgit/stackforge/murano/commit/?id=b5530560fb3cfa21c580f20d7d34f36e737ed957
Submitter: Jenkins
Branch: master

commit b5530560fb3cfa21c580f20d7d34f36e737ed957
Author: Steve McLellan <email address hidden>
Date: Sun Aug 17 18:45:06 2014 -0500

    Allow strings in heat SW config configSection

    For non-structured configs, configSection cannot be restricted to a
    Map. Patch removes the contract type, although it is worth revisiting
    this to refactor slightly in future (which should be possible without
    changing the interface).

    Change-Id: I179e350a724965c97e90c3a1927f955c20cc2a72
    Closes-Bug: 1358046

Changed in murano:
status: In Progress → Fix Committed
ruhe (ruhe)
Changed in murano:
status: Fix Committed → Fix Released
Changed in murano:
milestone: juno-3 → 2014.2
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.