)]}'
{"id":"openstack%2Fdevstack~24291","triplet_id":"openstack%2Fdevstack~master~I869f727900bb08c72e5e8f4e84858a9c77049715","project":"openstack/devstack","branch":"master","hashtags":[],"change_id":"I869f727900bb08c72e5e8f4e84858a9c77049715","subject":"Don\u0027t directly pip install tempest pip-requires","status":"MERGED","created":"2013-03-13 07:00:57.000000000","updated":"2013-03-13 12:15:41.000000000","submitted":"2013-03-13 12:15:39.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a3f76d99f4ce3c779546c806707e475f145e6bed","_number":24291,"virtual_id_number":24291,"owner":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-03-13 12:15:39.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-03-13 11:34:23.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"date":"2013-03-13 07:07:28.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-03-13 11:35:38.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"}],"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":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-03-13 11:35:38.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-03-13 12:15:39.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"2897bf5f1f9d2aaf490dd451db8d492f5fe5c95a","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-03-13 07:07:28.000000000","message":"Patch Set 1:\n\nSee I655f831718ae5f4e25e941ee206fe195214a9a91 for the failures this appears to cause","accounts_in_message":[],"_revision_number":1},{"id":"61d0675209e4efcb851ef315d7ff19db0f2b1b3a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-13 07:40:54.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/24291/1/check/gate-tempest-devstack-vm-full/8596 : SUCCESS\n- http://logs.openstack.org/24291/1/check/gate-tempest-devstack-vm-postgres-full/8312 : SUCCESS\n- http://logs.openstack.org/24291/1/check/gate-tempest-devstack-vm-quantum/13859 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"345ccfc08146276ca6e68849dd57ee9e69744680","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-13 11:34:23.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"00f57c44bf3a22bda2dad6a62576e4667f84fbcb","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-13 11:35:38.000000000","message":"Patch Set 1: Approved\n\nthanks Mark, getting this through quickly so we can get the rest of the oslo flips over.","accounts_in_message":[],"_revision_number":1},{"id":"e837435d45ed29748e32f2cfd3964f552cbb62a5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-13 11:35:40.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"68eecb1733da952d64270cb3b6b0678f4fba964c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-13 12:15:39.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/24291/1/gate/gate-tempest-devstack-vm-full/8621 : SUCCESS\n- http://logs.openstack.org/24291/1/gate/gate-tempest-devstack-vm-postgres-full/8337 : SUCCESS\n- http://logs.openstack.org/24291/1/gate/gate-tempest-devstack-vm-quantum/13885 : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"a3f76d99f4ce3c779546c806707e475f145e6bed","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-03-13 12:15:41.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"7ddbea25c4c443709809a8da92ecbb562cc3a62d","revisions":{"7ddbea25c4c443709809a8da92ecbb562cc3a62d":{"kind":"REWORK","_number":1,"created":"2013-03-13 07:00:57.000000000","uploader":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"ref":"refs/changes/91/24291/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/91/24291/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/91/24291/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/91/24291/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/91/24291/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/91/24291/1"}}},"commit":{"parents":[{"commit":"f69ce0bce5db74ca87214c80d3792066284e8c5a","subject":"Merge \"Quantum: use generic VIF driver\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f69ce0bce5db74ca87214c80d3792066284e8c5a"}]}],"author":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-03-13 06:57:19.000000000","tz":0},"committer":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-03-13 06:57:19.000000000","tz":0},"subject":"Don\u0027t directly pip install tempest pip-requires","message":"Don\u0027t directly pip install tempest pip-requires\n\nThis code:\n\n    # Tempest doesn\u0027t satisfy its dependencies on its own, so\n    # install them here instead.\n    pip_install -r $TEMPEST_DIR/tools/pip-requires\n\ncauses us to do:\n\n    Found existing installation: oslo.config 1.1.0\n     Uninstalling oslo.config:\n\nso rather than 1.1.0 installed, we end up with 1.1.0b1 and everything\ngoes kabloomy.\n\nIt looks like this comment came from a time (commit 9a352daf) before\ntempest included proper install_requires (tempest commit 5abb253c3).\n\nIt should be safe to delete this and allow setup_develop() do all the\nwork just like the other projects.\n\nChange-Id: I869f727900bb08c72e5e8f4e84858a9c77049715\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7ddbea25c4c443709809a8da92ecbb562cc3a62d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/7ddbea25c4c443709809a8da92ecbb562cc3a62d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}