)]}'
{"id":"openstack%2Fdevstack-gate~25743","triplet_id":"openstack%2Fdevstack-gate~master~Iee9530cf2a5482bf053ba72cd0448be5de0e2c1f","project":"openstack/devstack-gate","branch":"master","topic":"fix-grenade","hashtags":[],"change_id":"Iee9530cf2a5482bf053ba72cd0448be5de0e2c1f","subject":"Add option to ignore ZUUL_REF when setting up workspace.","status":"MERGED","created":"2013-03-29 18:29:38.000000000","updated":"2013-03-29 20:58:50.000000000","submitted":"2013-03-29 20:58:50.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"edb387f70e68b61aa820356bcdb1626476ce8c56","_number":25743,"virtual_id_number":25743,"owner":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"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":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":2,"date":"2013-03-29 20:58:50.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":2,"date":"2013-03-29 20:12:28.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":2,"date":"2013-03-29 20:08:18.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":1,"date":"2013-03-29 20:12:28.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"date":"2013-03-29 20:08:18.000000000","_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_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}},"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-03-29 18:54:43.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-29 20:08:18.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2013-03-29 20:58:50.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d3db1fb04a602b0284e1bcbbfd26352de24b7a8e","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-29 18:31:44.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nhard tabs, otherwise good","accounts_in_message":[],"_revision_number":1},{"id":"f292906445371ed29155623694fb579000624e3b","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2013-03-29 18:49:42.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"5fdfe9e7b3121530d21edd9de7dda7aa6ec4aba5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-29 18:54:43.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"fc9815cc9d7a4c33010a012b404a9004d8853fe5","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2013-03-29 19:24:09.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"e30cc5ecab6d35d9c975417cabd80bac7e1299da","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-29 20:04:52.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/25743/3/check/gate-tempest-devstack-vm-full/11038 : SUCCESS\n- http://logs.openstack.org/25743/3/check/gate-tempest-devstack-vm-postgres-full/10754 : SUCCESS\n- http://logs.openstack.org/25743/3/check/gate-tempest-devstack-vm-quantum/16666 : SUCCESS\n- http://logs.openstack.org/25743/3/check/gate-grenade-devstack-vm/126 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"3bc490f3cfdb901b243b1f644d96fcfb03bf54b8","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-03-29 20:08:18.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"128bfb2da604c2a639476d05156c4f8e032988d9","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2013-03-29 20:12:28.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"83e21847e9efe9cc814d5d21ca76a9ff2222087b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-29 20:12:29.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"eed0a0509a3c4fbc5cfa2e199cbc4dbb96e04570","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-29 20:58:50.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3},{"id":"721eca5fd509f33c8bdd442b2ccf844addd9419b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-29 20:58:50.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/25743/3/gate/gate-tempest-devstack-vm-full/11047 : SUCCESS\n- http://logs.openstack.org/25743/3/gate/gate-tempest-devstack-vm-postgres-full/10763 : SUCCESS\n- http://logs.openstack.org/25743/3/gate/gate-tempest-devstack-vm-quantum/16675 : SUCCESS\n","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"78f6221e2c86c601f234f1b0d68329afc00aab30","revisions":{"bbc55252150e9891a0eeca151c043ffb7b8612f9":{"kind":"REWORK","_number":1,"created":"2013-03-29 18:29:38.000000000","uploader":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"ref":"refs/changes/43/25743/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/43/25743/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/1"}}},"commit":{"parents":[{"commit":"f9dc24c696ec5d3b62e62b1de694f41bfeb6a91b","subject":"Merge \"if we are running grenade, use screen\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/f9dc24c696ec5d3b62e62b1de694f41bfeb6a91b"}]}],"author":{"name":"James E. Blair","email":"jeblair@openstack.org","date":"2013-03-29 18:26:48.000000000","tz":-420},"committer":{"name":"James E. Blair","email":"jeblair@openstack.org","date":"2013-03-29 18:26:48.000000000","tz":-420},"subject":"Add option to ignore ZUUL_REF when setting up workspace.","message":"Add option to ignore ZUUL_REF when setting up workspace.\n\nWhen setting up the old workspace for grenade tests, we would\nstill check for the ZUUL_REF and therefore might end up checking\nout a proposed change into the old repo instead of just leaving\nit at stable/foo.\n\nThis is a new parameter to setup_workspace.  Currently it\u0027s set\nto ignore zuul refs when setting up the old workspace, and use\nthem when setting up the new.  If we start doing forward grenade\ntesting (ie, a change to stable/folsom must not break upgrading\nfrom stable/folsom to master or stable/grizzly), we may need\nthe option of reversing this.  But that\u0027s a ways away.\n\nAlso, fix renaming the log files to .txt. and removing duplicates\nwhen we\u0027re an extra level deep in the log dir when running grenade.\n\nChange-Id: Iee9530cf2a5482bf053ba72cd0448be5de0e2c1f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/bbc55252150e9891a0eeca151c043ffb7b8612f9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/bbc55252150e9891a0eeca151c043ffb7b8612f9"}]},"branch":"refs/heads/master"},"7ea83939b2bdffd40de14bb33f88442c8b1e4a2e":{"kind":"REWORK","_number":2,"created":"2013-03-29 18:49:42.000000000","uploader":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"ref":"refs/changes/43/25743/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/43/25743/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/2"}}},"commit":{"parents":[{"commit":"f9dc24c696ec5d3b62e62b1de694f41bfeb6a91b","subject":"Merge \"if we are running grenade, use screen\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/f9dc24c696ec5d3b62e62b1de694f41bfeb6a91b"}]}],"author":{"name":"James E. Blair","email":"jeblair@openstack.org","date":"2013-03-29 18:26:48.000000000","tz":-420},"committer":{"name":"James E. Blair","email":"jeblair@openstack.org","date":"2013-03-29 18:49:12.000000000","tz":-420},"subject":"Add option to ignore ZUUL_REF when setting up workspace.","message":"Add option to ignore ZUUL_REF when setting up workspace.\n\nWhen setting up the old workspace for grenade tests, we would\nstill check for the ZUUL_REF and therefore might end up checking\nout a proposed change into the old repo instead of just leaving\nit at stable/foo.\n\nThis is a new parameter to setup_workspace.  Currently it\u0027s set\nto ignore zuul refs when setting up the old workspace, and use\nthem when setting up the new.  If we start doing forward grenade\ntesting (ie, a change to stable/folsom must not break upgrading\nfrom stable/folsom to master or stable/grizzly), we may need\nthe option of reversing this.  But that\u0027s a ways away.\n\nMove the grenade branch selection logic into devstack-vm-gate-wrap\nso that it is easy to update as we progress along releases.  Make\nDEVSTACK_GATE_GRENADE a binary variable like the other\nDEVSTACK_GATE variables.\n\nAlso, fix renaming the log files to .txt. and removing duplicates\nwhen we\u0027re an extra level deep in the log dir when running grenade.\n\nChange-Id: Iee9530cf2a5482bf053ba72cd0448be5de0e2c1f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/7ea83939b2bdffd40de14bb33f88442c8b1e4a2e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/7ea83939b2bdffd40de14bb33f88442c8b1e4a2e"}]},"branch":"refs/heads/master"},"78f6221e2c86c601f234f1b0d68329afc00aab30":{"kind":"REWORK","_number":3,"created":"2013-03-29 19:24:09.000000000","uploader":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"ref":"refs/changes/43/25743/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/43/25743/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/43/25743/3"}}},"commit":{"parents":[{"commit":"f9dc24c696ec5d3b62e62b1de694f41bfeb6a91b","subject":"Merge \"if we are running grenade, use screen\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/f9dc24c696ec5d3b62e62b1de694f41bfeb6a91b"}]}],"author":{"name":"James E. Blair","email":"jeblair@openstack.org","date":"2013-03-29 18:26:48.000000000","tz":-420},"committer":{"name":"James E. Blair","email":"jeblair@openstack.org","date":"2013-03-29 19:24:33.000000000","tz":-420},"subject":"Add option to ignore ZUUL_REF when setting up workspace.","message":"Add option to ignore ZUUL_REF when setting up workspace.\n\nWhen setting up the old workspace for grenade tests, we would\nstill check for the ZUUL_REF and therefore might end up checking\nout a proposed change into the old repo instead of just leaving\nit at stable/foo.\n\nThis is a new parameter to setup_workspace.  Currently it\u0027s set\nto ignore zuul refs when setting up the old workspace, and use\nthem when setting up the new.  If we start doing forward grenade\ntesting (ie, a change to stable/folsom must not break upgrading\nfrom stable/folsom to master or stable/grizzly), we may need\nthe option of reversing this.  But that\u0027s a ways away.\n\nMove the grenade branch selection logic into devstack-vm-gate-wrap\nso that it is easy to update as we progress along releases.  Make\nDEVSTACK_GATE_GRENADE a binary variable like the other\nDEVSTACK_GATE variables.\n\nAlso, fix renaming the log files to .txt. and removing duplicates\nwhen we\u0027re an extra level deep in the log dir when running grenade.\n\nChange-Id: Iee9530cf2a5482bf053ba72cd0448be5de0e2c1f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/78f6221e2c86c601f234f1b0d68329afc00aab30"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/78f6221e2c86c601f234f1b0d68329afc00aab30"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}