Assaf Muller has posted comments on this change.

Change subject: cleanup: move _updateTimestamp to API.py
......................................................................


Patch Set 3:

(1 comment)

....................................................
Commit Message
Line 15: 
Line 16: Ancient Engines used to call getVdsCaps() instead of the lightweight
Line 17: ping(). Even older Engines did nothing special and depended on its
Line 18: periodic list() call. For backward compatibility, we add
Line 19: _updateTimestamp on these specific verbs, too.
Could I ask you to state what versions of oVirt used getVdsCaps and list, and 
possibly add TO-DOs to remove those calls to updateTimestamp when those oVirt 
versions are no longer supported?
Line 20: 
Line 21: Change-Id: I4270df333ff5ff56e4375742cdc526ec76ed3bf1


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4270df333ff5ff56e4375742cdc526ec76ed3bf1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Assaf Muller <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to