)]}'
{"id":"openstack%2Fdevstack~45248","triplet_id":"openstack%2Fdevstack~master~Iaa68879e867a4426b1990d4d46164769177dc7cc","project":"openstack/devstack","branch":"master","topic":"permission-fix","hashtags":[],"change_id":"Iaa68879e867a4426b1990d4d46164769177dc7cc","subject":"Only run chmod/chown for local files","status":"MERGED","created":"2013-09-05 15:06:20.000000000","updated":"2013-09-15 14:05:26.000000000","submitted":"2013-09-15 14:05:25.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"8fb130146b7b8b2d6e3873eb81520a0a262badee","_number":45248,"virtual_id_number":45248,"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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":2,"date":"2013-09-15 14:05:25.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"recommended":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"all":[{"value":1,"date":"2013-09-11 00:51:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-09-15 13:37:09.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"date":"2013-09-10 19:17:56.000000000","permitted_voting_range":{"min":0,"max":1},"_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":2,"date":"2013-09-12 10:37:57.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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-09-15 13:37:09.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"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,"date":"2013-09-12 10:37:57.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":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_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-10 18:53:53.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-11 00:51:06.000000000","updated_by":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"reviewer":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"state":"REVIEWER"},{"updated":"2013-09-12 10:37:57.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-15 13:37:09.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2013-09-15 14:05:25.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"40890a610b43e9d85f5b20e6a787ebf14fe7eae3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-05 15:22:39.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\n- gate-tempest-devstack-vm-full http://logs.openstack.org/48/45248/1/check/gate-tempest-devstack-vm-full/a0a4cf3 : FAILURE in 6m 45s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/48/45248/1/check/gate-tempest-devstack-vm-postgres-full/72346b4 : FAILURE in 7m 02s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/48/45248/1/check/gate-tempest-devstack-vm-neutron/8af1b91 : FAILURE in 9m 19s\n- gate-grenade-devstack-vm http://logs.openstack.org/48/45248/1/check/gate-grenade-devstack-vm/ab591d1 : FAILURE in 16m 09s\n","accounts_in_message":[],"_revision_number":1},{"id":"fce68edc1a2bec73eab94be43c50d5200ea324ec","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-06 18:58:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"28e7516ae334f1d5d7d82b261ac88092ac01fb6b","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-09-06 18:58:55.000000000","message":"Patch Set 2:\n\nNew patchset patch-id matches previous patchset, but whitespace content has changed.","accounts_in_message":[],"_revision_number":2},{"id":"1a1b45f76e15020264f072147d7497a837ae6fe8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-06 19:28:31.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/48/45248/2/check/gate-tempest-devstack-vm-full/66d555c : FAILURE in 26m 43s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/48/45248/2/check/gate-tempest-devstack-vm-postgres-full/c790100 : SUCCESS in 29m 39s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/48/45248/2/check/gate-tempest-devstack-vm-neutron/7dd6910 : SUCCESS in 16m 37s\n- gate-grenade-devstack-vm http://logs.openstack.org/48/45248/2/check/gate-grenade-devstack-vm/17d104c : SUCCESS in 20m 41s\n","accounts_in_message":[],"_revision_number":2},{"id":"6c7f3a2324c433d7d32d5314d95c22f272eb6de2","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-06 19:37:49.000000000","message":"Patch Set 2:\n\nrecheck bug 1221899","accounts_in_message":[],"_revision_number":2},{"id":"379359e73452f2d8c5f004488c43643e2e9b8037","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-06 20:06:07.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/48/45248/2/check/gate-tempest-devstack-vm-full/049154d : SUCCESS in 28m 15s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/48/45248/2/check/gate-tempest-devstack-vm-postgres-full/5ac23ec : SUCCESS in 26m 28s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/48/45248/2/check/gate-tempest-devstack-vm-neutron/0988410 : SUCCESS in 15m 26s\n- gate-grenade-devstack-vm http://logs.openstack.org/48/45248/2/check/gate-grenade-devstack-vm/e8ea5a8 : SUCCESS in 21m 17s\n","accounts_in_message":[],"_revision_number":2},{"id":"aa37714a257b76b26ee0e61d17041c753b6ef0c4","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-10 12:27:16.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nI think a flag is the wrong option here, if there is a way to auto detect the specific kind of failure (or that the filesystem is on NFS) that\u0027s fine, but I think option proliferation is the wrong approach.","accounts_in_message":[],"_revision_number":2},{"id":"eca6badbd5763a4655c471766352353db786e47c","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-10 12:48:19.000000000","message":"Patch Set 2: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"73e0f0d2ad17f87b47f8a4bc38d5aee71d29ae8f","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 14:28:43.000000000","message":"Patch Set 2: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"13f68721bcc8434b3633479d50b211b7a8ad227e","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 15:27:28.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"07b80b1a3993a4afd9bbce38fa4539011dafc14d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-10 15:56:49.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/48/45248/3/check/gate-tempest-devstack-vm-full/17246f6 : SUCCESS in 29m 17s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/48/45248/3/check/gate-tempest-devstack-vm-postgres-full/a874795 : SUCCESS in 27m 02s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/48/45248/3/check/gate-tempest-devstack-vm-neutron/27d521b : SUCCESS in 16m 20s\n- gate-grenade-devstack-vm http://logs.openstack.org/48/45248/3/check/gate-grenade-devstack-vm/c208b98 : SUCCESS in 21m 23s\n","accounts_in_message":[],"_revision_number":3},{"id":"f30da9b66c4c742097e17e8fa27add6501ce437d","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-10 17:06:20.000000000","message":"Patch Set 3:\n\nok... so you are going to hate me for another comment here, but it seems like what we should actually do it take the logic you made and create:\n\nfunction safe_chown()....\n\nWhich encapsulates the nfs checking, the sudo escalation, etc. Then the main code could\n\nsafe_chown -R foo bar","accounts_in_message":[],"_revision_number":3},{"id":"58edc0fab0ff4bf94d76cb91951e1f571265477c","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-10 17:06:24.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n","accounts_in_message":[],"_revision_number":3},{"id":"293a9597ad684955389f0aa37a0d7e63a0fafc24","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 18:49:03.000000000","message":"Patch Set 3:\n\nYou know, I thought about that, but only after I\u0027d pasted it in several places, and then I got lazy. I can\u0027t slip anything past you, though. ;-)","accounts_in_message":[],"_revision_number":3},{"id":"b4ff7fa4e64d5ef5ba3ec7ef4fbcc6682f8046b7","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 18:50:55.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"ef42eb5835dd84a2a1c7553cfab2f35dbba48573","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-10 18:53:53.000000000","message":"Patch Set 4:\n\nI\u0027ve been pondering pushing the use of install(1) rather than sets of mkdir/chmod/chown commands.  If we build a function for this I\u0027d like for it to encapsulate the kinds of things install does.\n\nAlso, I\u0027m assuming the original issue is due to root_squash on the nfs mount.  I\u0027m surprised that having files/directories owned by nobody isn\u0027t causing other problems.","accounts_in_message":[],"_revision_number":4},{"id":"8a606623217f1c9e253f405a6756a9a6f18068b0","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 19:15:31.000000000","message":"Patch Set 4:\n\nI\u0027m working in a virtualbox managed by vagrant. The file ownership is mapped to my local user outside of the VM, and the chown/chmod operations to change stuff to be owned by the vagrant user is failing.\n\nI\u0027m not sure about all of the things install does, but I don\u0027t think that\u0027s going to fit with how chown/chmod are being used here. It\u0027s for things like changing the ownership of egg-info directories after running \"sudo python setup.py develop\", since on a local filesystem that leaves the egg-info directory and its contents owned by root, which may mean an sdist command fails later and breaks tox.","accounts_in_message":[],"_revision_number":4},{"id":"033956086b064de44330b97bce0ce9672bead2e0","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 19:16:38.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"d63ae715764564ffb0a973c4d12cdb8733c01f18","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2013-09-10 19:17:56.000000000","message":"Patch Set 5:\n\nTo be clear, chown/chmod *are* used for directories created with mkdir, it\u0027s just that those operations don\u0027t cause devstack.sh to exit with an error if they fail (the errors are ignored, and I get lots of annoying messages, but the script keeps running).","accounts_in_message":[],"_revision_number":5},{"id":"811b154cc4df56b98871ace107411fc367317730","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-10 19:45:37.000000000","message":"Patch Set 5: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/48/45248/5/check/gate-tempest-devstack-vm-full/010f43e : SUCCESS in 28m 56s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/48/45248/5/check/gate-tempest-devstack-vm-postgres-full/8c10dbf : SUCCESS in 28m 24s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/48/45248/5/check/gate-tempest-devstack-vm-neutron/8a9fe5e : SUCCESS in 18m 24s\n- gate-grenade-devstack-vm http://logs.openstack.org/48/45248/5/check/gate-grenade-devstack-vm/a03c5c4 : SUCCESS in 23m 32s\n","accounts_in_message":[],"_revision_number":5},{"id":"d3e4acd7b7af6e07d7ed58ec97024ff8e50ae9c5","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2013-09-11 00:51:06.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\nFWIW I have the same setup and issue with the chmod as Doug.","accounts_in_message":[],"_revision_number":5},{"id":"8fab38e511f06021f823556cb6860b6ce040a052","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-12 10:37:57.000000000","message":"Patch Set 5: Looks good to me (core reviewer)\n\nthanks for the changes Doug","accounts_in_message":[],"_revision_number":5},{"id":"3d5e2cea06c906a95e891b261cff1956f75621d4","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-09-15 13:37:09.000000000","message":"Patch Set 5: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":5},{"id":"99e91f8818d9a4646f6c4daa1aac992c228b694a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-15 13:37:10.000000000","message":"Patch Set 5: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":5},{"id":"a47b21292bc54dce22252358d37a7b8614857c1f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-15 14:05:25.000000000","message":"Patch Set 5: Verified\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/48/45248/5/gate/gate-tempest-devstack-vm-full/a536be7 : SUCCESS in 25m 52s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/48/45248/5/gate/gate-tempest-devstack-vm-postgres-full/c33a5e9 : SUCCESS in 28m 11s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/48/45248/5/gate/gate-tempest-devstack-vm-neutron/58e89c4 : SUCCESS in 18m 12s\n- gate-grenade-devstack-vm http://logs.openstack.org/48/45248/5/gate/gate-grenade-devstack-vm/42480de : SUCCESS in 22m 21s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/48/45248/5/gate/gate-swift-devstack-vm-functional/7e24196 : SUCCESS in 9m 01s\n","accounts_in_message":[],"_revision_number":5},{"id":"e2940fb518844ffdbdbb81bc92919a3b64cd00ee","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-15 14:05:26.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"e700267e33d748fe36c621cf16497597fcbe6aac","revisions":{"6c1f491cb44d289f3b1403fb3d70b98bd09dab01":{"kind":"REWORK","_number":1,"created":"2013-09-05 15:06:20.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/48/45248/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/45248/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/45248/1"}}},"commit":{"parents":[{"commit":"72854cc567d1696a3bf4358680f585097c5dfcb6","subject":"Merge \"xenapi: enable block device access for stack user\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/72854cc567d1696a3bf4358680f585097c5dfcb6"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-05 12:10:07.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-05 12:10:07.000000000","tz":-240},"subject":"Add flag to turn of chown of egg-info dirs","message":"Add flag to turn of chown of egg-info dirs\n\nWhen the /opt/stack directory is NFS mounted chown to\nthe local user might not work. In those cases, provide\na flag to turn off the step so the installation can\ncontinue anyway.\n\nChange-Id: Iaa68879e867a4426b1990d4d46164769177dc7cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6c1f491cb44d289f3b1403fb3d70b98bd09dab01"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6c1f491cb44d289f3b1403fb3d70b98bd09dab01"}]},"branch":"refs/heads/master"},"a3e8d87b7f8fbc31124f825921415a92ff5f06e7":{"kind":"REWORK","_number":2,"created":"2013-09-06 18:58:49.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/48/45248/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/45248/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/45248/2"}}},"commit":{"parents":[{"commit":"72854cc567d1696a3bf4358680f585097c5dfcb6","subject":"Merge \"xenapi: enable block device access for stack user\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/72854cc567d1696a3bf4358680f585097c5dfcb6"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-05 12:10:07.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-06 18:58:39.000000000","tz":-240},"subject":"Add flag to turn off chown of egg-info dirs","message":"Add flag to turn off chown of egg-info dirs\n\nWhen the /opt/stack directory is NFS mounted chown to\nthe local user might not work. In those cases, provide\na flag to turn off the step so the installation can\ncontinue anyway.\n\nChange-Id: Iaa68879e867a4426b1990d4d46164769177dc7cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3e8d87b7f8fbc31124f825921415a92ff5f06e7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3e8d87b7f8fbc31124f825921415a92ff5f06e7"}]},"branch":"refs/heads/master"},"12bc5f546ed00aff0a457fe926ff650e09d2198a":{"kind":"REWORK","_number":3,"created":"2013-09-10 15:27:28.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/48/45248/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/45248/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/45248/3"}}},"commit":{"parents":[{"commit":"72854cc567d1696a3bf4358680f585097c5dfcb6","subject":"Merge \"xenapi: enable block device access for stack user\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/72854cc567d1696a3bf4358680f585097c5dfcb6"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-05 12:10:07.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-10 15:26:30.000000000","tz":-240},"subject":"Only run chmod/chown for local files","message":"Only run chmod/chown for local files\n\nWhen the /opt/stack directory is NFS mounted, chown to\nthe local user might not work. Skip that step, as well\nas any recursive chmod calls, to avoid spurrious errors.\n\nChange-Id: Iaa68879e867a4426b1990d4d46164769177dc7cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/12bc5f546ed00aff0a457fe926ff650e09d2198a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/12bc5f546ed00aff0a457fe926ff650e09d2198a"}]},"branch":"refs/heads/master"},"cf505110f77adee975c512c9134fe28d0314d9bd":{"kind":"REWORK","_number":4,"created":"2013-09-10 18:50:55.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/48/45248/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/45248/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/45248/4"}}},"commit":{"parents":[{"commit":"72854cc567d1696a3bf4358680f585097c5dfcb6","subject":"Merge \"xenapi: enable block device access for stack user\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/72854cc567d1696a3bf4358680f585097c5dfcb6"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-05 12:10:07.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-10 18:50:49.000000000","tz":-240},"subject":"Only run chmod/chown for local files","message":"Only run chmod/chown for local files\n\nWhen the /opt/stack directory is NFS mounted, chown to\nthe local user might not work. Skip that step, as well\nas any recursive chmod calls, to avoid spurrious errors.\n\nChange-Id: Iaa68879e867a4426b1990d4d46164769177dc7cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cf505110f77adee975c512c9134fe28d0314d9bd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/cf505110f77adee975c512c9134fe28d0314d9bd"}]},"branch":"refs/heads/master"},"e700267e33d748fe36c621cf16497597fcbe6aac":{"kind":"REWORK","_number":5,"created":"2013-09-10 19:16:38.000000000","uploader":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"ref":"refs/changes/48/45248/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/48/45248/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/48/45248/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/48/45248/5"}}},"commit":{"parents":[{"commit":"72854cc567d1696a3bf4358680f585097c5dfcb6","subject":"Merge \"xenapi: enable block device access for stack user\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/72854cc567d1696a3bf4358680f585097c5dfcb6"}]}],"author":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-05 12:10:07.000000000","tz":-240},"committer":{"name":"Doug Hellmann","email":"doug.hellmann@dreamhost.com","date":"2013-09-10 19:15:58.000000000","tz":-240},"subject":"Only run chmod/chown for local files","message":"Only run chmod/chown for local files\n\nWhen the /opt/stack directory is NFS mounted, chown to\nthe local user might not work. Create safe_chown and\nsafe_chmod functions that do nothing on NFS filesystems\nto avoid spurrious errors.\n\nChange-Id: Iaa68879e867a4426b1990d4d46164769177dc7cc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e700267e33d748fe36c621cf16497597fcbe6aac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e700267e33d748fe36c621cf16497597fcbe6aac"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}