Mitaka and Newton jobs broken on splay error

Bug #1686455 reported by Ben Nemec
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Heat Templates
Fix Released
Undecided
Alex Schultz
tripleo
Fix Released
Critical
Alex Schultz

Bug Description

All mitaka and newton underclouds are failing with the following error:

[ERROR] Section end tag mismatch: splay}

There appears to be a typo in the mustache template for the splay option. It actually looks like:

{{/splay}

and should be

{{/splay}}

It's very confusing though. I see

# rpm -qf /usr/libexec/os-apply-config/templates/etc/os-collect-config.conf
python-heat-agent-0.0.1-0.20170426055943.2603dbe.el7.centos.noarch

but when I look at the heat-agents template it looks right: https://github.com/openstack/heat-agents/blob/master/heat-config/os-apply-config/etc/os-collect-config.conf

So I'm not sure where the bad template is coming from.

Tags: ci
Revision history for this message
Alex Schultz (alex-schultz) wrote :
Changed in tripleo:
assignee: nobody → Alex Schultz (alex-schultz)
Changed in heat-templates:
status: New → In Progress
assignee: nobody → Alex Schultz (alex-schultz)
Changed in tripleo:
milestone: none → pike-2
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat-templates (master)

Reviewed: https://review.openstack.org/460215
Committed: https://git.openstack.org/cgit/openstack/heat-templates/commit/?id=6d2659b468bab2630b1498f2f3ae6942ebd87828
Submitter: Jenkins
Branch: master

commit 6d2659b468bab2630b1498f2f3ae6942ebd87828
Author: Alex Schultz <email address hidden>
Date: Wed Apr 26 10:30:11 2017 -0600

    Fix splay config template

    The splay option was added in I88f623c9e8db9ed4a186918206a63faec8f7f673
    but was missing a trailing } which leads to parse errors. This fixes the
    parse errors.

    Change-Id: Ic5783220f5d027e0bde0080f50f9e10cbb68289f
    Closes-Bug: #1686455

Changed in heat-templates:
status: In Progress → Fix Released
Revision history for this message
Emilien Macchi (emilienm) wrote :

Alex, do we still need the alert? Do we need to backport it?

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

AFAICT, the original patch has no backports, so need no the fix. I think the alert mark can go now

tags: removed: alert
Revision history for this message
Alex Schultz (alex-schultz) wrote :

No we appear to be using head in mitaka/newton. I left it open until the jobs passed on the next package build. I'm going to verify those jobs are fixed.

Revision history for this message
Ben Nemec (bnemec) wrote :

Mitaka and Newton jobs are passing again, so I think we can close this out for tripleo.

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.