Change in vdsm[master]: net: ovs: remove initial nic ip, dont touch unchanged networks

danken at redhat.com danken at redhat.com
Thu Nov 26 10:04:38 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: ovs: remove initial nic ip, dont touch unchanged networks
......................................................................


net: ovs: remove initial nic ip, dont touch unchanged networks

According to ifcfg configurator behavior, now we remove inital IP
configuration of nics attached to VDSM network.

Also there is an improvement in IP changes handling: If IP configuration
was not changed, don't recreate it.

Signed-off-by: Petr Horáček <phoracek at redhat.com>
Change-Id: Ie9682c75cfb79f75b64cb8cb587fc3ed01cf15c8
Reviewed-on: https://gerrit.ovirt.org/46692
Continuous-Integration: Jenkins CI
Reviewed-by: Ido Barkan <ibarkan at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm_hooks/ovs/README
M vdsm_hooks/ovs/ovs_before_network_setup_ip.py
2 files changed, 41 insertions(+), 11 deletions(-)

Approvals:
  Ido Barkan: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/46692
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie9682c75cfb79f75b64cb8cb587fc3ed01cf15c8
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list