FFWD: boot_param_tasks.yaml doesn't set BOOTPROTO=none due to conditions are never met

Bug #1786212 reported by Sergii Golovatiuk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Sergii Golovatiuk

Bug Description

Ansible stores the interface data, it transcribes the '-' to '_' so br-ex becomes ansible_br_ex. It's very easy to check that with 'ansible -i inv all -m setup' so

      - replace:
          dest: "{{ item.path }}"
          regexp: '^BOOTPROTO=.*'
          replace: 'BOOTPROTO=none'
        when:

is never executed as when conditions are never met.

Expected result
===============
See 'BOOTPROTO=none' in /etc/sysconfig/network-scripts/ for all active interfaces except lo

Actual result
=============
BOOTPROTO was not changed as conditions were never met.

Changed in tripleo:
importance: Undecided → High
assignee: nobody → Sergii Golovatiuk (sgolovatiuk)
milestone: none → rocky-rc1
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/590208

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

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

commit 6ab5112eb207cd2f87956f1c574d0b70236ef2e3
Author: Sergii Golovatiuk <email address hidden>
Date: Thu Aug 9 12:13:46 2018 +0300

    Fix interfaces in boot_param_tasks

    Ansible stores all interfaces with _ in facts. It's very easy to check by
    running 'ansible -i inv all -m setup'. This patch adds replace - with _ to find
    interface properly in facts.

    Change-Id: Id290143d8a41964806ca9c5be4d7103a3833cb37
    Closes-Bug: #1786212

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.0rc1

This issue was fixed in the openstack/tripleo-heat-templates 9.0.0.0rc1 release candidate.

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.openstack.org/600374

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

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

commit c68f853204186feeca748432a332c402aa4aeeb8
Author: Sergii Golovatiuk <email address hidden>
Date: Thu Aug 9 12:13:46 2018 +0300

    Fix interfaces in boot_param_tasks

    Ansible stores all interfaces with _ in facts. It's very easy to check by
    running 'ansible -i inv all -m setup'. This patch adds replace - with _ to find
    interface properly in facts.

    Closes-Bug: #1786212
    Resolves: rhbz#1614036

    Change-Id: Id290143d8a41964806ca9c5be4d7103a3833cb37
    (cherry picked from commit 6ab5112eb207cd2f87956f1c574d0b70236ef2e3)

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

This issue was fixed in the openstack/tripleo-heat-templates 8.0.6 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.