Francesco Romani has submitted this change and it was merged.

Change subject: ovs: do not call ovs_info if ovs service is down
......................................................................


ovs: do not call ovs_info if ovs service is down

PPC downstream does not require OVS, therefore it is not guaranteed,
that openvswitch.service is always running. We have to handle such
cases in network/netswitch.py.

Calling ovs_net2bridge() when openvswitch service is down leads to:
(21, 'Executing commands failed: ovs-vsctl:
unix:/var/run/openvswitch/db.sock: database connection failed (No
such file or directory)')

With this patch, we just return None if OVS service is not running.

Change-Id: I8f28234cf8913d964ee60b3f396f6b62cec03383
Bug-Url: https://bugzilla.redhat.com/1195208
Signed-off-by: Petr Horáček <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/61241
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M lib/vdsm/network/netswitch.py
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8f28234cf8913d964ee60b3f396f6b62cec03383
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Petr Horáček <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Edward Haas <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]

Reply via email to