Adjust non-puppetish (ansible/external deploys) detection logic for YAML validate hiera interpolation

Bug #1765641 reported by Bogdan Dobrelya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Medium
Bogdan Dobrelya

Bug Description

See https://review.openstack.org/#/c/562809/
We need to come up with something better than just "if 'ansible' in f " to exclude ansible/external deployments from the hiera interpolation checks.

So the given example:

  container_registry_host:
    str_replace:
      template:
        "{{ hostvars.raw_get(groups['docker_registry'][0])['$REGISTRY_ip'] }}"
      params:
        $REGISTRY: {get_param: [ServiceNetMap, DockerRegistryNetwork]}

shall work for non-puppet based deployments in the end.

Tags: tech-debt
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

At the first glance, we could additionally verify if there is no external_deploy_tasks defined?

Changed in tripleo:
milestone: none → rocky-2
importance: Undecided → Medium
status: New → Triaged
tags: added: tech-debt
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/562996

Changed in tripleo:
assignee: nobody → Bogdan Dobrelya (bogdando)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/562996
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=68c60892a1df78d2cd13ff5789cce3aef104cb66
Submitter: Zuul
Branch: master

commit 68c60892a1df78d2cd13ff5789cce3aef104cb66
Author: Bogdan Dobrelya <email address hidden>
Date: Fri Apr 20 11:11:19 2018 +0200

    Omit hiera YAML validations for external deploys

    Closes-bug: #1765641

    Change-Id: I3e98521f6396acc9e4e1a7b32faec85e988ab0fe
    Signed-off-by: Bogdan Dobrelya <email address hidden>

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 9.0.0.0b3

This issue was fixed in the openstack/tripleo-heat-templates 9.0.0.0b3 development milestone.

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.