Gilad Chaplik has posted comments on this change.

Change subject: core, engine, webadmin: Shutdown/power off operations on a VM 
would ask for an optional reason
......................................................................


Patch Set 11: Code-Review-1

Exactly.
and I'm pretty confident about it.

Ading Einav to the thread.

I'm -1 till it gets resolved, hope that's fine with you.

my pro reason to use the comment field instead, is that I remember a discussion 
about it in devel mailing list, about removing this field (comment), because it 
had no actual meaning and we can use description instead, and probably now we 
found one for the field :-)

Thanks Ravi :-)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I17645e5bc97a1a4d460956ec45f88524465dfd7b
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Ravi Nori <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to