)]}'
{"id":"openstack%2Fdevstack~51659","triplet_id":"openstack%2Fdevstack~master~If966592bd089aa85b291d624eec267dfaa310d36","project":"openstack/devstack","branch":"master","topic":"drop-pip1.4-hack","hashtags":[],"change_id":"If966592bd089aa85b291d624eec267dfaa310d36","subject":"fixup_stuff.sh: Drop pip 1.4 permissions hack","status":"ABANDONED","created":"2013-10-14 16:26:27.000000000","updated":"2013-10-18 09:04:34.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"36167cf3378ca9ed5490f25476b790c241f78e71","_number":51659,"virtual_id_number":51659,"owner":{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2013-10-14 17:40:45.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-10-14 16:41:32.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2013-10-14 16:34:30.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"}],"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":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2013-10-14 16:41:32.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-14 16:41:32.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-14 17:40:45.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"c4ac750ff7f7763acf7ccde8f546a5b1f9c25261","author":{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"},"date":"2013-10-14 16:34:30.000000000","message":"Patch Set 1:\n\nI would appreciate if someone could take a look to see if this patch makes sense. At least with pip 1.4.1, the permissions problem that the hack purports to fix seems to no longer exist. Thanks.","accounts_in_message":[],"_revision_number":1},{"id":"b730a98cd5e7d8a7e957b78e1ec496ca7e6d6d90","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-14 16:41:32.000000000","message":"Patch Set 1:\n\nThis is exactly what should happen, eventually.  I\u0027d like to be a bit conservative about the timing though so we don\u0027t regress in unexpected places.","accounts_in_message":[],"_revision_number":1},{"id":"beb0ce946c91425b4810471f1e451755e0827bee","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-14 17:40:45.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/59/51659/1/check/check-tempest-devstack-vm-full/124d362 : SUCCESS in 1h 00m 48s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/59/51659/1/check/check-tempest-devstack-vm-postgres-full/fa494b6 : SUCCESS in 38m 51s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/59/51659/1/check/check-tempest-devstack-vm-neutron/b16fc42 : SUCCESS in 22m 34s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/59/51659/1/check/gate-tempest-devstack-vm-large-ops/9d64241 : SUCCESS in 20m 06s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/59/51659/1/check/gate-tempest-devstack-vm-neutron-large-ops/2f5df91 : SUCCESS in 15m 50s (non-voting)\n- check-grenade-devstack-vm http://logs.openstack.org/59/51659/1/check/check-grenade-devstack-vm/64aa1ed : SUCCESS in 39m 39s\n- check-swift-devstack-vm-functional http://logs.openstack.org/59/51659/1/check/check-swift-devstack-vm-functional/1a93f79 : SUCCESS in 17m 12s\n","accounts_in_message":[],"_revision_number":1},{"id":"a4814a791f98a11d32649f0e4b820a86183ec098","author":{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"},"date":"2013-10-15 06:29:37.000000000","message":"Patch Set 1:\n\nHi Dean,\n\nactually, the hack as it is breaks DevStack on RHEL (at least the one I tested this on). There\u0027s already a prettytable 0.6 on that one, and since pip_install doesn\u0027t set --upgrade, pip will just skip the installation. Hence, prettytable 0.7.2 never gets installed, and this line just fails:\n\nsudo chmod +r $dir/prettytable-0.7.2*/*\n\nAny thoughts on that?","accounts_in_message":[],"_revision_number":1},{"id":"60660837c4d12e633e0f539d2abab9641d59d3ee","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-16 16:24:58.000000000","message":"Patch Set 1:\n\nThe prettytable bit is causing problems on RHEL, I get that.  It also is supposed to only apply to 0.7.2 as that package is broken.\n\nI\u0027m going through all of the attempts to fix this and will figure out which one does what I want: fix 0.7.2 if present and leave everything else alone.  I specifically don\u0027t want to install 0.7.2 on top of the OS package on RHEL as the python config on RHEL is broken and installs pip-installed packages on top of the OS-installed package.","accounts_in_message":[],"_revision_number":1},{"id":"23b420a0a4fcdeb8dbf8effd71e887f0fe74f801","author":{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"},"date":"2013-10-16 17:20:39.000000000","message":"Patch Set 1:\n\nDean, I\u0027ve posted an alternative approach to resolving this in change I50d3b29ff1e2e0c5451700adaa4a634cf70209fc. Please take a look at that; I believe it may suit your preference better.","accounts_in_message":[],"_revision_number":1},{"id":"58b3296f982f386edd432cfca4e193481a2c6674","author":{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"},"date":"2013-10-18 09:04:34.000000000","message":"Patch Set 1: Abandoned\n\nProperly addressed in I99906451dc25654628187b383e8893cce0e276bf.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"24dd15557c9d5a8ab2778a9d2ce01627c305a075","revisions":{"24dd15557c9d5a8ab2778a9d2ce01627c305a075":{"kind":"REWORK","_number":1,"created":"2013-10-14 16:26:27.000000000","uploader":{"_account_id":2463,"name":"Florian Haas","email":"florian.haas@cleura.com","username":"fghaas"},"ref":"refs/changes/59/51659/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/59/51659/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/59/51659/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/59/51659/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/59/51659/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/59/51659/1"}}},"commit":{"parents":[{"commit":"3931573f2bdb542ff4299bd548cab3458c3b0c99","subject":"mute useless console output, when run ./stack.sh","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3931573f2bdb542ff4299bd548cab3458c3b0c99"}]}],"author":{"name":"Florian Haas","email":"florian@hastexo.com","date":"2013-10-14 16:19:52.000000000","tz":120},"committer":{"name":"Florian Haas","email":"florian@hastexo.com","date":"2013-10-14 16:21:49.000000000","tz":120},"subject":"fixup_stuff.sh: Drop pip 1.4 permissions hack","message":"fixup_stuff.sh: Drop pip 1.4 permissions hack\n\nThe hack involving broken package permissions after a pip install\nis no longer necessary, as the problem does no longer exist with\npip 1.4.1.\n\nTested on RHEL 6.\n\nChange-Id: If966592bd089aa85b291d624eec267dfaa310d36\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/24dd15557c9d5a8ab2778a9d2ce01627c305a075"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/24dd15557c9d5a8ab2778a9d2ce01627c305a075"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}