Hello community, here is the log from the commit of package monitoring-plugins-bonding for openSUSE:Factory checked in at 2018-08-02 15:00:11 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/monitoring-plugins-bonding (Old) and /work/SRC/openSUSE:Factory/.monitoring-plugins-bonding.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "monitoring-plugins-bonding" Thu Aug 2 15:00:11 2018 rev:6 rq:626961 version:0.002 Changes: -------- --- /work/SRC/openSUSE:Factory/monitoring-plugins-bonding/monitoring-plugins-bonding.changes 2018-07-19 23:01:56.856656595 +0200 +++ /work/SRC/openSUSE:Factory/.monitoring-plugins-bonding.new/monitoring-plugins-bonding.changes 2018-08-02 15:00:58.672564584 +0200 @@ -1,0 +2,5 @@ +Thu Jul 26 01:12:10 CEST 2018 - ro@suse.de + +- fix last change for non-lacp interfaces + +------------------------------------------------------------------- ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ check_bonding.pl ++++++ --- /var/tmp/diff_new_pack.X3HKqa/_old 2018-08-02 15:00:59.412565837 +0200 +++ /var/tmp/diff_new_pack.X3HKqa/_new 2018-08-02 15:00:59.412565837 +0200 @@ -88,12 +88,16 @@ } foreach (keys %{$data->{'slaves'}}) { my $result = $data->{'slaves'}->{$_}->{'mii'}; - $result = "churned" unless $data->{'slaves'}->{$_}->{'actor-churn'} eq 'none'; - $result = "churned" unless $data->{'slaves'}->{$_}->{'partner-churn'} eq 'none'; + $result = "churned" if $data->{'slaves'}->{$_}->{'actor-churn'} + && $data->{'slaves'}->{$_}->{'actor-churn'} ne 'none'; + $result = "churned" if $data->{'slaves'}->{$_}->{'partner-churn'} + && $data->{'slaves'}->{$_}->{'partner-churn'} ne 'none'; $config_str.= " $_ (" . $result . ")"; $error = 1 unless $data->{'slaves'}->{$_}->{'mii'} eq 'up'; - $error = 1 unless $data->{'slaves'}->{$_}->{'actor-churn'} eq 'none'; - $error = 1 unless $data->{'slaves'}->{$_}->{'partner-churn'} eq 'none'; + $error = 1 if $data->{'slaves'}->{$_}->{'actor-churn'} + && $data->{'slaves'}->{$_}->{'actor-churn'} ne 'none'; + $error = 1 if $data->{'slaves'}->{$_}->{'partner-churn'} + && $data->{'slaves'}->{$_}->{'partner-churn'} ne 'none'; $error = 2 if ($data->{'status'} ne 'up'); } return $error, $config_str;