)]}'
{"id":"openstack%2Fpython-openstackclient~276984","triplet_id":"openstack%2Fpython-openstackclient~master~Idc54ad21eaa1371ebd601327b8d962c7039f2de0","project":"openstack/python-openstackclient","branch":"master","topic":"bug/1477199","hashtags":[],"change_id":"Idc54ad21eaa1371ebd601327b8d962c7039f2de0","subject":"Compute: Fix DisplayCommandBase comments for cliff Lister subclass tests","status":"MERGED","created":"2016-02-06 02:51:17.000000000","updated":"2016-02-08 08:25:39.000000000","submitted":"2016-02-08 08:25:38.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b47cd48847c9c4600bd318b9f5608ba164bd0e17","_number":276984,"virtual_id_number":276984,"owner":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":2,"date":"2016-02-08 08:25:38.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"value":2,"date":"2016-02-08 06:45:45.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-02-08 07:20:22.000000000","_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"all":[{"value":0,"date":"2016-02-08 02:54:34.000000000","_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-02-08 07:20:22.000000000","_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"value":0,"date":"2016-02-08 02:54:28.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-02-08 02:54:28.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2016-02-08 02:54:34.000000000","updated_by":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"reviewer":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"state":"REVIEWER"},{"updated":"2016-02-08 06:45:45.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2016-02-08 07:20:22.000000000","updated_by":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"reviewer":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"state":"REVIEWER"},{"updated":"2016-02-08 08:25:38.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"3d7a1d832aded5c3f0b0db240a3f5a7f42bc9270","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-02-06 02:51:17.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"21d616f349c9f51dafcaed614bdf7d4869ba799a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-06 03:42:52.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-pep8 http://logs.openstack.org/84/276984/1/check/gate-python-openstackclient-pep8/8f1ae4d/ : SUCCESS in 1m 00s\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/84/276984/1/check/gate-python-openstackclient-docs/5be1805//doc/build/html/ : SUCCESS in 1m 29s\n- gate-python-openstackclient-python27 http://logs.openstack.org/84/276984/1/check/gate-python-openstackclient-python27/9b0389e/ : SUCCESS in 1m 23s\n- gate-python-openstackclient-python34 http://logs.openstack.org/84/276984/1/check/gate-python-openstackclient-python34/ce276e3/ : SUCCESS in 2m 17s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/84/276984/1/check/gate-tempest-dsvm-neutron-src-python-openstackclient/1d43f21/ : SUCCESS in 51m 24s\n- check-osc-plugins http://logs.openstack.org/84/276984/1/check/check-osc-plugins/9c64714/ : SUCCESS in 1m 23s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/84/276984/1/check/gate-osc-dsvm-functional/ea646f6/ : SUCCESS in 19m 18s","accounts_in_message":[],"_revision_number":1},{"id":"96d2c4b850c28520bfcf5ce629a43f6ef6e9ad2e","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-02-08 06:45:45.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"430551660bcfb12975972c4c7798da50baaa5dfa","author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"date":"2016-02-08 07:20:22.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"63dbc23bb7ef1edbd08cf27f37bde8a7ff29046a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-08 07:20:59.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"91434df6a8304e96fdfbfcc65b5fb25f0195166f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-08 08:25:38.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/84/276984/1/gate/gate-python-openstackclient-docs/b90de7e//doc/build/html/ : SUCCESS in 1m 57s\n- gate-python-openstackclient-pep8 http://logs.openstack.org/84/276984/1/gate/gate-python-openstackclient-pep8/4cd965d/ : SUCCESS in 1m 00s\n- gate-python-openstackclient-python27 http://logs.openstack.org/84/276984/1/gate/gate-python-openstackclient-python27/db3128d/ : SUCCESS in 1m 05s\n- gate-python-openstackclient-python34 http://logs.openstack.org/84/276984/1/gate/gate-python-openstackclient-python34/74622e6/ : SUCCESS in 1m 54s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/84/276984/1/gate/gate-tempest-dsvm-neutron-src-python-openstackclient/1cad2ac/ : SUCCESS in 43m 22s\n- gate-osc-dsvm-functional http://logs.openstack.org/84/276984/1/gate/gate-osc-dsvm-functional/7f8c9c4/ : SUCCESS in 22m 42s","accounts_in_message":[],"_revision_number":1},{"id":"5951c6eb3d1379f6d79a7c48cbd77440fce8cd50","date":"2016-02-08 08:25:39.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"3c67e8dd6ed20f9a3d6c1f8f000214cc7a8f384e","revisions":{"3c67e8dd6ed20f9a3d6c1f8f000214cc7a8f384e":{"kind":"REWORK","_number":1,"created":"2016-02-06 02:51:17.000000000","uploader":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"ref":"refs/changes/84/276984/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/84/276984/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/84/276984/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/84/276984/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/84/276984/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/84/276984/1"}}},"commit":{"parents":[{"commit":"b9de23d906a0ba640c336006e1b373a079846995","subject":"Compute: Fix DisplayCommandBase comments for cliff Command subclass tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/b9de23d906a0ba640c336006e1b373a079846995"}]}],"author":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-06 02:30:34.000000000","tz":480},"committer":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-06 02:30:34.000000000","tz":480},"subject":"Compute: Fix DisplayCommandBase comments for cliff Lister subclass tests","message":"Compute: Fix DisplayCommandBase comments for cliff Lister subclass tests\n\nAs bug #1477199 describes, the wrong comment below is all over the\nunit test code of OSC.\n\n    # DisplayCommandBase.take_action() returns two tuples\n\nThere is no such class named DisplayCommandBase in OSC. It is in cliff.\n\nAll OSC command classes inherit from the base classes in cliff,\nclass Command, class Lister and class ShowOne. It is like this:\n\nObject\n|--\u003e Command\n     |--\u003e DisplayCommandBase\n         |--\u003e Lister\n         |--\u003e ShowOne\n\ntake_action() is an abstract method of class Command, and generally is\noverwritten by subclasses.\n\n* Command.take_action() returns nothing.\n* Lister.take_action() returns a tuple which contains a tuple of\n  columns and a generator used to generate the data.\n* ShowOne.take_action() returns an iterator which contains a tuple\n  of columns and a tuple of data\n\nSo, this problem should be fixed in 3 steps:\n1. Remove all DisplayCommandBase comments for tests of classes\n   inheriting from class Command in cliff as it returns nothing.\n2. Fix all DisplayCommandBase comments for tests of classes\n   inheriting from class Lister in cliff. Lister.take_action()\n   returns a tuple and a generator.\n3. Fix all DisplayCommandBase comments for tests of classes\n   inheriting from class ShowOne in cliff. ShowOne.take_action()\n   returns two tuples.\n\nThis patch finishes step 2 in compute tests.\n\nChange-Id: Idc54ad21eaa1371ebd601327b8d962c7039f2de0\nPartial-bug: #1477199\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/3c67e8dd6ed20f9a3d6c1f8f000214cc7a8f384e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/3c67e8dd6ed20f9a3d6c1f8f000214cc7a8f384e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}