)]}'
{"id":"openstack%2Fdevstack-gate~369633","triplet_id":"openstack%2Fdevstack-gate~master~Ia0756baf0dfea790d489f7cbff68c0ec5887b3d1","project":"openstack/devstack-gate","branch":"master","topic":"fix-reproduce.sh","hashtags":[],"change_id":"Ia0756baf0dfea790d489f7cbff68c0ec5887b3d1","subject":"Handle multiline env vars in reproduce.sh","status":"MERGED","created":"2016-09-13 18:57:53.000000000","updated":"2016-09-14 00:24:59.000000000","submitted":"2016-09-14 00:24:59.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"525ac2b93f38d67e035f2cd3800ef1ae3b9a47a6","_number":369633,"virtual_id_number":369633,"owner":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":2,"date":"2016-09-14 00:24:59.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":2,"date":"2016-09-13 23:07:11.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-09-13 20:06:58.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":2,"date":"2016-09-13 23:24:03.000000000","_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2016-09-13 23:24:43.000000000","_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"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":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"all":[{"value":0,"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":1,"date":"2016-09-13 23:24:03.000000000","_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":1,"date":"2016-09-13 23:24:43.000000000","_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-09-13 20:06:58.000000000","updated_by":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"reviewer":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"state":"REVIEWER"},{"updated":"2016-09-13 23:07:11.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"},{"updated":"2016-09-13 23:24:03.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2016-09-13 23:24:43.000000000","updated_by":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"reviewer":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"state":"REVIEWER"},{"updated":"2016-09-14 00:24:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"03fcd83373f31ed0ef077ab5c1e596c83197309f","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-09-13 18:57:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"bb47f683c79706a07ce699a1af6f0f1a903ecc26","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-09-13 19:34:16.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"0c0cac2c210251fd29b12257cb519e641b860a5f","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2016-09-13 19:51:26.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a6f67c6b3ee9d3b0ceb39494309e04a2a29ad3a2","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2016-09-13 20:06:58.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"aca9a60d7abf93c39fb1bf8b82529fabf169a3bb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-13 21:50:17.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-grenade-dsvm-neutron-ubuntu-trusty http://logs.openstack.org/33/369633/2/check/gate-grenade-dsvm-neutron-ubuntu-trusty/b269899/ : SUCCESS in 56m 31s\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/606050b/ : SUCCESS in 1h 03m 11s\n- gate-tempest-dsvm-full-ubuntu-xenial http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-full-ubuntu-xenial/d5a8a63/ : SUCCESS in 1h 05m 29s\n- gate-grenade-dsvm-ubuntu-trusty http://logs.openstack.org/33/369633/2/check/gate-grenade-dsvm-ubuntu-trusty/45522db/ : SUCCESS in 37m 02s\n- gate-devstack-gate-unittests http://logs.openstack.org/33/369633/2/check/gate-devstack-gate-unittests/8e0d291/ : SUCCESS in 2m 42s\n- gate-devstack-gate-bashate http://logs.openstack.org/33/369633/2/check/gate-devstack-gate-bashate/eba52e1/ : SUCCESS in 2m 11s\n- gate-dg-hooks-dsvm http://logs.openstack.org/33/369633/2/check/gate-dg-hooks-dsvm/43fa173/ : SUCCESS in 8m 45s\n- gate-grenade-dsvm-multinode http://logs.openstack.org/33/369633/2/check/gate-grenade-dsvm-multinode/6784749/ : SUCCESS in 49m 39s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/33/369633/2/check/gate-grenade-dsvm-neutron-multinode/be1a2e7/ : SUCCESS in 55m 31s\n- gate-neutron-dsvm-functional-ubuntu-trusty-nv http://logs.openstack.org/33/369633/2/check/gate-neutron-dsvm-functional-ubuntu-trusty-nv/7423186/ : FAILURE in 30m 35s (non-voting)\n- gate-swift-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/33/369633/2/check/gate-swift-dsvm-functional-ubuntu-xenial-nv/135328d/ : SUCCESS in 31m 39s (non-voting)\n- gate-tempest-dsvm-full-ubuntu-trusty-liberty-nv http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-full-ubuntu-trusty-liberty-nv/04f2e14/ : SUCCESS in 41m 49s (non-voting)\n- gate-tempest-dsvm-neutron-full-ubuntu-trusty-liberty-nv http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-neutron-full-ubuntu-trusty-liberty-nv/6db591a/ : SUCCESS in 41m 02s (non-voting)\n- gate-tempest-dsvm-full-ubuntu-trusty-mitaka-nv http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-full-ubuntu-trusty-mitaka-nv/5f834c0/ : SUCCESS in 53m 52s (non-voting)\n- gate-tempest-dsvm-neutron-full-ubuntu-trusty-mitaka-nv http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-neutron-full-ubuntu-trusty-mitaka-nv/87a9c4d/ : SUCCESS in 56m 34s (non-voting)\n- gate-tempest-dsvm-platform-centos7-nv http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-platform-centos7-nv/ec38737/ : SUCCESS in 1h 14m 11s (non-voting)\n- gate-tempest-dsvm-multinode-full-ubuntu-xenial http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial/126197d/ : FAILURE in 1h 57m 01s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial http://logs.openstack.org/33/369633/2/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial/1340a18/ : FAILURE in 1h 14m 18s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"eec2e670cc212b5574e0e6f598a075ac14369e2a","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2016-09-13 23:07:11.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"c52bc40f6e2f082800846ad4d5b7087ef64fdf33","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2016-09-13 23:24:03.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nSo as discussed on IRC the problem was no so much multi-line env vars but vars with another \"\u003d\" in them like localrc stuff FOO\u003dBAR\u003dbaz (so the non-greedy bit) and also the potential that \"DEVSTACK\" or \"ZUUL\" appears in a env variable (nulls help there).  I manually tested both and seems ok, so LGTM","accounts_in_message":[],"_revision_number":2},{"id":"8fa6ed84d58c884cf2004ac15034b83ecd771fa2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-13 23:24:13.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"25522b5a8d156489361198e73f33f9e508aebd9b","author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"date":"2016-09-13 23:24:43.000000000","message":"Patch Set 2: Workflow+1 Code-Review+2\n\nRegex magic. You should bottle this stuff.","accounts_in_message":[],"_revision_number":2},{"id":"7ffa6b62fa4abb0b93f4aded83c7f08ba26191fe","date":"2016-09-14 00:24:59.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"f9432def1305b4934ecb4daf754b76a433435f87","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-14 00:24:59.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-grenade-dsvm-neutron-ubuntu-trusty http://logs.openstack.org/33/369633/2/gate/gate-grenade-dsvm-neutron-ubuntu-trusty/1405378/ : SUCCESS in 32m 23s\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/33/369633/2/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/461f344/ : SUCCESS in 1h 00m 14s\n- gate-tempest-dsvm-full-ubuntu-xenial http://logs.openstack.org/33/369633/2/gate/gate-tempest-dsvm-full-ubuntu-xenial/a8b0251/ : SUCCESS in 46m 28s\n- gate-grenade-dsvm-ubuntu-trusty http://logs.openstack.org/33/369633/2/gate/gate-grenade-dsvm-ubuntu-trusty/338ecdf/ : SUCCESS in 26m 16s\n- gate-devstack-gate-unittests http://logs.openstack.org/33/369633/2/gate/gate-devstack-gate-unittests/4307157/ : SUCCESS in 1m 36s\n- gate-grenade-dsvm-multinode http://logs.openstack.org/33/369633/2/gate/gate-grenade-dsvm-multinode/1a2063e/ : SUCCESS in 27m 47s\n- gate-grenade-dsvm-neutron-multinode http://logs.openstack.org/33/369633/2/gate/gate-grenade-dsvm-neutron-multinode/e5b4588/ : SUCCESS in 34m 40s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"99c191fb892b86c16f61f2a359c77ba6afeaaa35","revisions":{"69bae7dfb6f01879f8c7e5ea3d337291fdd09f5c":{"kind":"REWORK","_number":1,"created":"2016-09-13 18:57:53.000000000","uploader":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"ref":"refs/changes/33/369633/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/33/369633/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/1"}}},"commit":{"parents":[{"commit":"51dad3c7c093d8fae3fbe98db28563744c8e86e4","subject":"Merge \"Remove \u0027partial\u0027 support for Grenade jobs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/51dad3c7c093d8fae3fbe98db28563744c8e86e4"}]}],"author":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-09-13 18:56:48.000000000","tz":-420},"committer":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-09-13 18:56:48.000000000","tz":-420},"subject":"Handle multiline env vars in reproduce.sh","message":"Handle multiline env vars in reproduce.sh\n\nOur old reproduce.sh env var processing was line based which meant we\nmangled multiline env var values. Switch to using null delimiters when\nprocessing the env vars so that we can preserve any multiline env vars.\n\nChange-Id: Ia0756baf0dfea790d489f7cbff68c0ec5887b3d1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/69bae7dfb6f01879f8c7e5ea3d337291fdd09f5c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/69bae7dfb6f01879f8c7e5ea3d337291fdd09f5c"}]},"branch":"refs/heads/master"},"99c191fb892b86c16f61f2a359c77ba6afeaaa35":{"kind":"REWORK","_number":2,"created":"2016-09-13 19:51:26.000000000","uploader":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"ref":"refs/changes/33/369633/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/33/369633/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/33/369633/2"}}},"commit":{"parents":[{"commit":"51dad3c7c093d8fae3fbe98db28563744c8e86e4","subject":"Merge \"Remove \u0027partial\u0027 support for Grenade jobs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/51dad3c7c093d8fae3fbe98db28563744c8e86e4"}]}],"author":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-09-13 18:56:48.000000000","tz":-420},"committer":{"name":"Clark Boylan","email":"clark.boylan@gmail.com","date":"2016-09-13 19:50:50.000000000","tz":-420},"subject":"Handle multiline env vars in reproduce.sh","message":"Handle multiline env vars in reproduce.sh\n\nOur old reproduce.sh env var processing was line based which meant we\nmangled multiline env var values. Switch to using null delimiters when\nprocessing the env vars so that we can preserve any multiline env vars.\n\nChange-Id: Ia0756baf0dfea790d489f7cbff68c0ec5887b3d1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/99c191fb892b86c16f61f2a359c77ba6afeaaa35"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/99c191fb892b86c16f61f2a359c77ba6afeaaa35"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}