Dan Kenigsberg has submitted this change and it was merged.
Change subject: configNet: clear up atomicBackup arg
......................................................................
configNet: clear up atomicBackup arg
Apparently, os.path.join('/a/b/', '/a/b/c') == '/a/b/c'.
In my opinion, it is a Python bug.
In any case, 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 <danken(a)redhat.com>
---
M vdsm/configNetwork.py
1 file changed, 4 insertions(+), 4 deletions(-)
Approvals:
Dan Kenigsberg: Verified
Igor Lvovsky: Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/5953
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id4f4fc8dd1db785837d868a45787b3fa5be901a6
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>