Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Commit

Permalink
Use RootStackName parameter instead for consistency
Browse files Browse the repository at this point in the history
We don't need another parameter for the root stack and
also not required to pass it around.

This was added earlier with [1].

[1] https://review.opendev.org/openstack/tripleo-heat-templates/761845

Change-Id: I5dee09ab6c15c1887394b0dbda6866e94231f3a5
  • Loading branch information
rabi committed Dec 14, 2020
1 parent 7b12da0 commit 6d39c90
Show file tree
Hide file tree
Showing 7 changed files with 3 additions and 26 deletions.
4 changes: 0 additions & 4 deletions network/ports/from_service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ description: >
Returns an IP from a service mapped list of IPs
parameters:
StackName: # Here for compatibility
description: Name of the overcloud heat stack
default: overcloud
type: string
ServiceName:
description: Name of the service to lookup
default: ''
Expand Down
4 changes: 0 additions & 4 deletions network/ports/from_service_v6.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ description: >
Returns an IP from a service mapped list of IPv6 IPs
parameters:
StackName: # Here for compatibility
description: Name of the overcloud heat stack
default: overcloud
type: string
ServiceName:
description: Name of the service to lookup
default: ''
Expand Down
4 changes: 0 additions & 4 deletions network/ports/noop.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ description: >
Returns the control plane port (provisioning network) as the ip_address.
parameters:
StackName: # Here for compatibility
description: Name of the overcloud heat stack
default: overcloud
type: string
ServiceName: # Here for compatibility with from_service.yaml
description: Name of the service to lookup
default: ''
Expand Down
7 changes: 3 additions & 4 deletions network/ports/port.j2
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,8 @@ description: >
automatically if FixedIPs is empty.

parameters:
StackName:
description: Name of the overcloud heat stack
default: overcloud
RootStackName:
description: The name of the stack/plan.
type: string
{{network.name}}NetName:
description: The name of the {{network.name_lower}} network.
Expand Down Expand Up @@ -93,7 +92,7 @@ resources:
- str_replace:
template: tripleo_stack_name=$STACK_NAME
params:
$STACK_NAME: {get_param: StackName}
$STACK_NAME: {get_param: RootStackName}

outputs:
ip_address:
Expand Down
4 changes: 0 additions & 4 deletions network/ports/port_from_pool.j2
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,6 @@ description: >
- 1.2.3.5 # Second controller

parameters:
StackName: # Here for compatibility
description: Name of the overcloud heat stack
default: overcloud
type: string
{{network.name}}NetName:
description: The name of the {{network.name_lower}} network.
default: {{network.name_lower}}
Expand Down
1 change: 0 additions & 1 deletion overcloud.j2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -823,7 +823,6 @@ resources:
resource_def:
type: OS::TripleO::{{role.name}}
properties:
StackName: {get_param: 'OS::stack_name'}
CloudDomain: {get_param: CloudDomain}
ServiceNetMap: {get_attr: [ServiceNetMap, service_net_map]}
EndpointMap: {get_attr: [EndpointMapData, value]}
Expand Down
5 changes: 0 additions & 5 deletions puppet/role.role.j2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@
heat_template_version: rocky
description: 'OpenStack {{role.name}} node configured by Puppet'
parameters:
StackName:
description: Name of the overcloud heat stack
default: overcloud
type: string
{%- set default_flavor_name = role.FlavorDefault|default('baremetal') %}
{%- if role.deprecated_param_flavor is defined %}
{{role.deprecated_param_flavor}}:
Expand Down Expand Up @@ -304,7 +300,6 @@ resources:
{{network.name}}Port:
type: OS::TripleO::{{role.name}}::Ports::{{network.name}}Port
properties:
StackName: {get_param: StackName}
PortName:
list_join:
- '_'
Expand Down

0 comments on commit 6d39c90

Please sign in to comment.