From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: Remove the redundant Configurator.begin method ......................................................................
net: Remove the redundant Configurator.begin method
The begin() method included duplicated initializations of the configurator instance. It has been called when the configurator is used as a context manager.
As the configurator is always instantiated when used as a context manager, the duplication of initialization is redundant and therefore the begin() method is removed.
Change-Id: I8be59e9e9937378dae67ff9be111d916d8631023 Signed-off-by: Edward Haas edwardh@redhat.com --- M lib/vdsm/network/configurators/__init__.py M lib/vdsm/network/configurators/ifcfg.py M lib/vdsm/network/configurators/iproute2.py 3 files changed, 0 insertions(+), 16 deletions(-)
Approvals: Jenkins CI: Passed CI tests Petr Horáček: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Edward Haas: Verified