Ticket: CM-3208
Reviewed By:
Testing Done: Tested with testcase listed in the bug
This patch does the following:
- moves the interface error exit check to before upperifaces are brought
up
- changes errors to warns on upperiface error (this is because
upperiface 'up' is done as best effort to reconfigure the interface in
question as slave device to the upper device. But if the upper device
is not in a right state config steps can fail. And we should just
warn).
- Implicitly bringing up the upperifaces helps in most of the cases. especially
when a bond is brought down and up. The upperiface handling code adds
the bond back into bridges it was part of. or creates the vlan devices
on the bond that got deleted. But there can be cases where upperifaces are
not in the right state and this results in warnings.
To disable the implicit upperiface handling, this patch also supports
'skip_upperifaces=1' in /etc/network/ifupdown2/ifupdown2.conf
in future, i am thinking of an option --skip-upperifaces to ifup
Ticket: CM-3193
Reviewed By:
Testing Done: Tested logging of errors in syslog from ifup at bootup
Example error msg:
"Jul 10 03:13:17 cumulus /etc/init.d/networking[1183]: error:
/etc/network/interfaces: line16: invalid auto line 'auto'"
This patch logs into syslog only during bootup
errors to warns
Ticket: CM-3159
Reviewed By: briefly ran this by jtoppins and andy (sfeldma is on
vacation this week).
Testing Done: tested ifupdown2 with ethtool config during boot (sam will
also be adding the testcase mentioned in the bug to ifupdown2 smoke)
The kernel timeout increase helps right now.
we should revisit this again in 2.3 to close all corner cases.
ifupdown2 will now warn on ethtool errors and will also return
non-zero exit status
Ticket: CM-3176
Reviewed By: trivial
Testing Done: Tested ifreload with the testcase in the bug
This broke when i recently fixed --allow-classes support for ifup/ifdown
Ticket: CM-3007
Reviewed By: shm + patch was pasted in the bug for review
Testing Done: ran precommit + maliks test + malik ran his test on his
box
When -a is specified ifupdown2 works on all interfaces and since the the
upperiface check is a bit expensive i had a "skip" on that.
And so far all the user commands i have seen only work on the $IFACE and
not its dependents. So, never hit this case.
Ticket: CM-3208
Reviewed By:
Testing Done: Tested with testcase listed in the bug
This patch does the following:
- moves the interface error exit check to before upperifaces are brought
up
- changes errors to warns on upperiface error (this is because
upperiface 'up' is done as best effort to reconfigure the interface in
question as slave device to the upper device. But if the upper device
is not in a right state config steps can fail. And we should just
warn).
- Implicitly bringing up the upperifaces helps in most of the cases. especially
when a bond is brought down and up. The upperiface handling code adds
the bond back into bridges it was part of. or creates the vlan devices
on the bond that got deleted. But there can be cases where upperifaces are
not in the right state and this results in warnings.
To disable the implicit upperiface handling, this patch also supports
'skip_upperifaces=1' in /etc/network/ifupdown2/ifupdown2.conf
in future, i am thinking of an option --skip-upperifaces to ifup
Ticket: CM-3193
Reviewed By:
Testing Done: Tested logging of errors in syslog from ifup at bootup
Example error msg:
"Jul 10 03:13:17 cumulus /etc/init.d/networking[1183]: error:
/etc/network/interfaces: line16: invalid auto line 'auto'"
This patch logs into syslog only during bootup
errors to warns
Ticket: CM-3159
Reviewed By: briefly ran this by jtoppins and andy (sfeldma is on
vacation this week).
Testing Done: tested ifupdown2 with ethtool config during boot (sam will
also be adding the testcase mentioned in the bug to ifupdown2 smoke)
The kernel timeout increase helps right now.
we should revisit this again in 2.3 to close all corner cases.
ifupdown2 will now warn on ethtool errors and will also return
non-zero exit status
Ticket: CM-3176
Reviewed By: trivial
Testing Done: Tested ifreload with the testcase in the bug
This broke when i recently fixed --allow-classes support for ifup/ifdown
Ticket: CM-3208
Reviewed By:
Testing Done: Tested with testcase listed in the bug
This patch does the following:
- moves the interface error exit check to before upperifaces are brought
up
- changes errors to warns on upperiface error (this is because
upperiface 'up' is done as best effort to reconfigure the interface in
question as slave device to the upper device. But if the upper device
is not in a right state config steps can fail. And we should just
warn).
- Implicitly bringing up the upperifaces helps in most of the cases. especially
when a bond is brought down and up. The upperiface handling code adds
the bond back into bridges it was part of. or creates the vlan devices
on the bond that got deleted. But there can be cases where upperifaces are
not in the right state and this results in warnings.
To disable the implicit upperiface handling, this patch also supports
'skip_upperifaces=1' in /etc/network/ifupdown2/ifupdown2.conf
in future, i am thinking of an option --skip-upperifaces to ifup
Ticket: CM-3193
Reviewed By:
Testing Done: Tested logging of errors in syslog from ifup at bootup
Example error msg:
"Jul 10 03:13:17 cumulus /etc/init.d/networking[1183]: error:
/etc/network/interfaces: line16: invalid auto line 'auto'"
This patch logs into syslog only during bootup
errors to warns
Ticket: CM-3159
Reviewed By: briefly ran this by jtoppins and andy (sfeldma is on
vacation this week).
Testing Done: tested ifupdown2 with ethtool config during boot (sam will
also be adding the testcase mentioned in the bug to ifupdown2 smoke)
The kernel timeout increase helps right now.
we should revisit this again in 2.3 to close all corner cases.
ifupdown2 will now warn on ethtool errors and will also return
non-zero exit status
Ticket: CM-3176
Reviewed By: trivial
Testing Done: Tested ifreload with the testcase in the bug
This broke when i recently fixed --allow-classes support for ifup/ifdown
Ticket: CM-3007
Reviewed By: shm + patch was pasted in the bug for review
Testing Done: ran precommit + maliks test + malik ran his test on his
box
When -a is specified ifupdown2 works on all interfaces and since the the
upperiface check is a bit expensive i had a "skip" on that.
And so far all the user commands i have seen only work on the $IFACE and
not its dependents. So, never hit this case.
Ticket: CM-3007
Reviewed By: shm + patch was pasted in the bug for review
Testing Done: ran precommit + maliks test + malik ran his test on his
box
When -a is specified ifupdown2 works on all interfaces and since the the
upperiface check is a bit expensive i had a "skip" on that.
And so far all the user commands i have seen only work on the $IFACE and
not its dependents. So, never hit this case.
