ssl_ca render wrong yaml syntax in identity.yaml

Bug #1835490 reported by José Pekkarinen
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Glance-Simplestreams-Sync Charm
Fix Released
High
Alex Kavanagh

Bug Description

Hi,

As per the template, the render of this config parameter looks like:

{% if ssl_ca -%}
ssl_ca: {{ ssl_ca }}
{% endif -%}

This renders in a broken config that will eventually look like:

ssl_ca: LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUZ2ekNDQTZlZ0F3SUJBZ0lRRTFiZmFSdkhY
YUJKYWh4bk02NmszakFOQmdrcWhraUc5dzBCQVEwRkFEQkIKTVJNd0VRWUtDWkltaVpQeUxHUUJH
...

This parameter should look like a multiline comment instead:

ssl_ca: |
  LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUZ2ekNDQTZlZ0F3SUJBZ0lRRTFiZmFSdkhY
  YUJKYWh4bk02NmszakFOQmdrcWhraUc5dzBCQVEwRkFEQkIKTVJNd0VRWUtDWkltaVpQeUxHUUJH
...

Thanks!

José.

Tags: cdo-qa
Revision history for this message
José Pekkarinen (koalinux) wrote :
Changed in charm-glance-simplestreams-sync:
milestone: none → 19.07
David Ames (thedac)
Changed in charm-glance-simplestreams-sync:
milestone: 19.07 → 19.10
Revision history for this message
Dmitrii Shcherbakov (dmitriis) wrote :

Subscribed field-medium as glance-simplestreams-sync usage is mentioned in the official guide for using octavia-diskimage-retrofit:

https://docs.openstack.org/project-deploy-guide/charm-deployment-guide/latest/app-octavia.html#amphora-image

tags: added: cdo-qa
David Ames (thedac)
Changed in charm-glance-simplestreams-sync:
milestone: 19.10 → 20.01
Changed in charm-glance-simplestreams-sync:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Alex Kavanagh (ajkavanagh)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-glance-simplestreams-sync (master)

Fix proposed to branch: master
Review: https://review.opendev.org/692537

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on charm-glance-simplestreams-sync (master)

Change abandoned by Alex Kavanagh (tinwood) (<email address hidden>) on branch: master
Review: https://review.opendev.org/692537
Reason: Abandoning as it's a duplicate of https://review.opendev.org/#/q/I6dca3e2445286423a785e1c0973fa7e1d43b6797

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-glance-simplestreams-sync (master)

Reviewed: https://review.opendev.org/669617
Committed: https://git.openstack.org/cgit/openstack/charm-glance-simplestreams-sync/commit/?id=02299812b44adf25f5b18cbe3c1b734283c1ae29
Submitter: Zuul
Branch: master

commit 02299812b44adf25f5b18cbe3c1b734283c1ae29
Author: Chris MacNaughton <email address hidden>
Date: Mon Jul 8 10:09:40 2019 +0200

    SSL CS certs are multi-line values

    When rendering the multi-line string in the identity.yaml
    file, it is necessary to denote the multi-line string.

    Change-Id: I6dca3e2445286423a785e1c0973fa7e1d43b6797
    Closes-Bug: #1835490

Changed in charm-glance-simplestreams-sync:
status: In Progress → Fix Committed
James Page (james-page)
Changed in charm-glance-simplestreams-sync:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Patches

Remote bug watches

Bug watches keep track of this bug in other bug trackers.