)]}'
{"id":"openstack%2Ftempest~26515","triplet_id":"openstack%2Ftempest~master~Ib4333483c5734d14cac6376837ca770e96bd050e","project":"openstack/tempest","branch":"master","topic":"cleanup-smoke","hashtags":[],"change_id":"Ib4333483c5734d14cac6376837ca770e96bd050e","subject":"Remove smoke.py and clean up base test classes","status":"MERGED","created":"2013-04-09 18:35:09.000000000","updated":"2013-04-10 14:49:35.000000000","submitted":"2013-04-10 14:49:35.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"fc092ed2d9726d57a166179dd0d5c0ca2da4ed17","_number":26515,"virtual_id_number":26515,"owner":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-04-10 14:49:35.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"value":0,"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"}],"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":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"recommended":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-04-09 19:45:47.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"value":0,"date":"2013-04-10 14:03:18.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":2,"date":"2013-04-09 19:58:31.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"value":1,"date":"2013-04-10 14:03:18.000000000","_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"date":"2013-04-09 19:58:31.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"date":"2013-04-09 19:26:33.000000000","_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"value":0,"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"}],"values":{"-1":"Release Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-04-09 19:45:47.000000000","updated_by":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"reviewer":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"state":"REVIEWER"},{"updated":"2013-04-09 19:58:31.000000000","updated_by":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"reviewer":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"state":"REVIEWER"},{"updated":"2013-04-10 14:03:18.000000000","updated_by":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"reviewer":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"state":"REVIEWER"},{"updated":"2013-04-10 14:49:35.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"27d0fc9bfd04123da0b2ed692a8a07ecb724ad59","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-09 18:38:27.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/26515/1/check/gate-tempest-pep8/2949 : FAILURE\n- gate-tempest-devstack-vm-full : SKIPPED\n- gate-tempest-devstack-vm-postgres-full : SKIPPED\n- gate-tempest-devstack-vm-quantum : SKIPPED\n- gate-tempest-devstack-vm-quantum-full : SKIPPED (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"e06ab027c3adc74bc068828fdbfb641a84608ff7","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2013-04-09 18:53:58.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nNice cleanup.  Minor nit inline.","accounts_in_message":[],"_revision_number":1},{"id":"e5823e73a729651b1e93da7a60db92267769890c","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2013-04-09 19:26:33.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"1a0d3820a800042ddffa0f522a6596b2f4dc827d","author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"date":"2013-04-09 19:26:44.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"9aac6171b331e7d4191b08570cbeb4238af83ce5","author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"date":"2013-04-09 19:45:47.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nThank you!","accounts_in_message":[],"_revision_number":2},{"id":"a906df10f9dbc37e4260058d218211e38c31d30a","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2013-04-09 19:58:31.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"8cdec68076eb3da4982adafbd4c1c85415aab12e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-09 20:16:07.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/26515/2/check/gate-tempest-pep8/2950 : SUCCESS\n- http://logs.openstack.org/26515/2/check/gate-tempest-devstack-vm-full/12088 : SUCCESS\n- http://logs.openstack.org/26515/2/check/gate-tempest-devstack-vm-postgres-full/11803 : SUCCESS\n- http://logs.openstack.org/26515/2/check/gate-tempest-devstack-vm-quantum/17885 : SUCCESS\n- http://logs.openstack.org/26515/2/check/gate-tempest-devstack-vm-quantum-full/1101 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"dd62ac23bb4dcd0015ed32ac5a9a9e239b11a4b2","author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"date":"2013-04-10 14:03:18.000000000","message":"Patch Set 2: Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"0c15ec4bd10206826a700a9f6bdda49086561590","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-10 14:03:20.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"27a89d1ec8a4f7f4d0d7f56781874c6543e887d3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-10 14:49:35.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/26515/2/gate/gate-tempest-pep8/2960 : SUCCESS\n- http://logs.openstack.org/26515/2/gate/gate-tempest-devstack-vm-full/12186 : SUCCESS\n- http://logs.openstack.org/26515/2/gate/gate-tempest-devstack-vm-postgres-full/11901 : SUCCESS\n- http://logs.openstack.org/26515/2/gate/gate-tempest-devstack-vm-quantum/17994 : SUCCESS\n- http://logs.openstack.org/26515/2/gate/gate-tempest-devstack-vm-quantum-full/1110 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"4f2ab00d999b775e451edb8fa741d8d9d0c31b7c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-10 14:49:35.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"fbf40fdf24dcad79de86108fcc5325774a7412f8","revisions":{"d8ef1adf01358717083d606382956edff8e93530":{"kind":"REWORK","_number":1,"created":"2013-04-09 18:35:09.000000000","uploader":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"ref":"refs/changes/15/26515/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/15/26515/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/15/26515/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/15/26515/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/15/26515/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/15/26515/1"}}},"commit":{"parents":[{"commit":"28c1dcec02841db997ddb2681be918f290c4cda8","subject":"Fix docs for admin user config in conf sample.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/28c1dcec02841db997ddb2681be918f290c4cda8"}]}],"author":{"name":"Matthew Treinish","email":"treinish@linux.vnet.ibm.com","date":"2013-04-09 15:10:58.000000000","tz":-240},"committer":{"name":"Matthew Treinish","email":"treinish@linux.vnet.ibm.com","date":"2013-04-09 18:31:56.000000000","tz":-240},"subject":"Remove smoke.py and clean up base test classes","message":"Remove smoke.py and clean up base test classes\n\nThis commit removes the smoke.py file and reworks the classes in\ntest.py to be a bit simpler. Previously, DefaultClientTest\nwas only really needed by smoke.py for the smoke tests. (the only\nplace where python clients were used) However, it remained a separate\nclass because of the status_timeout() method. This breaks out that\nmethod and removes the class so that it is more obvious that the\nsmoke tests are using tempest.manager for it\u0027s clients.\n\nChange-Id: Ib4333483c5734d14cac6376837ca770e96bd050e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/d8ef1adf01358717083d606382956edff8e93530"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/d8ef1adf01358717083d606382956edff8e93530"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"28c1dcec02841db997ddb2681be918f290c4cda8","is_merged_in_target_branch":true,"change_id":"Ie30fb4e0a6104a3b315f355566a66a366ac98383","change_number":26399,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"fbf40fdf24dcad79de86108fcc5325774a7412f8":{"kind":"REWORK","_number":2,"created":"2013-04-09 19:26:33.000000000","uploader":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"ref":"refs/changes/15/26515/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/15/26515/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/15/26515/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/15/26515/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/15/26515/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/15/26515/2"}}},"commit":{"parents":[{"commit":"28c1dcec02841db997ddb2681be918f290c4cda8","subject":"Fix docs for admin user config in conf sample.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/28c1dcec02841db997ddb2681be918f290c4cda8"}]}],"author":{"name":"Matthew Treinish","email":"treinish@linux.vnet.ibm.com","date":"2013-04-09 15:10:58.000000000","tz":-240},"committer":{"name":"Matthew Treinish","email":"treinish@linux.vnet.ibm.com","date":"2013-04-09 19:25:45.000000000","tz":-240},"subject":"Remove smoke.py and clean up base test classes","message":"Remove smoke.py and clean up base test classes\n\nThis commit removes the smoke.py file and reworks the classes in\ntest.py to be a bit simpler. Previously, DefaultClientTest\nwas only really needed by smoke.py for the smoke tests. (the only\nplace where python clients were used) However, it remained a separate\nclass because of the status_timeout() method. This breaks out that\nmethod and removes the class so that it is more obvious that the\nsmoke tests are using tempest.manager for it\u0027s clients.\n\nChange-Id: Ib4333483c5734d14cac6376837ca770e96bd050e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/fbf40fdf24dcad79de86108fcc5325774a7412f8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/fbf40fdf24dcad79de86108fcc5325774a7412f8"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"28c1dcec02841db997ddb2681be918f290c4cda8","is_merged_in_target_branch":true,"change_id":"Ie30fb4e0a6104a3b315f355566a66a366ac98383","change_number":26399,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}