Hi, As you might know, that the net_online() function (in the network_rcm plugin) is actually a "net_undo_offline()" operation to reverse all operation has been done before offline fails.
As an example, ip_rcm plugin keeps a list of the IP interfaces on a specific device, and ip_undo_offline() will try to replumb all IP interfaces based on the list (which has been unplumbed in ip_offline()). To do the similar after vanity naming, network_rcm needs to keep a list of active aggregations and (explicit created) VLANs which are associated with each specific device, and I expect that code will be complicated. Another alternative is to create aggregations or VLANs based on the persistent configuration. The downside of this alternative is that this might not be a exact reverse operation because of the difference between the persistent configuration and the active configuration. Note that after DR in a device, we will also try to restore all configuration associated with the device, at that time, we can only do that based on the persistent configuration because the active configuration has been long gone. Because of this, I am proposing to do the same (restore based on the persistent configuration) in net_online(). That is also the proposal in the UV design document. I'd like to know whether this could cause any problem. Thanks - Cathy
