Skip to content

Commit 0212b35

Browse files
committed
Fix invocations to nmcli
1 parent ccdcfa5 commit 0212b35

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

lib/mm_network.pm

+6-6
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,11 @@ sub configure_static_ip {
5858
my $ip = $args{ip};
5959
my $mtu = $args{mtu} // get_var('MM_MTU', 1380);
6060
my $is_nm = $args{is_nm} // is_networkmanager();
61-
my $device = $args{device};
61+
my $device = $args{device} // '\S';
6262

6363
if ($is_nm) {
6464
my $nm_id;
65-
my $nm_list = script_output("nmcli -t -f DEVICE,NAME c | grep -v ^lo: | grep '$device' | head -n1");
65+
my $nm_list = script_output("nmcli -t -f DEVICE,NAME c | grep -v '^lo:' | grep -e '$device' | head -n1");
6666
($device, $nm_id) = split(':', $nm_list);
6767

6868
record_info('set_ip', "Device: $device\n NM ID: $nm_id\nIP: $ip\nMTU: $mtu");
@@ -105,11 +105,11 @@ sub configure_dhcp {
105105
sub configure_default_gateway {
106106
my (%args) = @_;
107107
my $is_nm = $args{is_nm} // is_networkmanager();
108-
my $device = $args{device};
108+
my $device = $args{device} // '\S';
109109
if ($is_nm) {
110110
my $nm_id;
111111
# When $device is not specified grep just does nothing and first connection is selected
112-
my $nm_list = script_output("nmcli -t -f DEVICE,NAME c | grep -v ^lo: | grep '$device' | head -n1");
112+
my $nm_list = script_output("nmcli -t -f DEVICE,NAME c | grep -v '^lo:' | grep -e '$device' | head -n1");
113113
($device, $nm_id) = split(':', $nm_list);
114114

115115
assert_script_run "nmcli connection modify '$nm_id' ipv4.gateway 10.0.2.2";
@@ -127,8 +127,8 @@ sub configure_static_dns {
127127
my $servers = join(" ", @{$conf->{nameserver}});
128128

129129
if ($is_nm) {
130-
$nm_id = script_output('nmcli -t -f NAME c | grep -v ^lo: | head -n 1') unless ($nm_id);
131-
130+
$nm_id = script_output("nmcli -t -f NAME c | grep -v '^lo' | head -n 1") unless ($nm_id);
131+
enter_cmd("clear");
132132
assert_script_run "nmcli connection modify '$nm_id' ipv4.dns '$servers'";
133133
} else {
134134
assert_script_run("sed -i -e 's|^NETCONFIG_DNS_STATIC_SERVERS=.*|NETCONFIG_DNS_STATIC_SERVERS=\"$servers\"|' /etc/sysconfig/network/config");

0 commit comments

Comments
 (0)