Yaniv Bronhaim has posted comments on this change.

Change subject: persist.py: Replace ovirtfunctions to utils
......................................................................


Patch Set 1: Code-Review-1

since what version of node.utils we have utils.fs.Config().persist? guess 
host-deploy should work also with older versions of utils.. you need to 
consider catching AttributeError here for backward compatibility as alon says

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4b24131bbddbcc0df9454cc617620cd9aff09c8d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabi...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to