)]}'
{"id":"openstack%2Fopenstack-ansible-tests~723219","triplet_id":"openstack%2Fopenstack-ansible-tests~master~I27f80d8aa4ac3688ee56e8a9234e34f4e09afe34","project":"openstack/openstack-ansible-tests","branch":"master","topic":"pip-install","hashtags":[],"change_id":"I27f80d8aa4ac3688ee56e8a9234e34f4e09afe34","subject":"Use \"pip -m install\"","status":"MERGED","created":"2020-04-26 12:48:47.000000000","updated":"2020-04-27 15:38:29.000000000","submitted":"2020-04-27 15:35:05.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"723219-1588001705388-166cf2d9","meta_rev_id":"aeb07f82e6192067021eb6d6126610fa1d66ccbb","_number":723219,"virtual_id_number":723219,"owner":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-04-27 15:35:05.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"}],"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":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"all":[{"value":2,"date":"2020-04-27 14:53:40.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-04-27 13:13:29.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"}],"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":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"all":[{"value":1,"date":"2020-04-27 14:53:40.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-04-27 13:13:29.000000000","updated_by":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"reviewer":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"state":"REVIEWER"},{"updated":"2020-04-27 14:53:40.000000000","updated_by":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"reviewer":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"state":"REVIEWER"},{"updated":"2020-04-27 15:35:05.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"e6e47f7feed233f591fbdd79cb951cf539860a2c","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2020-04-26 12:48:47.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"7e7540336ce1809cac67c11c0d3484a76c8ae95e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-26 13:35:48.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-ansible-linters https://zuul.opendev.org/t/openstack/build/65ddcedb4a7a4575aa3778f1b70ae80a : SUCCESS in 8m 31s\n- openstack-ansible-functional-debian-buster https://zuul.opendev.org/t/openstack/build/6736113470ab47e3b96ce8c2b7520729 : SUCCESS in 30m 49s\n- openstack-ansible-functional-centos-7 https://zuul.opendev.org/t/openstack/build/20eec0a57fb4438c9a11ac5770621a17 : SUCCESS in 32m 43s\n- openstack-ansible-functional-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f276e811f4be48e69450e1864e762944 : SUCCESS in 26m 44s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/590a762ea2934d9fad518edf0be1671f : SUCCESS in 4m 54s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/06c2b0dd1eaf4835aba33ae9714c2b80 : SUCCESS in 3m 10s","accounts_in_message":[],"_revision_number":1},{"id":"3108720cd31d33488f07a0aad538eec24ba974a3","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2020-04-27 13:13:29.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"6cb52918d1dc7a626b58477450f5a8ccb33afd27","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2020-04-27 14:53:40.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"1c0caebc8f9b88fb7e0d5c2610886fbdfafbfcef","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-27 14:53:52.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"696476ccdcd56c1b1e76d6a0fa70cf471e4d25b6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-27 15:35:05.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"c140ae50bdccf8f14d8c6b1493adc191d0e82872","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-27 15:35:05.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-ansible-linters https://zuul.opendev.org/t/openstack/build/c857451d8e0c4e949b46ec457b7702b3 : SUCCESS in 9m 39s\n- openstack-ansible-functional-debian-buster https://zuul.opendev.org/t/openstack/build/4af0b2623ca8451f816f9f9c902aec01 : SUCCESS in 35m 39s\n- openstack-ansible-functional-centos-7 https://zuul.opendev.org/t/openstack/build/a29144bd5b504414a57991690c6d1601 : SUCCESS in 35m 22s\n- openstack-ansible-functional-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/373ac904cab5451bbc8a5e3f6cb16200 : SUCCESS in 31m 10s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c2229212c4364024bdc515db990e8592 : SUCCESS in 7m 18s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/3eb566fbb3bc4cfa93a13a1b25a90852 : SUCCESS in 3m 47s","accounts_in_message":[],"_revision_number":1},{"id":"3a9f6b04879a34e99056004a32557992b9350528","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-27 15:38:29.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/68ef54e5a1f744d590b8af7fc78c1e8c : SUCCESS in 2m 59s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/77aab59ba365418e8daf2339d5aba26e : SUCCESS in 2m 51s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"74797e4291750a08f5ece97053c041050691a89e","revisions":{"74797e4291750a08f5ece97053c041050691a89e":{"kind":"REWORK","_number":1,"created":"2020-04-26 12:48:47.000000000","uploader":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"ref":"refs/changes/19/723219/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-tests","ref":"refs/changes/19/723219/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/19/723219/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/19/723219/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/19/723219/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-tests refs/changes/19/723219/1"}}},"commit":{"parents":[{"commit":"ec787687e64644e4ea434141b9925eaabb6e55eb","subject":"Merge \"Cleanup py27 support\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/ec787687e64644e4ea434141b9925eaabb6e55eb"}]}],"author":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2020-04-26 12:42:30.000000000","tz":120},"committer":{"name":"Andreas Jaeger","email":"aj@suse.com","date":"2020-04-26 12:48:25.000000000","tz":120},"subject":"Use \"pip -m install\"","message":"Use \"pip -m install\"\n\nCurrently, we are overriding \u0027install_command\u0027 to use \u0027pip\u0027. This is\nconsidered poor behavior and \u0027python -m pip\u0027 should be used instead:\n\nhttps://snarky.ca/why-you-should-use-python-m-pip/\n\nIt turns out that this is the the default value provided by tox:\n\nhttps://tox.readthedocs.io/en/latest/config.html#conf-install_command\n\nSo we can remove the line and simply use the default value - after\nadding constraints to deps as is best practice as well.\n\nMosts tests itself use \"pip install\" in the shell scripts which sets\nalso the constraints. Change that to \"python -m pip install\" like done\nfor tox.\n\nChange-Id: I27f80d8aa4ac3688ee56e8a9234e34f4e09afe34\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/74797e4291750a08f5ece97053c041050691a89e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-tests/commit/74797e4291750a08f5ece97053c041050691a89e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}