)]}'
{"id":"openstack%2Fdevstack~3962","triplet_id":"openstack%2Fdevstack~master~I62b4c7182682deb4ef16ed0cf1a3b5a130c1c4ee","project":"openstack/devstack","branch":"master","topic":"bug/928883","hashtags":[],"change_id":"I62b4c7182682deb4ef16ed0cf1a3b5a130c1c4ee","subject":"Launch screen with bash as shell","status":"MERGED","created":"2012-02-09 15:39:19.000000000","updated":"2012-02-10 17:59:21.000000000","submitted":"2012-02-10 17:59:21.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"93c9ab5046852d279c15b23acc028ac8f2daf137","_number":3962,"virtual_id_number":3962,"owner":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":1,"date":"2012-02-10 17:59:21.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"all":[{"value":2,"date":"2012-02-10 17:29:38.000000000","_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"value":0,"date":"2012-02-09 16:33:29.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"date":"2012-02-10 17:36:06.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2012-02-09 17:21:00.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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"all":[{"value":1,"date":"2012-02-10 17:29:38.000000000","_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"value":0,"date":"2012-02-09 16:32:07.000000000","_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-02-09 17:21:00.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":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"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":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":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-02-09 17:21:00.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":"2012-02-10 17:29:38.000000000","updated_by":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"reviewer":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"state":"REVIEWER"},{"updated":"2012-02-10 17:59:21.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"b841f95fb77a47e2ab35dc3e5c70bf5b215ca707","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2012-02-09 16:14:59.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t submit this\n\nThis only looks at the user\u0027s default shell, not the one currently running.  screen uses $SHELL to decide what shell to use in a new window so it seems to be that is what is important here.","accounts_in_message":[],"_revision_number":1},{"id":"a51daac5621c609f4be007aaab4f2ed017a6f103","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2012-02-09 16:32:07.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3b6963b94d15d954e35600146c38fe5c454ae536","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2012-02-09 16:33:29.000000000","message":"Patch Set 2:\n\nRedoing it in an another way and launch screen with bash as login shell.","accounts_in_message":[],"_revision_number":2},{"id":"a19c11d4fd5ca6488286df57ec2232aeceacaa7e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2012-02-09 17:21:00.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nI like that much better, thanks","accounts_in_message":[],"_revision_number":2},{"id":"46c181acfec23c111d6d5981a21325a445590249","author":{"_account_id":112,"name":"Jesse Andrews","email":"anotherjesse@gmail.com","username":"anotherjesse"},"date":"2012-02-10 17:29:38.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"a0c708c7f41188672d3d6920e5e0860f6c26da7c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-02-10 17:36:06.000000000","message":"Patch Set 2: Fails\n\nBuild Failed \n \nhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/1312/ : FAILURE \nhttps://jenkins.openstack.org/job/gate-devstack-merge/114/ : SUCCESS","accounts_in_message":[],"_revision_number":2},{"id":"54be0fe3b72cd6b9f03ebabffb38e54f51904a5b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-02-10 17:59:21.000000000","message":"Patch Set 2: Verified\n\nBuild Successful \n \nhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/1319/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-devstack-merge/115/ : SUCCESS","accounts_in_message":[],"_revision_number":2},{"id":"a87b3101a8c029a46dec4ee3eb7f59e26ad7fd4e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-02-10 17:59:21.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"0cbe851b294a114e7ffd7c669f88cad41d019292","revisions":{"f39e0ca7af61412b78d60e0e20ad17fbc6db29f2":{"kind":"REWORK","_number":1,"created":"2012-02-09 15:39:19.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/62/3962/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/62/3962/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/62/3962/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/62/3962/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/62/3962/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/62/3962/1"}}},"commit":{"parents":[{"commit":"fcd135252b29d5bdd7349395734accafc85aed4f","subject":"Merge \"Separate out build_xva process so that it can be run on a machine besides dom0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fcd135252b29d5bdd7349395734accafc85aed4f"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2012-02-09 15:36:15.000000000","tz":60},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2012-02-09 15:36:15.000000000","tz":60},"subject":"Warn if we don\u0027t use bash as the user shell.","message":"Warn if we don\u0027t use bash as the user shell.\n\nSince screen commands are bash specifics make sure the user has bash as\nuser shell.\n\nFixes bug928883.\n\nChange-Id: I62b4c7182682deb4ef16ed0cf1a3b5a130c1c4ee\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f39e0ca7af61412b78d60e0e20ad17fbc6db29f2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f39e0ca7af61412b78d60e0e20ad17fbc6db29f2"}]},"branch":"refs/heads/master"},"0cbe851b294a114e7ffd7c669f88cad41d019292":{"kind":"REWORK","_number":2,"created":"2012-02-09 16:32:07.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/62/3962/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/62/3962/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/62/3962/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/62/3962/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/62/3962/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/62/3962/2"}}},"commit":{"parents":[{"commit":"fcd135252b29d5bdd7349395734accafc85aed4f","subject":"Merge \"Separate out build_xva process so that it can be run on a machine besides dom0\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fcd135252b29d5bdd7349395734accafc85aed4f"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2012-02-09 15:36:15.000000000","tz":60},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","date":"2012-02-09 16:31:07.000000000","tz":60},"subject":"Launch screen with bash as shell","message":"Launch screen with bash as shell\n\nSince screen commands are bash specifics make sure we launch screen with\nbash login shell instead of other shells.\n\nFixes bug928883.\n\nChange-Id: I62b4c7182682deb4ef16ed0cf1a3b5a130c1c4ee\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0cbe851b294a114e7ffd7c669f88cad41d019292"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0cbe851b294a114e7ffd7c669f88cad41d019292"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}