From d4936d1ca77b14c2c7aaaa99f376219135892fc8 Mon Sep 17 00:00:00 2001 From: Till Maas Date: Mon, 15 Oct 2018 20:55:58 +0200 Subject: [PATCH] Require bridge-utils only for older distributions The initscripts provider uses iproute2 in newer releases to setup bridges. This fixes #84 --- defaults/main.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/defaults/main.yml b/defaults/main.yml index f8d52ee..2a92813 100644 --- a/defaults/main.yml +++ b/defaults/main.yml @@ -21,7 +21,12 @@ network_service_name_default_initscripts: network # initscripts requires bridge-utils to manage bridges, install it when the # 'bridge' type is used in network_connections -_network_packages_default_initscripts_bridge: ["{% if ['bridge'] in network_connections|json_query('[*][type]') %}bridge-utils{% endif %}"] +_network_packages_default_initscripts_bridge: ["{% if ['bridge'] in network_connections|json_query('[*][type]') and +( + (ansible_distribution in ['RedHat', 'CentOS'] and ansible_distribution_major_version is version('7', '<=')) or + (ansible_distribution == 'Fedora' and ansible_distribution_major_version is version('28', '<=')) +) +%}bridge-utils{% endif %}"] # convert _network_packages_default_initscripts_bridge to an empty list if it # contains only the empty string and add it to the default package list # |select() filters the list to include only values that evalueate to true (the empty string is false)