Yair Zaslavsky has posted comments on this change.

Change subject: tools : rhevm-manage-domains location option no longer valid 
(#871085)
......................................................................


Patch Set 1: (1 inline comment)

Hi Ravi,
Can you please also check what happens with action=list ?
I am quite sure you will get prints for each domain that is is a remote domain. 
This information is redundant (no need to mention the domain is remote).

Thanks!

....................................................
Commit Message
Line 3: AuthorDate: 2012-10-31 23:29:48 -0400
Line 4: Commit:     Ravi Nori <rn...@redhat.com>
Line 5: CommitDate: 2012-10-31 23:37:22 -0400
Line 6: 
Line 7: tools : rhevm-manage-domains location option no longer valid (#871085)
Please do not mention rhevm upstream.
Do not include bug number at title.
Line 8: 
Line 9: In command line parameters for rhevm-manage-domains the -location 
(=remote/local)
Line 10: parameter is an obsolete option but the command gives no error when 
added to the
Line 11: command, and has no effect.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3384c6981d38b03d2f2856ff8af69ac9d1c65056
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to