+++ This bug was initially created as a clone of Bug #2116633 +++

Description of problem:
While ethtool_opts are added or modified, os-net-config down affected interfaces.
However, ethtool command doesn't require to restart the interface to apply the change.

So, os-net-config should not restart interfaces which have a change of ethtool_opts and apply the change with ethtool command online. 

Version-Release number of selected component (if applicable):
os-net-config-11.3.2-1.20210406083710.f49ab16.el8ost.noarch

How reproducible:
Everytime to apply the change of ethtool_opts.

Steps to Reproduce:
1. Setup network interface with os-net-config
2. Add or modify ethtool_opts in config.json
3. Run os-net-config.

Actual results:
Affected interfaces are restarted.

Expected results:
Affected interfaces are not restarted and apply the change by ethtool online.


Additional info:

--- Additional comment from Karthik Sundaravel on 2022-08-10 17:16:49 JST ---

os-net-config generates the ifcfg files based on config.json. If there is a change in the current ifcfg vs new one, it performs ifdown and ifup for the same. Since the issue is not reproducible with network-scripts can you please help us with more clarity on why the routes are not added back. 

Also if the change in ethtool_opts needs to be handled without restarting the interface it would be a RFE.

--- Additional comment from Keigo Noha on 2022-08-10 17:26:27 JST ---

Hi Karthik,

Currently, we're working with SBR-Networking team for the issue that routes are not back.
On the other hand, changing or adding ETHTOOL_OPTS can be applied online. So, the current implementation has a room to improve.

Once I get a result from the investigation from networking side, I'll update this case.

Best Regards,
Keigo Noha

--- Additional comment from Keigo Noha on 2022-08-15 09:29:20 JST ---

Hi Karthik,

I can reproduce the issue manually.

The key thing is what interface runs ifdown/ifup.

The proper down step is

# ifdown eth0
# ifdown eth1
# ifdown bond0
# ifup eth0
# ifup eth1
# ifup bond0

This correct steps doesn't trigger the issue.

However, os-net-config did the following steps.

# ifdown eth0
# ifdown eth1
# ifup eth0
# ifup eth1
# ifup bond0

The way in os-net-config lacks 'ifdown bond0'. Due to that, ifup-routes are not run.
I think we should create a new bugzilla to focus on adding steps to run ifdown bond0.
I'll clone this bugzilla for adding steps to run ifdown bond0.

Best Regards,
Keigo Noha