keepalived container fails to configure correctly with facter 3

Bug #1812938 reported by Alex Schultz on 2019-01-23
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
High
Alex Schultz

Bug Description

The interface_for_ip function assumes facter 2 syntax for facts but under fedora 28 we end up with a newer facter that no longer provides the interface_<interface> facts.

This results in the keepalived container failing to start because the 'interface' config line doesn't have an interface configured.

Fix proposed to branch: master
Review: https://review.openstack.org/632591

Changed in tripleo:
status: Triaged → In Progress
summary: - keepalived container fails to configure correctly with facter
+ keepalived container fails to configure correctly with facter 3
Bogdan Dobrelya (bogdando) wrote :

sigh, is it yey the time to abandon KA for the unified pacemaker tool used for single node and multi node?

tags: added: tech-debt

Reviewed: https://review.openstack.org/632591
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=5fff984cecf68c256e60ba6017f37b9d1e76e31b
Submitter: Zuul
Branch: master

commit 5fff984cecf68c256e60ba6017f37b9d1e76e31b
Author: Alex Schultz <email address hidden>
Date: Tue Jan 22 17:37:29 2019 -0700

    Fix interface_for_ip for newer facter versions

    In facter 3.x, the networking facts are in a complex hash. Previously
    the interface facts where flat so we would lookup the vars. This change
    first tries the facter 2.x lookup before falling back to the facter 3.x
    syntax.

    Change-Id: I260b18ae3479da4f5176027d4b26b3e1c6b22063
    Closes-Bug: #1812938

Changed in tripleo:
status: In Progress → Fix Released

This issue was fixed in the openstack/puppet-tripleo 10.3.0 release.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers