)]}'
{"id":"openstack%2Ftempest~80201","triplet_id":"openstack%2Ftempest~master~I761c82338eb513a4870b2e3cdf6ab7e98790b6f4","project":"openstack/tempest","branch":"master","topic":"bug/1291826","hashtags":[],"change_id":"I761c82338eb513a4870b2e3cdf6ab7e98790b6f4","subject":"Fix issue with pep8 gate job","status":"MERGED","created":"2014-03-13 07:47:33.000000000","updated":"2014-03-13 19:02:49.000000000","submitted":"2014-03-13 19:02:48.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"209bd05639cc826585858078ea03403554267f7d","_number":80201,"virtual_id_number":80201,"owner":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":2,"date":"2014-03-13 19:02:48.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"recommended":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"all":[{"value":0,"date":"2014-03-13 08:17:27.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2014-03-13 08:34:00.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":2,"date":"2014-03-13 10:28:37.000000000","_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":2,"date":"2014-03-13 11:09:42.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"date":"2014-03-13 08:14:05.000000000","_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2014-03-13 08:34:00.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"date":"2014-03-13 10:11:05.000000000","_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":1,"date":"2014-03-13 11:09:42.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-03-13 08:34:00.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":"2014-03-13 10:11:05.000000000","updated_by":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"reviewer":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"state":"REVIEWER"},{"updated":"2014-03-13 11:09:42.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2014-03-13 19:02:48.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"7632d7decce8a08edec4e40553857f693c421488","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-13 07:47:34.000000000","message":"Patch Set 1:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"8ef0839388dbb35630e90372210defcc9949ba11","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2014-03-13 07:56:08.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"98cc2155a703ad676b34a27b898e0aa848d02069","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2014-03-13 07:59:57.000000000","message":"Patch Set 1: No score\n\nActually just had a second thought - isn\u0027t it possible to lazily generate the schema file say in setUpClass rather than at import time? Doing this sort of stuff at import time has caused us problems in the past.","accounts_in_message":[],"_revision_number":1},{"id":"aabb0da0a2933ce032a80bce849463ab20f74800","author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"date":"2014-03-13 08:14:05.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ed07cc229c0f16f1528778ba8593840c1dec2b38","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-13 08:14:08.000000000","message":"Patch Set 2:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"48dd382b53d79772efaeb0cfdaf8f80295fe3e08","author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"date":"2014-03-13 08:17:27.000000000","message":"Patch Set 2:\n\n@Chris: AFAIK it\u0027s not since scenario lib must register it\u0027s test at import time.","accounts_in_message":[],"_revision_number":2},{"id":"44a0f489d9a8a252c47f21c2d0d4aa6b9199d252","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2014-03-13 08:34:00.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nIf it really cases valid pyflakes issue misses, it should be fixed ASAP.\n\nI think the flake8 should not try to import modules at all,\non Linux by using the imp module it is achievable, but for some strange reason it did not worked on OSX.","accounts_in_message":[],"_revision_number":2},{"id":"d4ee5b73934eba599e82cec40eeeebc99e1b0473","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2014-03-13 10:11:05.000000000","message":"Patch Set 2:\n\n@Marc - so I had a look at the code and it looks like it doesn\u0027t need to be registered at import time. The generate_scenario is a static method and whilst it generates the schema files I think that can be done at anytime before execute is called in the individual tests. Eg it could be done in setUpClass","accounts_in_message":[],"_revision_number":2},{"id":"b52798c0ae852d40c870caa8ffdeaec722c6bffe","author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"date":"2014-03-13 10:28:37.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nLGTM - @Marc - thanks for explaining the use of testscenarios to me. I agree there\u0027s no quick alternative fix here.","accounts_in_message":[],"_revision_number":2},{"id":"5431e98ae7747d613ef2a7d5358c11da0f864b40","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-13 10:50:45.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- check-grenade-dsvm http://logs.openstack.org/01/80201/2/check/check-grenade-dsvm/5b7620e : SUCCESS in 30m 59s\n- check-tempest-dsvm-full http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-full/b6241f8 : SUCCESS in 1h 14m 01s\n- check-tempest-dsvm-neutron http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-neutron/99c59f1 : SUCCESS in 37m 22s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-neutron-heat-slow/32e55fb : SUCCESS in 22m 23s\n- gate-tempest-pep8 http://logs.openstack.org/01/80201/2/check/gate-tempest-pep8/d4933b0 : SUCCESS in 3m 45s\n- gate-tempest-docs http://logs.openstack.org/01/80201/2/check/gate-tempest-docs/17ebff1 : SUCCESS in 4m 21s\n- gate-tempest-python27 http://logs.openstack.org/01/80201/2/check/gate-tempest-python27/b33fd24 : SUCCESS in 3m 16s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-postgres-full/59e5325 : SUCCESS in 59m 20s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-neutron-full/8d779b5 : SUCCESS in 1h 00m 56s (non-voting)\n- check-tempest-dsvm-cells-full http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-cells-full/52e9bf9 : SUCCESS in 57m 31s (non-voting)\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/01/80201/2/check/gate-tempest-dsvm-large-ops/59bdf91 : SUCCESS in 15m 21s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/01/80201/2/check/gate-tempest-dsvm-neutron-large-ops/df4dc23 : SUCCESS in 14m 05s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/01/80201/2/check/check-grenade-dsvm-partial-ncpu/6d44c4c : FAILURE in 26m 16s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-ironic-nv/7a73401 : SUCCESS in 29m 47s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-ironic-postgres-nv/25696eb : SUCCESS in 30m 27s (non-voting)\n- check-tempest-dsvm-sahara-full-nv http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-sahara-full-nv/db71ba7 : FAILURE in 58m 39s (non-voting)\n- check-tempest-dsvm-sahara-postgres-nv http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-sahara-postgres-nv/5ab4f06 : FAILURE in 23m 40s (non-voting)\n- check-tempest-dsvm-sahara-neutron-nv http://logs.openstack.org/01/80201/2/check/check-tempest-dsvm-sahara-neutron-nv/5594e34 : FAILURE in 33m 56s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"f1ccef10c3ef6281f7e9de4682c403ad76df266b","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-03-13 11:05:34.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nthis is a trace but not an actual blocking issue on pep8. I think the real thing is to get us to change hacking to stop doing this import.","accounts_in_message":[],"_revision_number":2},{"id":"7610dc7c1ee19e95c0a0bebf2b5afb2fea69fbed","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-03-13 11:09:42.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\napproving for now, however we really need to address this at the hacking level and stop doing module imports.","accounts_in_message":[],"_revision_number":2},{"id":"f764a491f9186dc95ac2d97da66fa5d00934fe3a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-13 11:09:44.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"61c4864b23cd541b2a20a1906be550c2888cab15","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2014-03-13 12:03:00.000000000","message":"Patch Set 2:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://70dcd673047f0ebed6a7-dda4e480e005aaa13ec303551d2d8155.r49.cf5.rackcdn.com/01/80201/2/results.html\n\nStandard recheck supported; use \"recheck xenserver\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.","accounts_in_message":[],"_revision_number":2},{"id":"010c19ccd9aa8d4f9f3419317ad17898ef001b44","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-13 19:02:48.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- gate-grenade-dsvm http://logs.openstack.org/01/80201/2/gate/gate-grenade-dsvm/c5c3f01 : SUCCESS in 40m 45s\n- gate-tempest-dsvm-full http://logs.openstack.org/01/80201/2/gate/gate-tempest-dsvm-full/6609c75 : SUCCESS in 59m 41s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/01/80201/2/gate/gate-tempest-dsvm-neutron/f9c963b : SUCCESS in 42m 14s\n- gate-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/01/80201/2/gate/gate-tempest-dsvm-neutron-heat-slow/e27235b : SUCCESS in 27m 17s\n- gate-tempest-pep8 http://logs.openstack.org/01/80201/2/gate/gate-tempest-pep8/c4a114d : SUCCESS in 3m 34s\n- gate-tempest-docs http://logs.openstack.org/01/80201/2/gate/gate-tempest-docs/c22d324 : SUCCESS in 3m 45s\n- gate-tempest-python27 http://logs.openstack.org/01/80201/2/gate/gate-tempest-python27/608a91a : SUCCESS in 3m 14s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/01/80201/2/gate/gate-tempest-dsvm-postgres-full/822ce06 : SUCCESS in 59m 52s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/01/80201/2/gate/gate-tempest-dsvm-large-ops/44245cb : SUCCESS in 16m 57s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/01/80201/2/gate/gate-tempest-dsvm-neutron-large-ops/5003b35 : SUCCESS in 16m 57s\n","accounts_in_message":[],"_revision_number":2},{"id":"9811a543f35e5f5edcf5fabf12f67f0f6dcabcbf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-13 19:02:49.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"9698a4f4bd4bd92a44a588da2e2137fad5981fa0","revisions":{"4fb243b1ef6c9d1b70cc8633b93249585974c838":{"kind":"REWORK","_number":1,"created":"2014-03-13 07:47:33.000000000","uploader":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"ref":"refs/changes/01/80201/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/01/80201/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/01/80201/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/01/80201/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/01/80201/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/01/80201/1"}}},"commit":{"parents":[{"commit":"7889ceba67758cbf1d2fa6fe734ccba4833532b5","subject":"Merge \"Add agents tests for Nova v3 API\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/7889ceba67758cbf1d2fa6fe734ccba4833532b5"}]}],"author":{"name":"Marc Koderer","email":"m.koderer@telekom.de","date":"2014-03-13 07:37:39.000000000","tz":60},"committer":{"name":"Marc Koderer","email":"m.koderer@telekom.de","date":"2014-03-13 07:44:06.000000000","tz":60},"subject":"Fix issue with pep8 gate job","message":"Fix issue with pep8 gate job\n\nSince test-scenario is working on import level the configuration is\nmay not in place. This causes an error in the pep8 job.\n\nChange-Id: I761c82338eb513a4870b2e3cdf6ab7e98790b6f4\nImplements: bug 1291826\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/4fb243b1ef6c9d1b70cc8633b93249585974c838"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/4fb243b1ef6c9d1b70cc8633b93249585974c838"}]},"branch":"refs/heads/master"},"9698a4f4bd4bd92a44a588da2e2137fad5981fa0":{"kind":"REWORK","_number":2,"created":"2014-03-13 08:14:05.000000000","uploader":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"ref":"refs/changes/01/80201/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/01/80201/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/01/80201/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/01/80201/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/01/80201/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/01/80201/2"}}},"commit":{"parents":[{"commit":"7889ceba67758cbf1d2fa6fe734ccba4833532b5","subject":"Merge \"Add agents tests for Nova v3 API\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/7889ceba67758cbf1d2fa6fe734ccba4833532b5"}]}],"author":{"name":"Marc Koderer","email":"m.koderer@telekom.de","date":"2014-03-13 07:37:39.000000000","tz":60},"committer":{"name":"Marc Koderer","email":"m.koderer@telekom.de","date":"2014-03-13 08:13:33.000000000","tz":60},"subject":"Fix issue with pep8 gate job","message":"Fix issue with pep8 gate job\n\nSince test-scenario is working on import level the configuration is\nmay not in place. This causes an error in the pep8 job.\n\nChange-Id: I761c82338eb513a4870b2e3cdf6ab7e98790b6f4\nImplements: bug 1291826\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/9698a4f4bd4bd92a44a588da2e2137fad5981fa0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/9698a4f4bd4bd92a44a588da2e2137fad5981fa0"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}