Dan Kenigsberg has submitted this change and it was merged. Change subject: net: introducing KernelConfig ......................................................................
net: introducing KernelConfig KernelConfig can be created from a NetInfo object. It reflects the network kernel state and exposes an API similar to RunningConfig. At this stage, It can compare itself to a RunninConfig instance in order to be tested properly and prove it is 'correct'. Later, it should be able to tell which networks/bonds it obtained from the kernel are different from a given PersistentConfig's networks/bodns. This is useful at boot time in order to allow skipping restoring already configured networks. Since VDSM network api has language differences from netinfo module and also has many default parameters, KernelConfig tries to do 2 things: 1. It tries to be expilict as possible (no defaults) 2. It knows how to 'normalize' a running/persistent config so it would also be explicit. Note the custom properties are never included in KernelConfig, as they only serve as hints to Vdsm and its hooks. Therefore, tests which set custom properties would certainly get into RunningConfig!=KernelConfig. These tests pass test_kernel_config=False so as not to fail. Change-Id: I79f1eb553a42f1398ad12aa1bc33522f8af30c79 Signed-off-by: Ido Barkan <[email protected]> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1203422 Reviewed-on: https://gerrit.ovirt.org/41973 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/42960 Continuous-Integration: Dan Kenigsberg <[email protected]> --- M lib/vdsm/netconfpersistence.py M lib/vdsm/netinfo.py M tests/functional/networkTests.py 3 files changed, 264 insertions(+), 12 deletions(-) Approvals: Ido Barkan: Verified Dan Kenigsberg: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42960 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I79f1eb553a42f1398ad12aa1bc33522f8af30c79 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Ido Barkan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
