Dan Kenigsberg has posted comments on this change.
Change subject: Move guest agent timeout config reading to clientIF.__init__
......................................................................
Patch Set 1: I would prefer that you didn't submit this
(1 inline comment)
....................................................
File vdsm/clientIF.py
Line 99
Line 100
Line 101
Line 102
Line 103
this is already read (and used) in __init__ only.
why do we need it as a (public) data member? your commit message does not give a hint.
--
To view, visit
http://gerrit.ovirt.org/9396
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id6138f2b47934cb8afc89cd9a2d14867d9db03a9
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server