container-puppet-keystone_init_tasks seems to be running in multiple steps

Bug #1870291 reported by Rabi Mishra
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Medium
Rabi Mishra

Bug Description

I think it's expected to run only on step3[1]. But seems to be running for all subsequent steps[2].

[1] https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/keystone/keystone-container-puppet.yaml#L774

[2] https://4d54b9492ec449d86ffb-176806fde7fb6bec4df9cbd788be57b5.ssl.cf1.rackcdn.com/716217/1/check/tripleo-ci-centos-8-containers-multinode/3589cbc/logs/undercloud/var/log/extra/podman/containers/container-puppet-keystone_init_tasks/stdout.log

I think the 'host_container_puppet_tasks' variable does not include step, hence it's host-container-puppet-tasks.yaml is included in all subsequent steps.

Rabi Mishra (rabi)
Changed in tripleo:
importance: Undecided → Medium
status: New → Triaged
assignee: nobody → Rabi Mishra (rabi)
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.opendev.org/716879

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

Reviewed: https://review.opendev.org/716879
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=6db0ac0856b369ee13dc17db7be7ef8c124a8b4b
Submitter: Zuul
Branch: master

commit 6db0ac0856b369ee13dc17db7be7ef8c124a8b4b
Author: Rabi Mishra <email address hidden>
Date: Thu Apr 2 12:51:21 2020 +0530

    Include {{step}} when setting facts

    We don't set the fact specific to a step and it seems
    to run those puppet tasks for subsequent steps too.

    Change-Id: Ic276ea6f34dd638ad66505aa77bddff0e4071ad4
    Closes-Bug #1870291

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

Fix proposed to branch: stable/train
Review: https://review.opendev.org/717526

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

Reviewed: https://review.opendev.org/717526
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=86f149ae3059c1a0ab7ec525afe1fc63f29a36a3
Submitter: Zuul
Branch: stable/train

commit 86f149ae3059c1a0ab7ec525afe1fc63f29a36a3
Author: Rabi Mishra <email address hidden>
Date: Thu Apr 2 12:51:21 2020 +0530

    Include {{step}} when setting facts

    We don't set the fact specific to a step and it seems
    to run those puppet tasks for subsequent steps too.

    Change-Id: Ic276ea6f34dd638ad66505aa77bddff0e4071ad4
    Closes-Bug #1870291
    (cherry picked from commit 6db0ac0856b369ee13dc17db7be7ef8c124a8b4b)

tags: added: in-stable-train
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.