Cisco ML2 extraconfig template outputs an additional incorrect key into the hiera file

Bug #1791044 reported by Sam Betts
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Undecided
James Slagle

Bug Description

Description
===========

Since os-apply-config changed to the hiera hook here:
https://github.com/openstack/tripleo-heat-templates/commit/c5d10cd9fc94e6557417673190b73867a83cbb7b
the mapped_data key is no longer required and results in an additional
incorrect key being written to the hiera data.

Steps to reproduce
==================

Run a tripleO overcloud deployment with -e /usr/share/openstack-tripleo-heat-templates/puppet/extraconfig/all_nodes/neutron-ml2-cisco-nexus-ucsm.yaml to configure the Cisco ML2 hiera data.

Then on one of the controller nodes `cat /etc/puppet/hieradata/neutron_cisco_data.json` you can see the hieradata that was generated.

Expected result
===============

{
 "neutron::plugins::ml2::cisco::ucsm::ucsm_ip": "1.2.3.4",
  ...
}

Actual result
=============

{
  "mapped_data": {
      "neutron::plugins::ml2::cisco::ucsm::ucsm_ip": "1.2.3.4",
       ....
  }
}

Environment
===========

RH OSP13 (queens)

Sam Betts (sambetts)
Changed in tripleo:
assignee: nobody → Sam Betts (sambetts)
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/600396

Changed in tripleo:
status: New → In Progress
Changed in tripleo:
assignee: Sam Betts (sambetts) → James Slagle (james-slagle)
Changed in tripleo:
assignee: James Slagle (james-slagle) → Sam Betts (sambetts)
Changed in tripleo:
assignee: Sam Betts (sambetts) → James Slagle (james-slagle)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/600396
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=95245f6ad4a85584081cfe1004e44329995a85b1
Submitter: Zuul
Branch: master

commit 95245f6ad4a85584081cfe1004e44329995a85b1
Author: Sam Betts <email address hidden>
Date: Thu Sep 6 10:33:18 2018 +0100

    Remove incorrect mapped_data key from cisco ml2 hieradata

    Since os-apply-config changed to the hiera hook here:
    https://github.com/openstack/tripleo-heat-templates/commit/c5d10cd9fc94e6557417673190b73867a83cbb7b
    the mapped_data key is no longer required and results in an additional
    incorrect key being written to the hiera data. This patch removes it to
    ensure the hiera data for cisco ml2 works as expected.

    Change-Id: I730b107d4b5a002f82f658a83d11162606e44a16
    Closes-Bug: #1791044

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 10.4.0

This issue was fixed in the openstack/tripleo-heat-templates 10.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/699421

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

Reviewed: https://review.opendev.org/699421
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=7044c1858d03d92fe38a41aa05cd8c578348121d
Submitter: Zuul
Branch: stable/queens

commit 7044c1858d03d92fe38a41aa05cd8c578348121d
Author: Sam Betts <email address hidden>
Date: Thu Sep 6 10:33:18 2018 +0100

    Remove incorrect mapped_data key from cisco ml2 hieradata

    Since os-apply-config changed to the hiera hook here:
    https://github.com/openstack/tripleo-heat-templates/commit/c5d10cd9fc94e6557417673190b73867a83cbb7b
    the mapped_data key is no longer required and results in an additional
    incorrect key being written to the hiera data. This patch removes it to
    ensure the hiera data for cisco ml2 works as expected.

    Change-Id: I730b107d4b5a002f82f658a83d11162606e44a16
    Closes-Bug: #1791044
    (cherry picked from commit 95245f6ad4a85584081cfe1004e44329995a85b1)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates queens-eol

This issue was fixed in the openstack/tripleo-heat-templates queens-eol release.

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.