)]}'
{"id":"openstack%2Fdevstack~70867","triplet_id":"openstack%2Fdevstack~master~Id2ea2f4ca60feb9fddc7b3181063760d2044b421","project":"openstack/devstack","branch":"master","topic":"no-need-to-loop","hashtags":[],"change_id":"Id2ea2f4ca60feb9fddc7b3181063760d2044b421","subject":"No need to loop over with pkill -f","status":"MERGED","created":"2014-02-03 22:46:08.000000000","updated":"2014-02-06 17:21:05.000000000","submitted":"2014-02-06 17:21:05.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"815a705b7aa2d0396ee00757cf0ccaab2160e2bc","_number":70867,"virtual_id_number":70867,"owner":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":2,"date":"2014-02-06 17:21:05.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},{"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},"all":[{"value":0,"date":"2014-02-04 00:04:00.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2014-02-06 16:54:13.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},{"value":2,"date":"2014-02-04 05:51:56.000000000","_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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"date":"2014-02-04 00:04:00.000000000","_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},{"value":1,"date":"2014-02-06 16:08:24.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},"Review-Priority":{"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-02-06 16:08:24.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":"2014-02-06 16:54:13.000000000","updated_by":{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},"reviewer":{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},"state":"REVIEWER"},{"updated":"2014-02-06 17:21:05.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"098e7285c612af4688c1c8291b4c253659343734","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-03 23:57:34.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/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/67/70867/1/check/gate-devstack-bash8/fef8ea7 : SUCCESS in 5s\n- check-devstack-dsvm-cells http://logs.openstack.org/67/70867/1/check/check-devstack-dsvm-cells/03b276a : SUCCESS in 10m 01s\n- check-tempest-dsvm-full http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-full/3e348bb : SUCCESS in 1h 11m 15s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-postgres-full/dfc84fc : FAILURE in 1h 08m 50s\n- check-tempest-dsvm-neutron http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-neutron/643e119 : SUCCESS in 35m 39s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/67/70867/1/check/gate-tempest-dsvm-large-ops/9c99c3e : SUCCESS in 10m 56s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/67/70867/1/check/gate-tempest-dsvm-neutron-large-ops/2defe41 : SUCCESS in 12m 58s\n- check-grenade-dsvm http://logs.openstack.org/67/70867/1/check/check-grenade-dsvm/b722870 : SUCCESS in 31m 35s\n- check-swift-dsvm-functional http://logs.openstack.org/67/70867/1/check/check-swift-dsvm-functional/7c285a7 : SUCCESS in 13m 31s\n- check-devstack-dsvm-neutron http://logs.openstack.org/67/70867/1/check/check-devstack-dsvm-neutron/7004314 : FAILURE in 17m 29s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-ironic-nv/b7fab96 : FAILURE in 38m 47s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-ironic-postgres-nv/29a66ca : FAILURE in 36m 08s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"bafb12c4e606f37e51faa137d7d3f5212bca1c0b","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-02-03 23:58:15.000000000","message":"Patch Set 1:\n\nI noticed tempest failed, I think you hit bug(s):\n\n- https://bugs.launchpad.net/bugs/1253896\n- https://bugs.launchpad.net/bugs/1275267\n\nWe don\u0027t automatically recheck or reverify, so please consider\ndoing that manually if someone hasn\u0027t already. For a code review\nwhich is not yet approved, you can recheck by leaving a code\nreview comment with just the text:\n\n    recheck bug 1253896\n\nFor a code review which has been approved but failed to merge,\nyou can reverify by leaving a comment like this:\n\n    reverify bug 1253896","accounts_in_message":[],"_revision_number":1},{"id":"a3c4069150afdfab2b21854f52ea44c20a94743d","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-04 00:04:00.000000000","message":"Patch Set 1:\n\nrecheck bug 1253896","accounts_in_message":[],"_revision_number":1},{"id":"8af1c6dc9775c0821f43fe83e9f24def5b21ad50","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-04 02:22:13.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/67/70867/1/check/gate-devstack-bash8/d25f353 : SUCCESS in 6s\n- check-devstack-dsvm-cells http://logs.openstack.org/67/70867/1/check/check-devstack-dsvm-cells/0018441 : SUCCESS in 26m 01s\n- check-tempest-dsvm-full http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-full/922a459 : SUCCESS in 1h 26m 15s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-postgres-full/a8267f3 : SUCCESS in 56m 44s\n- check-tempest-dsvm-neutron http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-neutron/cafd7a9 : SUCCESS in 45m 44s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/67/70867/1/check/gate-tempest-dsvm-large-ops/598fe52 : SUCCESS in 23m 45s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/67/70867/1/check/gate-tempest-dsvm-neutron-large-ops/9d42d5b : SUCCESS in 26m 42s\n- check-grenade-dsvm http://logs.openstack.org/67/70867/1/check/check-grenade-dsvm/dcb2338 : SUCCESS in 42m 01s\n- check-swift-dsvm-functional http://logs.openstack.org/67/70867/1/check/check-swift-dsvm-functional/7c61dec : SUCCESS in 20m 07s\n- check-devstack-dsvm-neutron http://logs.openstack.org/67/70867/1/check/check-devstack-dsvm-neutron/152a7cd : FAILURE in 29m 07s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-ironic-nv/fe04c64 : FAILURE in 32m 31s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-ironic-postgres-nv/e6f59f8 : FAILURE in 38m 27s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"26baae09e02bfaaaed48484a6260a6b27c7202d5","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-04 05:51:56.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\nHow asleep was I when I didn\u0027t remove that for loop?","accounts_in_message":[],"_revision_number":1},{"id":"3d0c9e6b242eccdc042d84b0e2564042b170800a","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-04 09:14:19.000000000","message":"Patch Set 1:\n\nI think I was the reviewer who should have catch it 8-)","accounts_in_message":[],"_revision_number":1},{"id":"0106500b9abc89aa5a55134e447fc86f3860e1bc","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-06 16:08:24.000000000","message":"Patch Set 1: Approved\n\n","accounts_in_message":[],"_revision_number":1},{"id":"41a3f4cc6f233fa90899ae166bf84da882fbaa32","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-06 16:08:27.000000000","message":"Patch Set 1: No score\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"5a6ed94385c8e14528cf3da078bac2dbb862322c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-06 16:08:40.000000000","message":"Patch Set 1:\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"cf61e611e3edc250426149b12b38369d9ab3b69c","author":{"_account_id":6966,"name":"Jeffrey J. Walls","email":"jeff.walls@hpe.com","username":"ftcjeff"},"date":"2014-02-06 16:54:13.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"3de782dc99ed909cc188165089ab396bc9de852b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-06 17:03:45.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/67/70867/1/check/gate-devstack-bash8/25e09e2 : SUCCESS in 46s\n- check-devstack-dsvm-cells http://logs.openstack.org/67/70867/1/check/check-devstack-dsvm-cells/ba9f3fc : SUCCESS in 14m 38s\n- check-tempest-dsvm-full http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-full/dc1a2a6 : SUCCESS in 48m 10s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-postgres-full/b320324 : SUCCESS in 50m 50s\n- check-tempest-dsvm-neutron http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-neutron/ca6f636 : SUCCESS in 32m 35s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/67/70867/1/check/gate-tempest-dsvm-large-ops/b541112 : SUCCESS in 19m 17s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/67/70867/1/check/gate-tempest-dsvm-neutron-large-ops/614170f : SUCCESS in 18m 54s\n- check-grenade-dsvm http://logs.openstack.org/67/70867/1/check/check-grenade-dsvm/7243560 : SUCCESS in 43m 03s\n- check-swift-dsvm-functional http://logs.openstack.org/67/70867/1/check/check-swift-dsvm-functional/18fb28a : SUCCESS in 18m 03s\n- check-devstack-dsvm-neutron http://logs.openstack.org/67/70867/1/check/check-devstack-dsvm-neutron/fed0fbc : FAILURE in 19m 46s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-ironic-nv/ec867a9 : SUCCESS in 37m 16s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-ironic-postgres-nv/054c3a8 : SUCCESS in 38m 12s (non-voting)\n- check-tempest-dsvm-savanna-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-savanna-nv/b536e87 : SUCCESS in 24m 15s (non-voting)\n- check-tempest-dsvm-savanna-postgres-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-savanna-postgres-nv/51abbe3 : SUCCESS in 33m 41s (non-voting)\n- check-tempest-dsvm-savanna-neutron-nv http://logs.openstack.org/67/70867/1/check/check-tempest-dsvm-savanna-neutron-nv/0d974c2 : SUCCESS in 34m 23s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"89a6be4a03a94760f90c7ab25788ca09862aafcc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-06 17:21:05.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/67/70867/1/gate/gate-devstack-bash8/dd08fcf : SUCCESS in 45s\n- gate-devstack-dsvm-cells http://logs.openstack.org/67/70867/1/gate/gate-devstack-dsvm-cells/2e2a0db : SUCCESS in 18m 31s\n- gate-tempest-dsvm-full http://logs.openstack.org/67/70867/1/gate/gate-tempest-dsvm-full/5acb09f : SUCCESS in 1h 07m 24s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/67/70867/1/gate/gate-tempest-dsvm-postgres-full/4a407df : SUCCESS in 50m 59s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/67/70867/1/gate/gate-tempest-dsvm-neutron/f554d7c : SUCCESS in 36m 34s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/67/70867/1/gate/gate-tempest-dsvm-large-ops/a3fdb94 : SUCCESS in 20m 57s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/67/70867/1/gate/gate-tempest-dsvm-neutron-large-ops/cf13301 : SUCCESS in 16m 55s\n- gate-grenade-dsvm http://logs.openstack.org/67/70867/1/gate/gate-grenade-dsvm/9430ce6 : SUCCESS in 34m 38s\n- gate-swift-dsvm-functional http://logs.openstack.org/67/70867/1/gate/gate-swift-dsvm-functional/56e3a5c : SUCCESS in 17m 12s\n","accounts_in_message":[],"_revision_number":1},{"id":"1cc9f2f300fd2a926d807483c2bef47bf8be0706","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-06 17:21:05.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"f36a9b2136b4ba56ac2989f7829c55b4eb1c08af","revisions":{"f36a9b2136b4ba56ac2989f7829c55b4eb1c08af":{"kind":"REWORK","_number":1,"created":"2014-02-03 22:46:08.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/67/70867/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/67/70867/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/67/70867/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/67/70867/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/67/70867/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/67/70867/1"}}},"commit":{"parents":[{"commit":"daa0ce2f052eeeae2ee0b2dc1265279e4528f880","subject":"Merge \"Use service role with glance service\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/daa0ce2f052eeeae2ee0b2dc1265279e4528f880"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2014-02-03 22:44:47.000000000","tz":60},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2014-02-03 22:44:47.000000000","tz":60},"subject":"No need to loop over with pkill -f","message":"No need to loop over with pkill -f\n\nI guess four times is better than one but if we need four times to kill\nswift processes there is something pretty bad with it.\n\nChange-Id: Id2ea2f4ca60feb9fddc7b3181063760d2044b421\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f36a9b2136b4ba56ac2989f7829c55b4eb1c08af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f36a9b2136b4ba56ac2989f7829c55b4eb1c08af"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}