)]}'
{"id":"openstack%2Fdevstack~39236","triplet_id":"openstack%2Fdevstack~master~Ibb9868ea7bf44480be76a8ea9d7d9fbc278a8ef3","project":"openstack/devstack","branch":"master","topic":"length-fix","hashtags":[],"change_id":"Ibb9868ea7bf44480be76a8ea9d7d9fbc278a8ef3","subject":"Some bash versions don\u0027t support negative lengths","status":"MERGED","created":"2013-07-30 08:46:47.000000000","updated":"2013-08-06 19:36:01.000000000","submitted":"2013-08-06 19:36:00.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"30488560a430db7a86680804f0173413fd9e4329","_number":39236,"virtual_id_number":39236,"owner":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-08-06 19:36:00.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"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-08-06 16:15:17.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-08-06 18:42:28.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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-08-06 16:15:17.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-08-06 18:42:28.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}],"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":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-08-06 16:15:17.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":"2013-08-06 18:42:28.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-08-06 19:36:00.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"50a0dd07ca8eda317d42266a14f19071e9cfc7b1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-30 09:31:17.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/36/39236/1/check/gate-tempest-devstack-vm-full/37281 : SUCCESS in 43m 10s\n- http://logs.openstack.org/36/39236/1/check/gate-tempest-devstack-vm-cells-full/3968 : FAILURE in 33m 36s (non-voting)\n- http://logs.openstack.org/36/39236/1/check/gate-tempest-devstack-vm-testr-full/2785 : FAILURE in 28m 09s (non-voting)\n- http://logs.openstack.org/36/39236/1/check/gate-tempest-devstack-vm-postgres-full/36957 : SUCCESS in 44m 23s\n- http://logs.openstack.org/36/39236/1/check/gate-tempest-devstack-vm-neutron/9372 : FAILURE in 16m 41s (non-voting)\n- http://logs.openstack.org/36/39236/1/check/gate-grenade-devstack-vm/22658 : SUCCESS in 22m 03s\n","accounts_in_message":[],"_revision_number":1},{"id":"5ce9721b219f399ca8b106179a804cc78a561111","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-30 15:52:42.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nJust for curiosity what version of bash are you dealing with here?","accounts_in_message":[],"_revision_number":1},{"id":"ca3b208ea6fac0eac6e7a13b8ac1edff1efc1c63","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-07-30 16:18:56.000000000","message":"Patch Set 1:\n\nCentOS release 6.4 (Final)\n\n[stack@rbobo devstack]$ echo ${$SHELL::-1}\n-bash: ${$SHELL::-1}: bad substitution\n\n[stack@rbobo devstack]$ $SHELL --version\n$SHELL --version\nGNU bash, version 4.1.2(1)-release (x86_64-redhat-linux-gnu)\nCopyright (C) 2009 Free Software Foundation, Inc.\nLicense GPLv3+: GNU GPL version 3 or later \u003chttp://gnu.org/licenses/gpl.html\u003e\n\nI\u0027ll upload a new commit soon.","accounts_in_message":[],"_revision_number":1},{"id":"4c653b0bc50b6385d879aa554393ae12c3468ec1","author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"date":"2013-08-06 13:42:23.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"60e22b024fe97d4fcdb86b3341ee97cbcb3d6b8e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 14:28:30.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/36/39236/2/check/gate-tempest-devstack-vm-full/426ce5c : SUCCESS in 44m 34s\n- http://logs.openstack.org/36/39236/2/check/gate-tempest-devstack-vm-cells-full/d580a7b : FAILURE in 12m 25s (non-voting)\n- http://logs.openstack.org/36/39236/2/check/gate-tempest-devstack-vm-testr-full/3a12673 : FAILURE in 30m 31s (non-voting)\n- http://logs.openstack.org/36/39236/2/check/gate-tempest-devstack-vm-postgres-full/616a024 : SUCCESS in 45m 40s\n- http://logs.openstack.org/36/39236/2/check/gate-tempest-devstack-vm-neutron/5dd6a18 : SUCCESS in 20m 53s\n- http://logs.openstack.org/36/39236/2/check/gate-grenade-devstack-vm/c33e3e4 : SUCCESS in 27m 53s\n","accounts_in_message":[],"_revision_number":2},{"id":"65d47759304115e11263d7a59d5e1d74cfea0c05","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-08-06 16:15:17.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nThanks Bob","accounts_in_message":[],"_revision_number":2},{"id":"49ddfdde51fd4a3121e11edb3e50632ac33a9c8f","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-08-06 18:42:28.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"4f822106de7efbdf590320975a2d095f4220adb5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 18:42:30.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"708ab76116984daba6428bf3a001f23f91ad0920","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 19:36:00.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/36/39236/2/gate/gate-tempest-devstack-vm-full/4516c2d : SUCCESS in 53m 26s\n- http://logs.openstack.org/36/39236/2/gate/gate-tempest-devstack-vm-postgres-full/1437f59 : SUCCESS in 43m 07s\n- http://logs.openstack.org/36/39236/2/gate/gate-tempest-devstack-vm-neutron/00bbeb9 : SUCCESS in 18m 00s\n- http://logs.openstack.org/36/39236/2/gate/gate-grenade-devstack-vm/fe01fae : SUCCESS in 25m 47s\n","accounts_in_message":[],"_revision_number":2},{"id":"deb1e78e468fa782a73f583403b55bba865dd670","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-06 19:36:01.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"46287d8d71f358b330a5b2c28dace72cbdd3492f","revisions":{"8f00bfadc71a9606a1fd6a9661e30914976c41c2":{"kind":"REWORK","_number":1,"created":"2013-07-30 08:46:47.000000000","uploader":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"ref":"refs/changes/36/39236/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/36/39236/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/39236/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/39236/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/39236/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/36/39236/1"}}},"commit":{"parents":[{"commit":"314ae87a911b457590cd268a6d50872c957de343","subject":"Merge \"xenapi: Add qemu-utils as a cinder dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/314ae87a911b457590cd268a6d50872c957de343"}]}],"author":{"name":"Bob Ball","email":"bob.ball@citrix.com","date":"2013-07-30 08:43:17.000000000","tz":60},"committer":{"name":"Bob Ball","email":"bob.ball@citrix.com","date":"2013-07-30 08:46:19.000000000","tz":60},"subject":"Some bash versions don\u0027t support negative lengths","message":"Some bash versions don\u0027t support negative lengths\n\nFix to make the length based on the length of the service string\n\nChange-Id: Ibb9868ea7bf44480be76a8ea9d7d9fbc278a8ef3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8f00bfadc71a9606a1fd6a9661e30914976c41c2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/8f00bfadc71a9606a1fd6a9661e30914976c41c2"}]},"branch":"refs/heads/master"},"46287d8d71f358b330a5b2c28dace72cbdd3492f":{"kind":"REWORK","_number":2,"created":"2013-08-06 13:42:23.000000000","uploader":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"ref":"refs/changes/36/39236/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/36/39236/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/39236/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/39236/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/36/39236/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/36/39236/2"}}},"commit":{"parents":[{"commit":"6c84463071e1ff23e20e4ef4fb863aba0732bebc","subject":"Update projects to global requirements before setup","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6c84463071e1ff23e20e4ef4fb863aba0732bebc"}]}],"author":{"name":"Bob Ball","email":"bob.ball@citrix.com","date":"2013-07-30 08:43:17.000000000","tz":60},"committer":{"name":"Bob Ball","email":"bob.ball@citrix.com","date":"2013-08-06 13:32:53.000000000","tz":60},"subject":"Some bash versions don\u0027t support negative lengths","message":"Some bash versions don\u0027t support negative lengths\n\nFix to make the service string work on these versions of bash too\n\nChange-Id: Ibb9868ea7bf44480be76a8ea9d7d9fbc278a8ef3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/46287d8d71f358b330a5b2c28dace72cbdd3492f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/46287d8d71f358b330a5b2c28dace72cbdd3492f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}