Change in vdsm[ovirt-4.1]: net: drop special ovirt-node handling from netconfpersistence

2017-01-03 Thread Code Review
From Dan Kenigsberg :

Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: drop special ovirt-node handling from netconfpersistence
..


net: drop special ovirt-node handling from netconfpersistence

The removed code was introduced in https://gerrit.ovirt.org/50984
to solve https://bugzilla.redhat.com/1293617 in vintage ovirt-node

In current ovirt-node implementation, files do not have any different
owneership or permission. So this code can be dropped.

Change-Id: I8ee5300a6e8a59638d7754d1f66c13118bdd7edd
Signed-off-by: Petr Horáček 
Bug-Url: https://bugzilla.redhat.com/1349391
---
M lib/vdsm/network/netconfpersistence.py
M tests/network/conf_persistence_test.py
2 files changed, 0 insertions(+), 11 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/69496
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8ee5300a6e8a59638d7754d1f66c13118bdd7edd
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
Gerrit-Owner: Petr Horáček 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček 
Gerrit-Reviewer: gerrit-hooks 
___
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org


Change in vdsm[ovirt-4.1]: net: drop special ovirt-node handling from netconfpersistence

2017-01-03 Thread Code Review
From Dan Kenigsberg :

Dan Kenigsberg has posted comments on this change.

Change subject: net: drop special ovirt-node handling from netconfpersistence
..


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8ee5300a6e8a59638d7754d1f66c13118bdd7edd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
Gerrit-Owner: Petr Horáček 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Edward Haas 
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček 
Gerrit-Reviewer: gerrit-hooks 
Gerrit-HasComments: No
___
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org