)]}'
{"id":"openstack%2Fdevstack-gate~50143","triplet_id":"openstack%2Fdevstack-gate~master~I21438ce6aa178c3e02ac98d98df9ed88a646d8ce","project":"openstack/devstack-gate","branch":"master","topic":"bp/fail-gate-on-log-errors","hashtags":[],"change_id":"I21438ce6aa178c3e02ac98d98df9ed88a646d8ce","subject":"Call a script to check for ERRORs in logs on tempest success","status":"MERGED","created":"2013-10-07 18:00:48.000000000","updated":"2013-10-09 01:31:56.000000000","submitted":"2013-10-09 01:31:56.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f6ce2a891a7f927bfe46ba2dba168bdfbea5bbbb","_number":50143,"virtual_id_number":50143,"owner":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":2,"date":"2013-10-09 01:31:56.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"}],"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":2,"date":"2013-10-08 16:52:11.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":2,"date":"2013-10-09 00:30:27.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":1,"date":"2013-10-08 23:53:56.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"}],"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":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"all":[{"value":0,"date":"2013-10-08 16:52:11.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":1,"date":"2013-10-09 00:30:27.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-10-08 15:28:46.000000000","_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-08 16:52:11.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"},{"updated":"2013-10-08 23:53:56.000000000","updated_by":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"reviewer":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"state":"REVIEWER"},{"updated":"2013-10-09 00:30:27.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2013-10-09 01:31:56.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"f0f3102a4e34613699cc25487c30dab09c1e3162","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-07 18:40:47.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/43/50143/1/check/check-tempest-devstack-vm-full/19c6c67 : SUCCESS in 39m 56s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/43/50143/1/check/check-tempest-devstack-vm-postgres-full/5eaa279 : SUCCESS in 37m 35s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/43/50143/1/check/check-tempest-devstack-vm-neutron/0ee3815 : SUCCESS in 25m 11s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/43/50143/1/check/gate-tempest-devstack-vm-large-ops/2c5ef40 : SUCCESS in 11m 56s\n- check-grenade-devstack-vm http://logs.openstack.org/43/50143/1/check/check-grenade-devstack-vm/038e3b3 : SUCCESS in 25m 26s\n- check-swift-devstack-vm-functional http://logs.openstack.org/43/50143/1/check/check-swift-devstack-vm-functional/458bcf3 : SUCCESS in 14m 49s\n","accounts_in_message":[],"_revision_number":1},{"id":"159a4f1b1da85663d3a02c86f456e5c65a445802","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-10-07 19:52:56.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nLooks pretty good to me, one nit.","accounts_in_message":[],"_revision_number":1},{"id":"1e1c768bfb920a271ac16774307d174287268a4f","author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"date":"2013-10-08 15:28:46.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ab54baf4c764e60cc47d1af99e85bfcd0fd49d7e","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-10-08 15:28:51.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but whitespace content has changed.","accounts_in_message":[],"_revision_number":2},{"id":"6a2228aaca47e371a552315f9e513cb77b1fdba5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-08 15:58:00.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/43/50143/2/check/check-tempest-devstack-vm-full/1d48b55 : SUCCESS in 29m 12s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/43/50143/2/check/check-tempest-devstack-vm-postgres-full/d72661c : SUCCESS in 29m 08s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/43/50143/2/check/check-tempest-devstack-vm-neutron/47ec7ce : SUCCESS in 18m 28s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/43/50143/2/check/gate-tempest-devstack-vm-large-ops/349d970 : SUCCESS in 12m 05s\n- check-grenade-devstack-vm http://logs.openstack.org/43/50143/2/check/check-grenade-devstack-vm/d48f622 : SUCCESS in 25m 12s\n- check-swift-devstack-vm-functional http://logs.openstack.org/43/50143/2/check/check-swift-devstack-vm-functional/9a8121f : SUCCESS in 14m 11s\n","accounts_in_message":[],"_revision_number":2},{"id":"459e3f1f655d6faab87cfb078b0611698b7058ea","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2013-10-08 16:52:11.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"a82849a8d1cbb592dcaa54dfbff4cf8cfc0d1e12","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2013-10-08 23:53:56.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nThanks","accounts_in_message":[],"_revision_number":2},{"id":"a9ad39ade798ff56b8a912a0203462f4e0676493","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-09 00:30:27.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"a8d5919e07e226e67a38f1af84f929b39e324621","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-10-09 00:30:27.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\nI note that check_logs.py doesn\u0027t do anything yet. Putting this in place will make it easy to slip in actual checks. However if we end up going in another direction we should remember to clean this up in d-g.","accounts_in_message":[],"_revision_number":2},{"id":"58edb8fb97d0c8c684a5fc554bca1d38ff945e77","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-09 01:31:56.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2},{"id":"819d09c11076085bede50700c61d9af2c8e3483c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-09 01:31:56.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/43/50143/2/gate/gate-tempest-devstack-vm-full/12db7f3 : SUCCESS in 32m 10s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/43/50143/2/gate/gate-tempest-devstack-vm-postgres-full/f1f58ea : SUCCESS in 33m 17s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/43/50143/2/gate/gate-tempest-devstack-vm-neutron/d5a926b : SUCCESS in 21m 33s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/43/50143/2/gate/gate-tempest-devstack-vm-large-ops/a1a296a : SUCCESS in 16m 12s\n- gate-grenade-devstack-vm http://logs.openstack.org/43/50143/2/gate/gate-grenade-devstack-vm/34d2d45 : SUCCESS in 28m 01s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/43/50143/2/gate/gate-swift-devstack-vm-functional/542a66b : SUCCESS in 15m 07s\n","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"3291c0097f204c78d94a42daf63e92e7a20aba40","revisions":{"3b42efda325a0aaa13644b07bb73f0231e4a3c42":{"kind":"REWORK","_number":1,"created":"2013-10-07 18:00:48.000000000","uploader":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"ref":"refs/changes/43/50143/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/43/50143/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/1"}}},"commit":{"parents":[{"commit":"32b2b0e7408bf947d52589e61226227c280e1e4c","subject":"Make sure tempest directories exist when modifying","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/32b2b0e7408bf947d52589e61226227c280e1e4c"}]}],"author":{"name":"David Kranz","email":"dkranz@redhat.com","date":"2013-10-07 17:56:04.000000000","tz":-240},"committer":{"name":"David Kranz","email":"dkranz@redhat.com","date":"2013-10-07 18:00:33.000000000","tz":-240},"subject":"Call a script to check for ERRORs in logs on tempest success","message":"Call a script to check for ERRORs in logs on tempest success\n\nSince this script does not exist pre-havana the call is skipped for those\ntest runs. Stress tests already do their own checks. The check_logs script\nhas a whitelist for existing errors that is maintained in tempest and\nshould be reduced as bugs causing spurious errors are fixed.\n\nPartially implements blueprint fail-gate-on-log-errors\n\nChange-Id: I21438ce6aa178c3e02ac98d98df9ed88a646d8ce\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/3b42efda325a0aaa13644b07bb73f0231e4a3c42"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/3b42efda325a0aaa13644b07bb73f0231e4a3c42"}]},"branch":"refs/heads/master"},"3291c0097f204c78d94a42daf63e92e7a20aba40":{"kind":"REWORK","_number":2,"created":"2013-10-08 15:28:46.000000000","uploader":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"ref":"refs/changes/43/50143/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/43/50143/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/43/50143/2"}}},"commit":{"parents":[{"commit":"32b2b0e7408bf947d52589e61226227c280e1e4c","subject":"Make sure tempest directories exist when modifying","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/32b2b0e7408bf947d52589e61226227c280e1e4c"}]}],"author":{"name":"David Kranz","email":"dkranz@redhat.com","date":"2013-10-07 17:56:04.000000000","tz":-240},"committer":{"name":"David Kranz","email":"dkranz@redhat.com","date":"2013-10-08 15:28:35.000000000","tz":-240},"subject":"Call a script to check for ERRORs in logs on tempest success","message":"Call a script to check for ERRORs in logs on tempest success\n\nSince this script does not exist pre-havana the call is skipped for those\ntest runs. Stress tests already do their own checks. The check_logs script\nhas a whitelist for existing errors that is maintained in tempest and\nshould be reduced as bugs causing spurious errors are fixed.\n\nPartially implements blueprint fail-gate-on-log-errors\n\nChange-Id: I21438ce6aa178c3e02ac98d98df9ed88a646d8ce\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/3291c0097f204c78d94a42daf63e92e7a20aba40"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/3291c0097f204c78d94a42daf63e92e7a20aba40"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}