Bug 1328149

Summary: Default value for ControlPlaneSubnetCidr in /usr/share/openstack-tripleo-heat-templates/environments/network-environment.yaml should be a string
Product: Red Hat OpenStack Reporter: Pedro Navarro <pnavarro>
Component: openstack-tripleo-heat-templatesAssignee: Jiri Stransky <jstransk>
Status: CLOSED DUPLICATE QA Contact: Arik Chernetsky <achernet>
Severity: unspecified Docs Contact:
Priority: high    
Version: 8.0 (Liberty)CC: aschultz, bnemec, cpaquin, jraju, jstransk, mburns, ohochman, racedoro, rhel-osp-director-maint, vcojot
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-24 21:36:39 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Pedro Navarro 2016-04-18 15:11:02 UTC 
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:
Install RH OSP 8

Steps to Reproduce:
1. Plan a network isolation architecture
2. Add Default value /usr/share/openstack-tripleo-heat-templates/environments/network-environment.yaml in your "openstack overcloud deploy"
3.

Actual results:

The overcloud is deployed

Expected results:

2016-04-18 14:55:03 [overcloud-Controller-jyyn6nnvrymh-0-aqu5ymdtp33g]: CREATE_FAILED  Resource CREATE failed: resources.NetworkConfig: Property error: resources.OsNetConfigImpl.properties.config: Items to join 
must be strings not 24


Additional info:
Comment 5 Ramon Acedo 2016-04-28 10:26:12 UTC 
The fix should be this one which involves numeric to string conversion: https://bugs.launchpad.net/heat/+bug/1567373
Comment 6 Jiri Stransky 2016-05-03 09:00:31 UTC 
As Ramon wrote this will most probably be addressed in Heat. Submitted a t-h-t fix too just to make sure, since it's a one liner.
Comment 7 Jiri Stransky 2016-05-03 12:11:48 UTC 
The fix was already proposed by Rhys earlier actually, updating the t-h-t patch reference.
Comment 8 Alex Schultz 2017-01-27 23:23:29 UTC 
*** Bug 1344799 has been marked as a duplicate of this bug. ***
Comment 9 Ben Nemec 2017-03-21 19:51:54 UTC 
I believe this has been fixed upstream for a long time.  We just need to verify that it doesn't happen in recent releases.

Note that the workaround for anyone who is hitting this in older releases is to ensure the value for this parameter is a string, i.e. '24' not 24.
Comment 10 Alex Schultz 2017-03-24 21:36:39 UTC 

*** This bug has been marked as a duplicate of bug 1334073 ***