)]}'
{"id":"openstack%2Fdevstack~48411","triplet_id":"openstack%2Fdevstack~master~Iab898f5e8a422cc0cbfe44839c938f22c7525fd8","project":"openstack/devstack","branch":"master","topic":"bug/1231334","hashtags":[],"change_id":"Iab898f5e8a422cc0cbfe44839c938f22c7525fd8","subject":"Just reset the repo instead of trying to co files","status":"MERGED","created":"2013-09-26 09:28:17.000000000","updated":"2013-10-24 16:48:55.000000000","submitted":"2013-10-24 16:48:54.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":10,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"e924fabb1d02782b65c025d795da4ea7378feadb","_number":48411,"virtual_id_number":48411,"owner":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},{"value":2,"date":"2013-10-24 16:48:54.000000000","_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"},"recommended":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":1,"date":"2013-10-17 16:12:03.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-10-17 11:43:57.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":2,"date":"2013-10-17 11:43:56.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2013-10-24 15:37:24.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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},{"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,"date":"2013-10-22 22:34:55.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-10-24 15:37:24.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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},{"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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_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"},{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-17 11:43:57.000000000","updated_by":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"reviewer":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"state":"REVIEWER"},{"updated":"2013-10-17 16:12:03.000000000","updated_by":{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},"reviewer":{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},"state":"REVIEWER"},{"updated":"2013-10-22 22:34:55.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-10-24 15:37:24.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-10-24 16:48:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"080cc6e45d8680dca2e3fb7db9bbda1d07ebb146","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-26 09:55:03.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/11/48411/1/check/gate-tempest-devstack-vm-full/820cfbb : SUCCESS in 26m 33s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/11/48411/1/check/gate-tempest-devstack-vm-postgres-full/65a64d5 : SUCCESS in 26m 41s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/11/48411/1/check/gate-tempest-devstack-vm-neutron/659a381 : SUCCESS in 20m 21s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/11/48411/1/check/gate-tempest-devstack-vm-large-ops/b009962 : SUCCESS in 12m 07s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/11/48411/1/check/gate-grenade-devstack-vm/9869209 : SUCCESS in 23m 36s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/11/48411/1/check/gate-swift-devstack-vm-functional/65f62f1 : SUCCESS in 10m 18s\n","accounts_in_message":[],"_revision_number":1},{"id":"a49833b2e1fcfe7ede6c7fd4d7bdc3c7e999f6d7","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-26 12:16:01.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"bbf0b87a01f195ac8eb08cb05fe60fad49e32034","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-26 12:32:48.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"64da16ca8d81a9c2b4f997bc84934d9281f4bb8f","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-26 12:53:55.000000000","message":"Patch Set 1: (3 inline comments)\n\nDean, this logic is going to get messy if the requirements updated changes more files. What do you think about going back to git reset?","accounts_in_message":[],"_revision_number":1},{"id":"6ac76ac58c58d925f3ff187f81c7d2cb7c5113cf","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-26 13:50:06.000000000","message":"Patch Set 1: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"9667b511d9b0a57a984f079aed86d4912c069d0a","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-26 13:58:33.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"8a695ab963799ab55cf5f798c0c2d06280cdff3b","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-26 14:31:51.000000000","message":"Patch Set 1:\n\n@Doug, I do think I prefer git --reset, then the only question is detecting when to skip the update.py.  Is swift3 repo the only outlier here?  Its the only one I see so far.","accounts_in_message":[],"_revision_number":1},{"id":"ecce96f764c7a1b838f1db1b5e22fccae08215de","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-26 15:45:21.000000000","message":"Patch Set 1:\n\n@Dean, if we go back to git reset, I think we can safely do the update for all projects, as long as there are no changes in the repository. The problem, IIUC, is that the current \"git checkout\" command fails because one or more of the files mentioned explicitly does not exist in some cases.","accounts_in_message":[],"_revision_number":1},{"id":"77707255b8e7e8ef412e19364aca2c0a7ee36254","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-27 10:09:53.000000000","message":"Patch Set 1:\n\ndoug statement is correct, this is because swift3 is the outsider here and does not have the *requirements and pbr stuff there.\n\nI am not sure I understand the going back to git reset, maybe i wasn\u0027t following some previous discussions/patch.\n\nand please please don\u0027t make the decision to remove swift3 out of this :\\\n\nlet me know if you agree dean and I\u0027ll update the patch with some of Doug\u0027s comments.\n\n(and btw: there may be some better way to detect changes using ::\n\nCHANGED\u003d$(status --porcelain) \nif [[ -n ${CHANGED} ]].... etc..\n)","accounts_in_message":[],"_revision_number":1},{"id":"8e3a4ca3a3107e76c2f7020afbc3ccdfd54bb04c","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-27 10:10:32.000000000","message":"Patch Set 1:\n\nI did mean git status --porcelain here which has the advantage to detect the new add and not just the modification of existing staged files.","accounts_in_message":[],"_revision_number":1},{"id":"3636cdd3bd6b10e9b98d1bf35d8f3d2b9a24f72a","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-27 14:11:54.000000000","message":"Patch Set 1: (2 inline comments)\n\nSorry, Chmouel, Dean and I were referring to a discussion we had on the review where I added this feature.\n\nI\u0027ve left inline comments describing what I think is the \"right\" way to fix this issue. It will work for all projects, including swift, and should continue to work even if the requirements updater script starts creating or modifying other files.\n\nDean, could you check my work just to make sure, please? I think the --hard option is needed, right?","accounts_in_message":[],"_revision_number":1},{"id":"5796b555a039b135bd25a196c858f31a1ae646a1","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-27 14:45:23.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"9d8750d56443fd44a71f7def29544f4ecd87d79f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-27 18:08:13.000000000","message":"Patch Set 2: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/11/48411/2/check/gate-tempest-devstack-vm-full/b8c1684 : SUCCESS in 38m 39s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/11/48411/2/check/gate-tempest-devstack-vm-postgres-full/66a8ca9 : FAILURE in 41m 49s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/11/48411/2/check/gate-tempest-devstack-vm-neutron/096d697 : SUCCESS in 28m 21s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/11/48411/2/check/gate-tempest-devstack-vm-large-ops/83c01a4 : SUCCESS in 23m 00s\n- gate-grenade-devstack-vm http://logs.openstack.org/11/48411/2/check/gate-grenade-devstack-vm/b5837c1 : FAILURE in 26m 51s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/11/48411/2/check/gate-swift-devstack-vm-functional/50504ba : SUCCESS in 10m 47s\n","accounts_in_message":[],"_revision_number":2},{"id":"19e69de5ceced5a388f1c0ea4c60853a8b492de8","author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"date":"2013-09-27 18:09:16.000000000","message":"Patch Set 2:\n\nI noticed tempest failed, I think you hit bug https://bugs.launchpad.net/bugs/1226337","accounts_in_message":[],"_revision_number":2},{"id":"d65b91e5e75fa743ce8dcb2374b8a9cd1032116e","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-27 19:23:39.000000000","message":"Patch Set 2:\n\nrecheck bug 1226337","accounts_in_message":[],"_revision_number":2},{"id":"c8fe4221db848e85cd2c8f80a4043a17c3481a72","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-27 20:38:32.000000000","message":"Patch Set 2:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-full/141ce93 : SUCCESS in 36m 19s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-postgres-full/327bfa2 : SUCCESS in 36m 26s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-neutron/993027f : FAILURE in 34m 52s\n- check-tempest-devstack-vm-large-ops http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-large-ops/5310050 : FAILURE in 16m 56s\n- check-grenade-devstack-vm http://logs.openstack.org/11/48411/2/check/check-grenade-devstack-vm/3b1e412 : SUCCESS in 32m 42s\n- check-swift-devstack-vm-functional http://logs.openstack.org/11/48411/2/check/check-swift-devstack-vm-functional/98b7ddd : SUCCESS in 14m 03s\n","accounts_in_message":[],"_revision_number":2},{"id":"40d21cb41dcf32a295879ccf75cb7df892d304db","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-30 15:47:23.000000000","message":"Patch Set 2:\n\nrecheck bug 1223161","accounts_in_message":[],"_revision_number":2},{"id":"723d661d397899f18faae449bd4f21fe8a14b42c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-30 16:52:13.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-full/1320a61 : SUCCESS in 37m 00s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-postgres-full/5ee393a : SUCCESS in 38m 18s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/11/48411/2/check/check-tempest-devstack-vm-neutron/5557ce8 : SUCCESS in 26m 04s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/11/48411/2/check/gate-tempest-devstack-vm-large-ops/c4c1fc5 : SUCCESS in 20m 09s\n- check-grenade-devstack-vm http://logs.openstack.org/11/48411/2/check/check-grenade-devstack-vm/c7ad38b : SUCCESS in 57m 54s\n- check-swift-devstack-vm-functional http://logs.openstack.org/11/48411/2/check/check-swift-devstack-vm-functional/842cf2a : SUCCESS in 10m 55s\n","accounts_in_message":[],"_revision_number":2},{"id":"f3c8d139259ae3807792f89b5ac377358446002c","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-30 17:06:30.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"1bb878190cba4a32b0335d8b3417b6bb2ffc1a2e","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-30 17:23:40.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"3cb5cfe57dbc4dda47972338dfb093a4c989bc4a","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-04 11:56:09.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nso many (too many) people develop directly in their devstack trees. I think this is going to break them terribly, no? Maybe I just need more coffee.","accounts_in_message":[],"_revision_number":2},{"id":"ae241090fcfe6a0606d2b6389964e772f3730611","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-10-04 12:39:10.000000000","message":"Patch Set 2:\n\nThe logic above only updates the requirements in the local devstack repo if there are not already uncommitted changes. This changeset fixes the logic for undoing those changes, but the reset is only performed if the requirements were updated in the first place.","accounts_in_message":[],"_revision_number":2},{"id":"d6263ed79e267293ebf5002d232a95a7d6c50224","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-12 06:03:09.000000000","message":"Patch Set 2: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":2},{"id":"8f8ffd9e77783883818af0921a3fee4827e3d492","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-12 06:03:09.000000000","message":"Patch Set 2:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":2},{"id":"b3e238744d51495af3703404de91d254e6bad3ea","author":{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},"date":"2013-10-17 09:11:00.000000000","message":"Patch Set 2:\n\nWhy not just check whether the requirements files exist prior to the checkout ?","accounts_in_message":[],"_revision_number":2},{"id":"63dbadd97b7545c8d9a4f418bb239f859afa55a0","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-10-17 11:43:44.000000000","message":"Patch Set 2: Restored\n\n@sean is that answering your concerns?","accounts_in_message":[],"_revision_number":2},{"id":"c60472dc2e64074abf488ba207e84e61283f5db5","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-10-17 11:43:48.000000000","message":"Patch Set 2: Rebased","accounts_in_message":[],"_revision_number":2},{"id":"73ef74478433beb2f24682ff1caa6c9b3536e81f","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-17 11:43:56.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":3},{"id":"b67a32a87dafcc7d649e7afd347572358bb929b9","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-10-17 11:43:57.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":3},{"id":"1bdf143f26268ef29b1ba5e42bf6b2511a682a01","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-17 11:43:57.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":3},{"id":"fe04f685f25918937c3c8e8364a0faa9678c8e70","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-17 12:37:37.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/11/48411/3/check/check-tempest-devstack-vm-full/44b3832 : SUCCESS in 53m 40s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/11/48411/3/check/check-tempest-devstack-vm-postgres-full/3b4a556 : SUCCESS in 46m 15s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/11/48411/3/check/check-tempest-devstack-vm-neutron/eb6f3c9 : SUCCESS in 18m 00s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/11/48411/3/check/gate-tempest-devstack-vm-large-ops/88dde43 : SUCCESS in 13m 53s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/11/48411/3/check/gate-tempest-devstack-vm-neutron-large-ops/6a1cfbc : SUCCESS in 16m 48s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/11/48411/3/check/check-grenade-devstack-vm/7736731 : SUCCESS in 27m 41s\n- check-swift-devstack-vm-functional http://logs.openstack.org/11/48411/3/check/check-swift-devstack-vm-functional/ba3b020 : SUCCESS in 14m 32s\n","accounts_in_message":[],"_revision_number":3},{"id":"0c8de28e47a1d1f98fb069aae589341697389a62","author":{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},"date":"2013-10-17 16:12:03.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"f1ebe27b77280cf8d056fd94a62a1a830b20fb1f","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-22 22:34:55.000000000","message":"Patch Set 3:\n\nI don\u0027t develop in the devstack trees so I may not be testing this completely, but once I remembered to turn off RECLONE my edited source tree was unmolested.\n\nSean, do you still have heartburn over this?","accounts_in_message":[],"_revision_number":3},{"id":"72a2473500a7110bdb351ffd1c773ed04e071af5","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-24 15:37:24.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\nok, I think it\u0027s probably safe enough given that it\u0027s hidden behind the update_requirements flag.","accounts_in_message":[],"_revision_number":3},{"id":"9b49f1f781e15eb235212735b4a34d701960e3e5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 15:40:40.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"f2636f8d6c2072b062e373c75dcc972445c76a62","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 16:48:54.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/11/48411/3/gate/gate-devstack-bash8/c2771a0 : SUCCESS in 4s\n- gate-tempest-devstack-vm-full http://logs.openstack.org/11/48411/3/gate/gate-tempest-devstack-vm-full/a34740a : SUCCESS in 29m 55s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/11/48411/3/gate/gate-tempest-devstack-vm-postgres-full/b9ed1fd : SUCCESS in 32m 21s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/11/48411/3/gate/gate-tempest-devstack-vm-neutron/9f3135f : SUCCESS in 18m 21s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/11/48411/3/gate/gate-tempest-devstack-vm-large-ops/ceef1fd : SUCCESS in 14m 37s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/11/48411/3/gate/gate-tempest-devstack-vm-neutron-large-ops/b83664d : SUCCESS in 17m 38s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/11/48411/3/gate/gate-grenade-devstack-vm/b777500 : SUCCESS in 27m 49s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/11/48411/3/gate/gate-swift-devstack-vm-functional/fd7b0be : SUCCESS in 11m 03s\n","accounts_in_message":[],"_revision_number":3},{"id":"d02aae9929ad15ca2c454666e5e5e84f918e7c3f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-24 16:48:55.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"cbce1fa418ccb271879040b117b96038fefb479f","revisions":{"159b67b7d2b3f264726218b8112ccf8cdee98b1b":{"kind":"REWORK","_number":1,"created":"2013-09-26 09:28:17.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/11/48411/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/11/48411/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/11/48411/1"}}},"commit":{"parents":[{"commit":"633501aaf65ab92d487541b69e79de4f47545b7c","subject":"Merge \"Fix typo in lib/heat\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/633501aaf65ab92d487541b69e79de4f47545b7c"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2013-09-26 09:20:04.000000000","tz":0},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2013-09-26 09:27:40.000000000","tz":0},"subject":"Don\u0027t checkout requirements when there isn\u0027t","message":"Don\u0027t checkout requirements when there isn\u0027t\n\n- Fixes checking out swift3 middleware.\n\nCloses-Bug: 1231334\nChange-Id: Iab898f5e8a422cc0cbfe44839c938f22c7525fd8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/159b67b7d2b3f264726218b8112ccf8cdee98b1b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/159b67b7d2b3f264726218b8112ccf8cdee98b1b"}]},"branch":"refs/heads/master"},"b2d60e9b140480c8af00206df18023c51a63c381":{"kind":"REWORK","_number":2,"created":"2013-09-27 14:45:23.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/11/48411/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/11/48411/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/11/48411/2"}}},"commit":{"parents":[{"commit":"633501aaf65ab92d487541b69e79de4f47545b7c","subject":"Merge \"Fix typo in lib/heat\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/633501aaf65ab92d487541b69e79de4f47545b7c"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2013-09-26 09:20:04.000000000","tz":0},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2013-09-27 14:44:32.000000000","tz":0},"subject":"Just reset the repo instead of trying to co files","message":"Just reset the repo instead of trying to co files\n\nWe can just reset the repo instead of trying to checkout specific files.\nThis fix external repos which doesn\u0027t not have requirements.txt or\ntest-requirements.txt.\n\nCloses-Bug: 1231334\nChange-Id: Iab898f5e8a422cc0cbfe44839c938f22c7525fd8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b2d60e9b140480c8af00206df18023c51a63c381"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b2d60e9b140480c8af00206df18023c51a63c381"}]},"branch":"refs/heads/master"},"cbce1fa418ccb271879040b117b96038fefb479f":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2013-09-27 14:45:23.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/11/48411/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/11/48411/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/11/48411/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/11/48411/3"}}},"commit":{"parents":[{"commit":"af0912b51257b5423fabd41fddef282da2c8386f","subject":"Merge \"Add stack phases to extras.d handling\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/af0912b51257b5423fabd41fddef282da2c8386f"}]}],"author":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2013-09-26 09:20:04.000000000","tz":0},"committer":{"name":"Gerrit Code Review","email":"review@openstack.org","date":"2013-10-17 11:43:48.000000000","tz":0},"subject":"Just reset the repo instead of trying to co files","message":"Just reset the repo instead of trying to co files\n\nWe can just reset the repo instead of trying to checkout specific files.\nThis fix external repos which doesn\u0027t not have requirements.txt or\ntest-requirements.txt.\n\nCloses-Bug: 1231334\nChange-Id: Iab898f5e8a422cc0cbfe44839c938f22c7525fd8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cbce1fa418ccb271879040b117b96038fefb479f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cbce1fa418ccb271879040b117b96038fefb479f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}