)]}'
{"id":"openstack%2Fdevstack~161161","triplet_id":"openstack%2Fdevstack~master~I4a723f179bdc28d39a4910fb9e3787e9e67c354b","project":"openstack/devstack","branch":"master","topic":"no-rpm-unisntall","hashtags":[],"change_id":"I4a723f179bdc28d39a4910fb9e3787e9e67c354b","subject":"Do not install the python-virtualenv on Fedora","status":"MERGED","created":"2015-03-04 10:24:08.000000000","updated":"2015-03-05 14:44:15.000000000","submitted":"2015-03-05 14:44:14.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"be6073ae2ed8dae2779133a52c5ee14b6f0d2219","_number":161161,"virtual_id_number":161161,"owner":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"recommended":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"all":[{"value":1,"date":"2015-03-04 18:45:31.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":2,"date":"2015-03-05 14:44:14.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2015-03-04 14:23:16.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"recommended":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"all":[{"value":0,"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":1,"date":"2015-03-05 09:35:50.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":1,"date":"2015-03-05 06:25:36.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":1,"date":"2015-03-05 08:16:03.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":2,"date":"2015-03-04 21:43:29.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2015-03-05 13:39:45.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":1,"date":"2015-03-05 13:39:45.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":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"value":0,"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-04 10:43:36.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":"2015-03-04 10:43:54.000000000","updated_by":{"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"},"reviewer":{"_account_id":12535,"name":"Yaniv Kaul","email":"yaniv.kaul@emc.com","username":"yanivk"},"state":"REVIEWER"},{"updated":"2015-03-04 13:25:14.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":"2015-03-04 18:45:31.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-03-04 21:43:29.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2015-03-05 06:25:36.000000000","updated_by":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"reviewer":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"state":"REVIEWER"},{"updated":"2015-03-05 08:16:03.000000000","updated_by":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"reviewer":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"state":"REVIEWER"},{"updated":"2015-03-05 09:35:50.000000000","updated_by":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"reviewer":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"state":"REVIEWER"},{"updated":"2015-03-05 13:39:45.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":"2015-03-05 14:44:14.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"a9cd9e56a689c130da337f1836d7fe02ee6cb81d","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-04 10:24:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"07c7d90f421ca2d35a40a942921bddbe7c80ac1a","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2015-03-04 13:25:14.000000000","message":"Patch Set 1: Code-Review-1\n\nIt would appear then that the real fix is to remove python-virtualenv from files/rpms/general and not install it in the first place.  Actually that was my oversight in the original change.  At the _very_ least, this is a problem only for the is_fedora() distros and should be restricted to them.\n\nThe situation with pip-installed packages on RH distros is broken.  I don\u0027t have the history of why it has never been fixed, I don\u0027t care.  We\u0027ve had to work around that brokenness more than once and it is a pain to debug.\n\nHaving a package installer fight dropping files on top of each other is not a solution.  Removing one of those package managers from the picture is the only solution that works in the current RH environment.\n\nsite-packages vs dist-packages.  Jeez, even Perl4 got this right on Windows back in 1995...","accounts_in_message":[],"_revision_number":1},{"id":"d0f7ddf5e53a03798a05fe120d5a202971d610cc","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-04 13:59:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"526f4411faa1af1ada977c64cc1a2986abcb60bb","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-04 14:23:16.000000000","message":"Patch Set 2:\n\ncpan already solved this issue in many ways, but python/pip does not.\nAFAIK there is no python __upstream__ guideline or tool for using the dist-packages.\n\nThe dist-packages uses violates the http://refspecs.linuxfoundation.org/fhs.shtml .\n\nThe pip installed things should be installed to /usr/local or /opt.\n\nTo have Fedora to use different location with rpm it needs to came from python upstream and it needs to supported by upstream tools.\n\npip should use different location like \u0027/usr/local/python/site-packages\u0027 and Python should search for modules first on that location.\n\nAs my cpan did by default:\nInstalling /usr/local/lib64/perl5/FCGI.pm\nInstalling /usr/local/lib64/perl5/auto/FCGI/FCGI.so\nInstalling /usr/local/share/man/man3/FCGI.3pm","accounts_in_message":[],"_revision_number":2},{"id":"03f8b42f4b4b2ba0ca40ec115aa153ddce7d1924","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-04 17:37:18.000000000","message":"Patch Set 2:\n\nSorry:\nThe /usr/lib*/python*/site-packages usage _by pip_ violates the http://refspecs.linuxfoundation.org/fhs.shtml .","accounts_in_message":[],"_revision_number":2},{"id":"a8ad34aa84ec588412674ae59c6012de8fe6d147","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-03-04 18:45:31.000000000","message":"Patch Set 2: Verified+1\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/61/161161/2/11706/results.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":2},{"id":"d5dd6265d75ac2916f4327edb017713c427b1a49","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-03-04 21:43:29.000000000","message":"Patch Set 2: Code-Review+2\n\n1) the original commit should have probably been gated with some check.  it\u0027s not going to be true for perpetuity that the bug will exist in the python-virtualenv package\n\n2) it doesn\u0027t really matter, i don\u0027t see any need to install the packaged version, so this looks good\n\n3) yes we\u0027ve discussed this pip/rpm issue before.\n\na) i believe things are changing in fedora so that packages are built with pip, so essentially pip can overwrite them correctly.  the issue can be at the moment that pip doesn\u0027t know how to remove everything related to a package, so if an old file is left around sometimes it plays havoc.\n\nthat\u0027s why we get\n\n \"Uninstalling a distutils installed project (...) has been deprecated and will be removed in a future version.\"\n\ni believe in the future pip will know all the files installed by the rpm, get rid of them and start again.\n\nb) not that this really helps centos7; but we really are targeting a completely blank-install, so there\u0027s a limited number of python packages that *have* to be installed (things relating to cloud-init).  we really could/should strip out a lot of the package installs i think and reduce our exposure to the problem.","accounts_in_message":[],"_revision_number":2},{"id":"a9eeeffe4f548c3f52f61398666eb0a530854d29","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-04 22:59:07.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- check-tempest-dsvm-full http://logs.openstack.org/61/161161/2/check/check-tempest-dsvm-full/bb613db/ : SUCCESS in 47m 22s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/61/161161/2/check/check-tempest-dsvm-postgres-full/0b1300b/ : SUCCESS in 46m 04s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/61/161161/2/check/check-tempest-dsvm-neutron-full/1a36bd5/ : SUCCESS in 54m 32s\n- check-grenade-dsvm http://logs.openstack.org/61/161161/2/check/check-grenade-dsvm/9ebacf6/ : SUCCESS in 45m 00s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/61/161161/2/check/gate-tempest-dsvm-large-ops/28d1ad3/ : SUCCESS in 29m 47s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/61/161161/2/check/gate-tempest-dsvm-neutron-large-ops/d17db34/ : SUCCESS in 30m 54s\n- gate-devstack-bashate http://logs.openstack.org/61/161161/2/check/gate-devstack-bashate/fdb6e06/ : SUCCESS in 42s\n- gate-devstack-unit-tests http://logs.openstack.org/61/161161/2/check/gate-devstack-unit-tests/6872616/ : SUCCESS in 16s\n- check-devstack-dsvm-cells http://logs.openstack.org/61/161161/2/check/check-devstack-dsvm-cells/35c69fe/ : SUCCESS in 24m 26s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/61/161161/2/check/check-grenade-dsvm-partial-ncpu/52714d8/ : SUCCESS in 47m 29s\n- check-swift-dsvm-functional http://logs.openstack.org/61/161161/2/check/check-swift-dsvm-functional/5bbe6d5/ : SUCCESS in 29m 28s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/61/161161/2/check/check-tempest-dsvm-ironic-pxe_ssh/e79c019/ : SUCCESS in 39m 14s\n- check-devstack-dsvm-updown http://logs.openstack.org/61/161161/2/check/check-devstack-dsvm-updown/6ee7254/ : SUCCESS in 24m 43s\n- check-tempest-dsvm-f21 http://logs.openstack.org/61/161161/2/check/check-tempest-dsvm-f21/ad87ad8/ : FAILURE in 50m 28s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/61/161161/2/check/check-tempest-dsvm-centos7/ee41e9d/ : FAILURE in 12m 58s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/61/161161/2/check/gate-devstack-publish-docs/3d5e024//doc/build/html/ : SUCCESS in 1m 48s","accounts_in_message":[],"_revision_number":2},{"id":"caab5e01124b3f38fefaa63459f5ef0b0e0caa7b","author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"date":"2015-03-05 06:25:36.000000000","message":"Patch Set 2: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"50b35b303d7cecafb1c8b3c904932c71bd12397c","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2015-03-05 07:19:09.000000000","message":"Patch Set 2:\n\n@ianw: We should not have to limit the existing distro packages on the system, just because the python install tools by default does not respects the FHS.\n\nEven if pip removes the content of the rpm package, when you re-install or upgrade the distro package the egg files MAY conflict. \n\nThe expected operation:\nIf I already have a compatible site-package devstack pip whatever will simply use it.\n\nIf the already existing package not good, devstack/setup.py.. installs a different version to a single dedicated location used by all Openstack competent like $DEST/pythonlib, PYTHONPATH should be extended by devstack.\nThe /usr/lib* files installed by rpm MUST not be touched.\n\nIf I am able to install every required dependency from rpm before starting devstack, the $DEST/pythonlib should be empty.","accounts_in_message":[],"_revision_number":2},{"id":"4ea41d03ac9c37e5b570b8ea005b225f60b98a4d","author":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"date":"2015-03-05 08:16:03.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"c3db1a91e2bcf872afe3aff411bfa66a2f06dc49","author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"date":"2015-03-05 09:35:50.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"e2bc42fb79de49477a7f9113922be9670f5131ed","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-05 13:39:45.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"580f3163ba74146796511e7d797bf6e9428ae336","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-03-05 13:39:45.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"85db79acfdc30e9d81babf26e1a06fa5a4b88532","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-05 14:44:14.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/61/161161/2/gate/gate-tempest-dsvm-full/0826328/ : SUCCESS in 45m 07s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/61/161161/2/gate/gate-tempest-dsvm-postgres-full/62de0d0/ : SUCCESS in 43m 32s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/61/161161/2/gate/gate-tempest-dsvm-neutron-full/505652f/ : SUCCESS in 59m 14s\n- gate-grenade-dsvm http://logs.openstack.org/61/161161/2/gate/gate-grenade-dsvm/d615ecf/ : SUCCESS in 44m 56s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/61/161161/2/gate/gate-tempest-dsvm-large-ops/e148fb0/ : SUCCESS in 31m 28s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/61/161161/2/gate/gate-tempest-dsvm-neutron-large-ops/50f5fe8/ : SUCCESS in 30m 48s\n- gate-devstack-bashate http://logs.openstack.org/61/161161/2/gate/gate-devstack-bashate/d5df5da/ : SUCCESS in 27s\n- gate-devstack-dsvm-cells http://logs.openstack.org/61/161161/2/gate/gate-devstack-dsvm-cells/2761443/ : SUCCESS in 26m 11s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/61/161161/2/gate/gate-grenade-dsvm-partial-ncpu/ced3c8c/ : SUCCESS in 42m 19s\n- gate-swift-dsvm-functional http://logs.openstack.org/61/161161/2/gate/gate-swift-dsvm-functional/60fe3d1/ : SUCCESS in 26m 02s","accounts_in_message":[],"_revision_number":2},{"id":"961d575a811cd61a612ded846231ce1e8b33d891","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-05 14:44:15.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"802473e45ab897144d81d48164d8342763a119d8","revisions":{"e574c148c5b2f1a93199f78581a51944e0ef356e":{"kind":"REWORK","_number":1,"created":"2015-03-04 10:24:08.000000000","uploader":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"ref":"refs/changes/61/161161/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/61/161161/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/161161/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/161161/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/161161/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/61/161161/1"}}},"commit":{"parents":[{"commit":"f6a4305ccf31ed73f007d1dcc1d221f79b7807dd","subject":"Merge \"Revert change to remove lbaas from devstack; it breaks grenade.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f6a4305ccf31ed73f007d1dcc1d221f79b7807dd"}]}],"author":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-04 10:14:00.000000000","tz":60},"committer":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-04 10:14:00.000000000","tz":60},"subject":"Do not uninstall the python-virtualenv","message":"Do not uninstall the python-virtualenv\n\nThe files/rpms/general:python-virtualenv explicitly\ninstalled this package on Fedoras.\n\nIf you stack/unstack the package gets reinstalled,\nand confuses the system about the installed virtual-env version.\nThe uninstall works in CI, but it can be problematic when you do\nreinstalls on the same machine.\n\nThe uninstall introduced by\n834b804d3eda9029d3c66db0ab732a76a22ed08b, this commit deos not\nhas any reference to the external bug what it supposed to solve.\n\nI think both of us knows the rpm vs. pip issues.\nUsually `safer` to handle this issues by installing the\nrpm and overriding it with pip than not installing the rpm.\n\nRelated RDO thread started here:\nhttps://www.redhat.com/archives/rdo-list/2015-March/msg00015.html\n\nChange-Id: I4a723f179bdc28d39a4910fb9e3787e9e67c354b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e574c148c5b2f1a93199f78581a51944e0ef356e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e574c148c5b2f1a93199f78581a51944e0ef356e"}]},"branch":"refs/heads/master"},"802473e45ab897144d81d48164d8342763a119d8":{"kind":"REWORK","_number":2,"created":"2015-03-04 13:59:27.000000000","uploader":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"ref":"refs/changes/61/161161/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/61/161161/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/161161/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/161161/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/61/161161/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/61/161161/2"}}},"commit":{"parents":[{"commit":"f6a4305ccf31ed73f007d1dcc1d221f79b7807dd","subject":"Merge \"Revert change to remove lbaas from devstack; it breaks grenade.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f6a4305ccf31ed73f007d1dcc1d221f79b7807dd"}]}],"author":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-04 10:14:00.000000000","tz":60},"committer":{"name":"Attila Fazekas","email":"afazekas@redhat.com","date":"2015-03-04 13:57:44.000000000","tz":60},"subject":"Do not install the python-virtualenv on Fedora","message":"Do not install the python-virtualenv on Fedora\n\nThe files/rpms/general:python-virtualenv explicitly\ninstalled this package on Fedoras.\n\nThis package is not installed on other distros by devstack\nby default.\n\nIf you stack/unstack the package gets reinstalled,\nand confuses the system about the installed virtual-env version.\nThe uninstall works in CI, but it can be problematic when you do\nreinstalls on the same machine.\n\nThe uninstall introduced by\n834b804d3eda9029d3c66db0ab732a76a22ed08b, this commit deos not\nhas any reference to the external bug what it supposed to solve.\n\nRelated RDO thread started here:\nhttps://www.redhat.com/archives/rdo-list/2015-March/msg00015.html\n\nChange-Id: I4a723f179bdc28d39a4910fb9e3787e9e67c354b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/802473e45ab897144d81d48164d8342763a119d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/802473e45ab897144d81d48164d8342763a119d8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}