Nir Soffer has posted comments on this change.
Change subject: netinfo: remove an assertion from getBondingForNic ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/45072/1/lib/vdsm/netinfo.py File lib/vdsm/netinfo.py:
Line 898 Line 899 Line 900 Line 901 Line 902 This change is wrong.
asserts are designed exactly for this use case; It is impossible to have more then one bonding, so we assert that this is indeed true. This assert protect us from the impossible, and document our intent.
Without the assert, if our assumption is incorrect, we will silently ignore the reality.