)]}'
{"id":"openstack%2Fdevstack~196941","triplet_id":"openstack%2Fdevstack~master~Icb2465d0f834b8cb2d46dca3c7df4ae06e49d9b5","project":"openstack/devstack","branch":"master","topic":"bug/1469178","hashtags":[],"change_id":"Icb2465d0f834b8cb2d46dca3c7df4ae06e49d9b5","subject":"Move failure trap after functions it uses","status":"MERGED","created":"2015-06-30 03:50:40.000000000","updated":"2015-07-07 20:55:17.000000000","submitted":"2015-07-07 20:55:15.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"370dde6d79b0ca45897c7e4fe6dfa6b6e0d09cdd","_number":196941,"virtual_id_number":196941,"owner":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"date":"2015-06-30 04:46:37.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":2,"date":"2015-07-07 20:55:15.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2015-07-01 15:31:18.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2015-07-07 16:28:31.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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2015-07-07 16:28:31.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch 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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-06-30 04:46:37.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-07-01 15:31:18.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":"2015-07-07 16:28:31.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":"2015-07-07 20:55:15.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"db2f396224f0ba50120574c57478cc9a45b16af1","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-06-30 03:50:40.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c517f7ed74dd47e620f8d1de2fbc8b4a00db7bf3","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-06-30 04:46:37.000000000","message":"Patch Set 1:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/41/196941/1/2726/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":1},{"id":"30fa6a8ae3d2b735b0f01238b67edd487e5d715c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-06-30 05:50:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- check-tempest-dsvm-full http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-full/7929976/ : SUCCESS in 45m 37s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-postgres-full/0c97991/ : SUCCESS in 41m 59s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-neutron-full/3c36d53/ : SUCCESS in 46m 58s\n- check-grenade-dsvm http://logs.openstack.org/41/196941/1/check/check-grenade-dsvm/a3ac377/ : SUCCESS in 41m 31s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/41/196941/1/check/gate-tempest-dsvm-large-ops/db2a5c4/ : SUCCESS in 27m 39s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/41/196941/1/check/gate-tempest-dsvm-neutron-large-ops/29a67e0/ : SUCCESS in 24m 25s\n- gate-devstack-bashate http://logs.openstack.org/41/196941/1/check/gate-devstack-bashate/4275876/ : SUCCESS in 27s\n- gate-devstack-unit-tests http://logs.openstack.org/41/196941/1/check/gate-devstack-unit-tests/3dee47b/ : SUCCESS in 19s\n- check-devstack-dsvm-cells http://logs.openstack.org/41/196941/1/check/check-devstack-dsvm-cells/ae2aecc/ : SUCCESS in 23m 13s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/41/196941/1/check/check-grenade-dsvm-partial-ncpu/b86b1b9/ : SUCCESS in 40m 02s\n- check-swift-dsvm-functional http://logs.openstack.org/41/196941/1/check/check-swift-dsvm-functional/9cd04b0/ : SUCCESS in 28m 06s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-ironic-pxe_ssh/5707417/ : SUCCESS in 39m 38s\n- check-devstack-dsvm-updown http://logs.openstack.org/41/196941/1/check/check-devstack-dsvm-updown/3dfd0d7/ : SUCCESS in 20m 08s\n- check-tempest-dsvm-f21 http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-f21/11bbeda/ : FAILURE in 46m 06s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-centos7/372627e/ : UNSTABLE in 26m 29s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/41/196941/1/check/gate-devstack-publish-docs/e919165//doc/build/html/ : SUCCESS in 1m 56s\n- check-grenade-dsvm-neutron http://logs.openstack.org/41/196941/1/check/check-grenade-dsvm-neutron/3bbe8a9/ : SUCCESS in 45m 42s\n- check-tempest-dsvm-multinode-smoke http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-multinode-smoke/d6cffff/ : SUCCESS in 26m 16s (non-voting)\n- check-tempest-dsvm-neutron-multinode-smoke http://logs.openstack.org/41/196941/1/check/check-tempest-dsvm-neutron-multinode-smoke/f9b4bb7/ : SUCCESS in 26m 24s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6f578fe1f17c472d93e185754578395c87c49550","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-07-01 15:31:18.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"b4b320fa2a7994a80aad962946627a2ec11d5122","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-07-07 16:28:31.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"199ec1744b258dff52887ac4c122fc3ad93ed614","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-07 16:28:37.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"06f9abc0a315ad5bfe3ddab5aa30204239ab9160","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-07 20:55:15.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/41/196941/1/gate/gate-tempest-dsvm-full/4d3b603/ : SUCCESS in 54m 32s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/41/196941/1/gate/gate-tempest-dsvm-postgres-full/a0e4b5e/ : SUCCESS in 52m 30s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/41/196941/1/gate/gate-tempest-dsvm-neutron/5e99a7d/ : SUCCESS in 39m 28s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/41/196941/1/gate/gate-tempest-dsvm-neutron-full/484bffe/ : SUCCESS in 1h 02m 22s\n- gate-grenade-dsvm http://logs.openstack.org/41/196941/1/gate/gate-grenade-dsvm/f1f9853/ : SUCCESS in 46m 21s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/41/196941/1/gate/gate-tempest-dsvm-large-ops/3749830/ : SUCCESS in 41m 24s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/41/196941/1/gate/gate-tempest-dsvm-neutron-large-ops/7447c61/ : SUCCESS in 35m 14s\n- gate-devstack-bashate http://logs.openstack.org/41/196941/1/gate/gate-devstack-bashate/1e4f5df/ : SUCCESS in 21s\n- gate-devstack-dsvm-cells http://logs.openstack.org/41/196941/1/gate/gate-devstack-dsvm-cells/4e9388f/ : SUCCESS in 29m 07s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/41/196941/1/gate/gate-grenade-dsvm-partial-ncpu/648a4cb/ : SUCCESS in 47m 52s","accounts_in_message":[],"_revision_number":1},{"id":"0d3163306194ad26145b91643f16ba308ea321e7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-07 20:55:17.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"04c0f0be2643ef95380a7dcf4b510d86ff11cd58","revisions":{"04c0f0be2643ef95380a7dcf4b510d86ff11cd58":{"kind":"REWORK","_number":1,"created":"2015-06-30 03:50:40.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/41/196941/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/41/196941/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/196941/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/196941/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/41/196941/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/41/196941/1"}}},"commit":{"parents":[{"commit":"e4d57d6bed88ec11ca7fcc8dc286a69d284c70aa","subject":"Merge \"Fixes for Linux Bridge in the L3 agent\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e4d57d6bed88ec11ca7fcc8dc286a69d284c70aa"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2015-06-30 03:47:51.000000000","tz":600},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2015-06-30 03:48:53.000000000","tz":600},"subject":"Move failure trap after functions it uses","message":"Move failure trap after functions it uses\n\nMove the failure trap after the functions it uses, so that\n\"delete_all\" is defined when it is triggered.\n\nChange-Id: Icb2465d0f834b8cb2d46dca3c7df4ae06e49d9b5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/04c0f0be2643ef95380a7dcf4b510d86ff11cd58"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/04c0f0be2643ef95380a7dcf4b510d86ff11cd58"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}