)]}'
{"id":"openstack%2Fdevstack~24831","triplet_id":"openstack%2Fdevstack~master~I97d562adda51d39a5e7fa1dc4d945d4a396201cd","project":"openstack/devstack","branch":"master","topic":"quantum-process","hashtags":[],"change_id":"I97d562adda51d39a5e7fa1dc4d945d4a396201cd","subject":"Force Quantum processes to run in a subshell","status":"MERGED","created":"2013-03-19 20:51:43.000000000","updated":"2013-03-20 15:24:05.000000000","submitted":"2013-03-20 15:24:04.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"aec9474f7a1bad2b56d563ce7d19418e5036c75f","_number":24831,"virtual_id_number":24831,"owner":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-03-20 15:24:04.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-03-20 11:34:12.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-03-20 14:47:59.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":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-03-20 11:34:12.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"date":"2013-03-19 21:46:52.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-03-20 14:47:59.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":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-03-20 11:34:12.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2013-03-20 14:47:59.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":"2013-03-20 15:24:04.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"ba093f84628272ffe76e8680b867604e530498cf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-19 21:27:48.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/24831/1/check/gate-tempest-devstack-vm-full/9580 : SUCCESS\n- http://logs.openstack.org/24831/1/check/gate-tempest-devstack-vm-postgres-full/9295 : SUCCESS\n- http://logs.openstack.org/24831/1/check/gate-tempest-devstack-vm-quantum/14977 : FAILURE\n- http://logs.openstack.org/24831/1/check/gate-grenade-devstack-vm/24 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"fb0e1aa9f9f02ae0ee5aee515ce357352c99151a","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-03-19 21:46:52.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"1688ed6d32b8da2719473a25cd6dea097c15906a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-19 22:22:47.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/24831/2/check/gate-tempest-devstack-vm-full/9599 : SUCCESS\n- http://logs.openstack.org/24831/2/check/gate-tempest-devstack-vm-postgres-full/9314 : SUCCESS\n- http://logs.openstack.org/24831/2/check/gate-tempest-devstack-vm-quantum/14996 : SUCCESS\n- http://logs.openstack.org/24831/2/check/gate-grenade-devstack-vm/26 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"316c127c2565858f5e691b068503baf7d7f19fd3","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-03-20 11:34:12.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"be24eb6dab667aa4ebaaa7c27413853f86c0b1dd","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-20 14:47:59.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"23eccafc811628c67cdab855e3316b773846f2a0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-20 14:48:01.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"c188d7370ea784e4fdaabc51f33bfc6eaf3de3cc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-20 15:24:04.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/24831/2/gate/gate-tempest-devstack-vm-full/9701 : SUCCESS\n- http://logs.openstack.org/24831/2/gate/gate-tempest-devstack-vm-postgres-full/9416 : SUCCESS\n- http://logs.openstack.org/24831/2/gate/gate-tempest-devstack-vm-quantum/15112 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"b2f0a89c58b71c6064af4c7fa14ba19006f7f9c6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-20 15:24:05.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"58ab929de63a3a3d7a615edff6c04044071cee5d","revisions":{"7e1198b8743f1d7a66aebbca23732ee4d600adbd":{"kind":"REWORK","_number":1,"created":"2013-03-19 20:51:43.000000000","uploader":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"ref":"refs/changes/31/24831/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/24831/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/24831/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/24831/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/24831/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/24831/1"}}},"commit":{"parents":[{"commit":"b64c5d82c948ea223c5b89409d7d2efba8ade725","subject":"Merge \"turn off swift by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b64c5d82c948ea223c5b89409d7d2efba8ade725"}]}],"author":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2013-03-19 20:39:47.000000000","tz":-300},"committer":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2013-03-19 20:44:26.000000000","tz":-300},"subject":"Force Quantum processes to run under bash","message":"Force Quantum processes to run under bash\n\nForce bash to evaluate the command line passed to screen_it so the\ncommands are executed in a subshell.  This seems to make a difference\nwhen using run_service() when the server process is owned by init.\n\nThese Quantum services were the only ones in all of DevStack being\nspawned in this manner.  This will allow the run_service() patch\nhttps://review.openstack.org/#/c/23148/ to succesfully be merged\nand the number of test failures due to screen should be reduced to 0.\n\nChange-Id: I97d562adda51d39a5e7fa1dc4d945d4a396201cd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7e1198b8743f1d7a66aebbca23732ee4d600adbd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7e1198b8743f1d7a66aebbca23732ee4d600adbd"}]},"branch":"refs/heads/master"},"58ab929de63a3a3d7a615edff6c04044071cee5d":{"kind":"REWORK","_number":2,"created":"2013-03-19 21:46:52.000000000","uploader":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"ref":"refs/changes/31/24831/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/24831/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/24831/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/24831/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/24831/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/24831/2"}}},"commit":{"parents":[{"commit":"b64c5d82c948ea223c5b89409d7d2efba8ade725","subject":"Merge \"turn off swift by default\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b64c5d82c948ea223c5b89409d7d2efba8ade725"}]}],"author":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2013-03-19 20:39:47.000000000","tz":-300},"committer":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2013-03-19 21:46:33.000000000","tz":-300},"subject":"Force Quantum processes to run in a subshell","message":"Force Quantum processes to run in a subshell\n\nForce bash to evaluate the command line passed to screen_it so the\ncommands are executed in a subshell.  This seems to make a difference\nwhen using run_service() when the server process is owned by init.\n\nThese Quantum services were the only ones in all of DevStack being\nspawned in this manner.  This will allow the run_service() patch\nhttps://review.openstack.org/#/c/23148/ to succesfully be merged\nand the number of test failures due to screen should be reduced to 0.\n\nChange-Id: I97d562adda51d39a5e7fa1dc4d945d4a396201cd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/58ab929de63a3a3d7a615edff6c04044071cee5d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/58ab929de63a3a3d7a615edff6c04044071cee5d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}