Error validating environment for plan overcloud: ERROR: Property error: resources.CephStorageIpListMap.properties: Unknown Property StorageMgmtIpList

Bug #1813950 reported by Harald Jensås
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Medium
Harald Jensås

Bug Description

There is a fallback in the jinja templating, {{network.name}}IpList property is passed (using the ctlplane IP for all networks) to OS::TripleO::Network::Ports::NetIpListMap resource even if the network is not enabled or not defined in the role.

overcloud.j2.yaml:549
  {%- for network in networks %}
    {%- if network.enabled|default(true) and network.name in role.networks|default([]) %}
      {{network.name}}IpList: {get_attr: [{{role.name}}, {{network.name_lower}}_ip_address]}
    {%- else %}
      {{network.name}}IpList: {get_attr: [{{role.name}}, ip_address]}
    {%- endif %}
  {%- endfor %}

The fix is probably to remove these lines:
    {%- else %}
      {{network.name}}IpList: {get_attr: [{{role.name}}, ip_address]}

Revision history for this message
Tim Rozet (trozet) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)

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

Changed in tripleo:
assignee: nobody → Harald Jensås (harald-jensas)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (master)

Change abandoned by Harald Jensås (<email address hidden>) on branch: master
Review: https://review.openstack.org/633975
Reason: We improved the upstream fix to rename the existing queue. The service will now start properly without having to delete the old queue first. https://review.openstack.org/638114

Revision history for this message
Harald Jensås (harald-jensas) wrote :

@Tim, Is this still and issue after https://review.openstack.org/638440 ?

Revision history for this message
Tim Rozet (trozet) wrote :

I tried to test it but there are some other bugs with the deployment that are stopping me from validating it. That said, I'm not really working on this anymore so I think you can close it.

Changed in tripleo:
milestone: stein-3 → stein-rc1
Changed in tripleo:
importance: Undecided → Medium
milestone: stein-rc1 → train-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Harald Jensås (<email address hidden>) on branch: master
Review: https://review.opendev.org/633975

Changed in tripleo:
milestone: train-1 → train-2
Changed in tripleo:
milestone: train-2 → train-3
Changed in tripleo:
milestone: train-3 → ussuri-1
Changed in tripleo:
milestone: ussuri-1 → ussuri-2
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-2 → ussuri-3
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-3 → ussuri-rc3
wes hayutin (weshayutin)
Changed in tripleo:
milestone: ussuri-rc3 → victoria-1
Revision history for this message
Harald Jensås (harald-jensas) wrote :
Changed in tripleo:
status: In Progress → Fix Released
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.