If anyone is allowed to update the wiki I can do so as I go along. One thing to note. Because we effectively removed tabularize and you removed my appending the display attribute to the config if it didn't exist we have created a user experience which will not be seamless when they upgrade.
I have begun working in the next set of features. I will make sure I run the pep8 module on it before I submit and keep the code as short as possible. Justin Sent from my iPhone On Apr 5, 2013, at 12:39 PM, "Rohit Yadav" <bhais...@apache.org> wrote: > > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10275/ > > We will deprecate and remove self.tabular and let's call the display type > "table" instead of tabularize as we use json and not jsonize etc.; we need to > update this on wiki when we are done. > I've removed comments, the idea should be to write code such that comments > should not be needed, of course that is my argument, nevertheless the code is > tiny 500 lines of code. > > Applied on master: > > commit ff74cbbeeac726a2f988f0a2fcda78e74f9af64d > Author: Rohit Yadav <bhais...@apache.org> > Date: Fri Apr 5 23:06:44 2013 +0530 > > cli: Remove tabularize, fix display types, make code pep8 compliant > > - Remove tabularize field > - Now display types are: default, json and table > - Make requester pep8 compliant > - Remove unnecessary comments, if we want them we should put them on each > method > > Signed-off-by: Rohit Yadav <bhais...@apache.org> > > commit 9fbae8579923ad7f2823c084e6112b406714652b > Author: Rohit Yadav <bhais...@apache.org> > Date: Fri Apr 5 22:56:07 2013 +0530 > > cli: Make cloudmonkey.py pep8 compliant > > Signed-off-by: Rohit Yadav <bhais...@apache.org> > > commit c75b11d13a49065554d492d5ed2fafc6bff9493e > Author: Justin Grudzien <grudz...@gmail.com> > Date: Wed Apr 3 15:00:00 2013 -0500 > > CLOUDSTACK-1875: add JSON output to cloudmonkey > > Added > 1. display = [default|json|tabularize] has been added in the config to > replace tabularize = [true|false] > 2. tabularize is deprecated but we will still set it as "false" once the > user removes it out of their config to avoid throwing an error. Th > 3. display = "default" is added to the [ui] section of the config if it > is not present. > 4. You can now output JSON formatted text by setting the config display = > json > 5. You can now filter text in JSON output mode. (i.e. list users > account=grudzien filter=account,id,email). Filtered output returns a prope > > Removed > 1. Removed the printing of attr keys in read_config(). > > Deprecated > 1. tabularize = [true|false] is now messaged as deprecated. > > Signed-off-by: Justin Grudzien <grudz...@gmail.com> > > - Rohit > > > On April 4th, 2013, 2:01 p.m., Justin Grudzien wrote: > > Review request for cloudstack and Rohit Yadav. > By Justin Grudzien. > Updated April 4, 2013, 2:01 p.m. > > Description > > Added > 1. display = [default|json|tabularize] has been added in the config to > replace tabularize = [true|false] > 2. tabularize is deprecated but we will still set it as "false" once the user > removes it out of their config to avoid throwing an error. This will be > removed in the next major version. > 3. display = "default" is added to the [ui] section of the config if it is > not present. > 4. You can now output JSON formatted text by setting the config display = json > 5. You can now filter text in JSON output mode. (i.e. list users > account=grudzien filter=account,id,email). Filtered output returns a properly > formatted JSON document. > > Removed > 1. Removed the printing of attr keys in read_config(). > > Deprecated > 1. tabularize = [true|false] is now messaged as deprecated. > Testing > > [ui] > color = true > prompt = > > tabularize = false > display = json > > $ python cloudmonkey.py list networks filter=name > > tabularize config parameter is deprecated: please switch to display = > [default,json,tabularize] > > { > "count": 2, > "network": [ > { > "name": "admin-network" > }, > { > "name": "GuestNetwork82" > } > ] > } > > > $ python cloudmonkey.py list networkofferings filter=name > > tabularize config parameter is deprecated: please switch to display = > [default,json,tabularize] > > { > "count": 8, > "networkoffering": [ > { > "name": "DefaultSharedNetworkOfferingWithSGService" > }, > { > "name": "DefaultSharedNetworkOffering" > }, > { > "name": "DefaultIsolatedNetworkOfferingWithSourceNatService" > }, > { > "name": "DefaultIsolatedNetworkOffering" > }, > { > "name": "DefaultSharedNetscalerEIPandELBNetworkOffering" > }, > { > "name": "DefaultIsolatedNetworkOfferingForVpcNetworks" > }, > { > "name": "DefaultIsolatedNetworkOfferingForVpcNetworksNoLB" > }, > { > "name": "Test Network Offering" > } > ] > } > Bugs: CLOUDSTACK-1875 > Diffs > > tools/cli/cloudmonkey/cloudmonkey.py (94006c9) > tools/cli/cloudmonkey/config.py (75605df) > View Diff