)]}'
{"id":"openstack%2Fdevstack~45478","triplet_id":"openstack%2Fdevstack~master~Ia2d928ade8141b59b56a2c4548d760bf6911a3e5","project":"openstack/devstack","branch":"master","topic":"reset-after-requirements-update","hashtags":[],"change_id":"Ia2d928ade8141b59b56a2c4548d760bf6911a3e5","subject":"Clean up automated changes to requirements","status":"MERGED","created":"2013-09-06 19:32:11.000000000","updated":"2013-09-25 17:12:13.000000000","submitted":"2013-09-25 17:12:13.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"df7ed1c35e2e2bcfd6024c3040b378cf4127bc3a","_number":45478,"virtual_id_number":45478,"owner":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":2,"date":"2013-09-25 17:12:13.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":2,"date":"2013-09-25 13:18:08.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-09-25 11:53:38.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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":1,"date":"2013-09-25 13:18:08.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-09-25 11:53:38.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":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"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":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-09-06 19:33:34.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2013-09-25 11:53:38.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-09-25 13:18:08.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-09-25 17:12:13.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"0f08449950f9a983b3c19287ea2959080c67c2c7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-06 19:59:08.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/78/45478/1/check/gate-tempest-devstack-vm-full/a10f0ad : SUCCESS in 25m 47s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/78/45478/1/check/gate-tempest-devstack-vm-postgres-full/e4827db : SUCCESS in 26m 53s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/78/45478/1/check/gate-tempest-devstack-vm-neutron/0a35d7d : SUCCESS in 14m 52s\n- gate-grenade-devstack-vm http://logs.openstack.org/78/45478/1/check/gate-grenade-devstack-vm/5b8a81a : SUCCESS in 25m 48s\n","accounts_in_message":[],"_revision_number":1},{"id":"e18b5f681f69c4cb549865087d633eb7d865b198","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-10 12:29:12.000000000","message":"Patch Set 1:\n\nI thought that we ended up modifying a different file during install? Did that get reverted?","accounts_in_message":[],"_revision_number":1},{"id":"29bb1122bed3c93b5a36de91a65e67dc4f46c8cb","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 15:29:45.000000000","message":"Patch Set 1:\n\nI\u0027m not sure about a different file. I\u0027m definitely seeing the requirements.txt and test-requirements.txt files with changes that don\u0027t get cleaned up, though.","accounts_in_message":[],"_revision_number":1},{"id":"a060dfe3a8c82d5846f63cb0678121f9200f9bb0","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-12 23:04:05.000000000","message":"Patch Set 1: (1 inline comment)\n\nI don\u0027t think the alternate file thing ever merged...and its dinnertime so I didn\u0027t look ;)\n\nAlternate-universe thought: \nWhat if we only revert requirements.txt and test-requirements.txt every time?  Do we still need to check for local mods?  It would make local mods to the req files harder, but that\u0027s partially the point: that is tested by modifying the requirements repo.\n\nI run with RECLONE\u003dyes most of the time so this usually doesn\u0027t bite me...","accounts_in_message":[],"_revision_number":1},{"id":"7b52c6de32e3395a234c05516fb4836c15880e60","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-23 12:48:07.000000000","message":"Patch Set 1:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":1},{"id":"23d6660f780a7d972ff1d1e105ccf64e573731a8","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-23 12:48:08.000000000","message":"Patch Set 1:\n\nrechecking because review is older than 5 days, and is at risk of merge conflict.","accounts_in_message":[],"_revision_number":1},{"id":"0d1de6f5b5c19598691594cdfd414d6939750988","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-23 12:48:21.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/GerritJenkinsGit#Test_Failures\n\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"a02efd5879339e190b90156b0d370251e9b4d43b","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-23 21:10:30.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"5068e2c4f33a1270faa0d05ee51213f49f22d8fa","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-23 21:11:12.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"a32d80a85209727b48b3f9985fdebda63eeb7f0b","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-23 21:14:58.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"e65a8ac5a2ba7e31fde7c5aad96be6349c2bbeb7","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-23 21:15:28.000000000","message":"Patch Set 2:\n\nUpdated to include setup.py as well.\n\nI\u0027m starting to think the reset command was a better way to do this, since it avoided having to list the files individually.","accounts_in_message":[],"_revision_number":2},{"id":"ee6f4ec67bc57cdd4d7fe4752cfbfaebc9021221","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-23 22:36:24.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/78/45478/3/check/gate-tempest-devstack-vm-full/fb5a2d2 : SUCCESS in 28m 43s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/78/45478/3/check/gate-tempest-devstack-vm-postgres-full/3c177a5 : SUCCESS in 31m 21s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/78/45478/3/check/gate-tempest-devstack-vm-neutron/60e0b72 : SUCCESS in 19m 09s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/78/45478/3/check/gate-tempest-devstack-vm-large-ops/46c019f : SUCCESS in 10m 55s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/78/45478/3/check/gate-grenade-devstack-vm/3fe7183 : SUCCESS in 27m 25s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/78/45478/3/check/gate-swift-devstack-vm-functional/471fd62 : SUCCESS in 14m 55s\n","accounts_in_message":[],"_revision_number":3},{"id":"6a8998eaa891313fd9bfe9382b745f3ac00a47b7","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-25 11:53:38.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"f49a9fb9d535e93ae948758cf31d651711ed2e8a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-25 13:18:08.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"ba88dae573c5bdc5d0d23773c254e7bf0fd92c03","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-25 13:18:08.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"011f07d061db227dfd7373694a1cb2df6c7ce2f7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-25 17:12:13.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3},{"id":"1cd742a17813d25029f6ccec7b3dc56dae46dbd8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-25 17:12:13.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/78/45478/3/gate/gate-tempest-devstack-vm-full/b96b89d : SUCCESS in 26m 50s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/78/45478/3/gate/gate-tempest-devstack-vm-postgres-full/ec9f203 : SUCCESS in 29m 02s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/78/45478/3/gate/gate-tempest-devstack-vm-neutron/4902511 : SUCCESS in 21m 04s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/78/45478/3/gate/gate-tempest-devstack-vm-large-ops/90545a5 : SUCCESS in 13m 01s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/78/45478/3/gate/gate-grenade-devstack-vm/f756b86 : SUCCESS in 24m 55s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/78/45478/3/gate/gate-swift-devstack-vm-functional/f480cad : SUCCESS in 11m 22s\n","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"c3431bfdd90b3d149b119038d19f6a22bc278dc0","revisions":{"19f17e41ff0b892dd9f1e56e352bbb61a1d00456":{"kind":"REWORK","_number":1,"created":"2013-09-06 19:32:11.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/78/45478/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/78/45478/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/78/45478/1"}}},"commit":{"parents":[{"commit":"b567fc1f6b412d8fca8035d1cb5fecc4f4e5f6ac","subject":"Merge \"Fix Neutron issues related to Baremetal service\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b567fc1f6b412d8fca8035d1cb5fecc4f4e5f6ac"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-06 19:30:22.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-06 19:30:22.000000000","tz":-240},"subject":"Clean up automated changes to requirements","message":"Clean up automated changes to requirements\n\nSome of us like to reuse sandboxes, and keep them up to date.  This\nis very difficult to do if devstack leaves modifications to\nrequirements.txt files after a run, since \u0027git pull\u0027 may refuse to\noverwrite those changes.\n\nThis modification has devstack reset the sandbox after installation\nwhen it made a requirements.txt update, to leave it in a clean state\nagain.\n\nChange-Id: Ia2d928ade8141b59b56a2c4548d760bf6911a3e5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/19f17e41ff0b892dd9f1e56e352bbb61a1d00456"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/19f17e41ff0b892dd9f1e56e352bbb61a1d00456"}]},"branch":"refs/heads/master"},"b948615f2ac4ed9ce3881abeab021012b6454a45":{"kind":"REWORK","_number":2,"created":"2013-09-23 21:10:30.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/78/45478/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/78/45478/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/78/45478/2"}}},"commit":{"parents":[{"commit":"e802cc451470d748c56039f392f6264b487bcbfe","subject":"Merge \"docker: ensure socat is installed\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e802cc451470d748c56039f392f6264b487bcbfe"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-06 19:30:22.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-23 21:10:00.000000000","tz":-240},"subject":"Clean up automated changes to requirements","message":"Clean up automated changes to requirements\n\nSome of us like to reuse sandboxes, and keep them up to date.  This\nis very difficult to do if devstack leaves modifications to\nrequirements.txt files after a run, since \u0027git pull\u0027 may refuse to\noverwrite those changes.\n\nThis modification has devstack undo the changes to the requirements\nfiles, to leave the sandbox in a clean state again.\n\nChange-Id: Ia2d928ade8141b59b56a2c4548d760bf6911a3e5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b948615f2ac4ed9ce3881abeab021012b6454a45"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b948615f2ac4ed9ce3881abeab021012b6454a45"}]},"branch":"refs/heads/master"},"c3431bfdd90b3d149b119038d19f6a22bc278dc0":{"kind":"REWORK","_number":3,"created":"2013-09-23 21:14:58.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/78/45478/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/78/45478/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/78/45478/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/78/45478/3"}}},"commit":{"parents":[{"commit":"e802cc451470d748c56039f392f6264b487bcbfe","subject":"Merge \"docker: ensure socat is installed\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e802cc451470d748c56039f392f6264b487bcbfe"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-06 19:30:22.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-23 21:14:49.000000000","tz":-240},"subject":"Clean up automated changes to requirements","message":"Clean up automated changes to requirements\n\nSome of us like to reuse sandboxes, and keep them up to date.  This\nis very difficult to do if devstack leaves modifications to\nrequirements.txt files after a run, since \u0027git pull\u0027 may refuse to\noverwrite those changes.\n\nThis modification has devstack undo the changes to the requirements\nfiles, to leave the sandbox in a clean state again.\n\nChange-Id: Ia2d928ade8141b59b56a2c4548d760bf6911a3e5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c3431bfdd90b3d149b119038d19f6a22bc278dc0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c3431bfdd90b3d149b119038d19f6a22bc278dc0"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}