Hello Igor Lvovsky,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/6211

to review the following change.

Change subject: configNet: clear up atomicBackup arg
......................................................................

configNet: clear up atomicBackup arg

Apparently, os.path.join('/a/b/', '/a/b/c') == '/a/b/c'.
Let us not trust this peculiarity, and have atomicBackup
expect a full path to the config file.

Change-Id: Id4f4fc8dd1db785837d868a45787b3fa5be901a6
Signed-off-by: Dan Kenigsberg <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/5953
Reviewed-by: Igor Lvovsky <[email protected]>
---
M vdsm/configNetwork.py
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/11/6211/1
--
To view, visit http://gerrit.ovirt.org/6211
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4f4fc8dd1db785837d868a45787b3fa5be901a6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.1
Gerrit-Owner: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to