Dan Kenigsberg has posted comments on this change.

Change subject: Get rid of legacy status 'Running'
......................................................................


Patch Set 1:

(1 comment)

....................................................
Commit Message
Line 6: 
Line 7: Get rid of legacy status 'Running'
Line 8: 
Line 9: The status 'Running' means 'Up' for any case, now we get rid of it
Line 10: completely as this is not helpful having two states for Up.
This deprecation may well be safe. But the commit message should state the 
measures used to prove that it is. Don't misunderstand me: I'd like to drop the 
Running state, and I like to drop it now.
Line 11: 
Line 12: Change-Id: Ie3c6e66365fe02158e5a7bb8db56c724b612b366


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie3c6e66365fe02158e5a7bb8db56c724b612b366
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[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