Change in vdsm[ovirt-3.5]: netinfotests: Do not assume the existence of /var/lib/vdsm/b...

ybronhei at redhat.com ybronhei at redhat.com
Thu Oct 23 07:41:39 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: netinfotests: Do not assume the existence of /var/lib/vdsm/bonding-defaults.json
......................................................................


netinfotests: Do not assume the existence of /var/lib/vdsm/bonding-defaults.json

The cached bonding defaults file is not available before installation
so it cannot be used in 'make check'. Use the one from the source tree
in this case.

Change-Id: I8ec872739729e11ff9e4a9d30cdac36733d60bae
Signed-off-by: Ondřej Svoboda <osvoboda at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/33776
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
---
M tests/netinfoTests.py
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Ondřej Svoboda: Verified
  Yaniv Bronhaim: Looks good to me, approved
  Antoni Segura Puimedon: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/33776
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8ec872739729e11ff9e4a9d30cdac36733d60bae
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list