)]}'
{"playbooks/fix-venv.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"cc32408e4e3c08a44e44be3be5f18f3a60c308be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: all"},{"line_number":2,"context_line":"  pre_tasks:"},{"line_number":3,"context_line":"    - name: Ensure virtualenv is capped"},{"line_number":4,"context_line":"      shell: |"},{"line_number":5,"context_line":"        set -euo pipefail"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1fa4df85_75d06220","line":2,"updated":"2020-03-02 13:40:47.000000000","message":"Is there a reason not to use ansible\u0027s pip module? E.g.:\n\n  pre_tasks:\n    - name: Ensure virtualenv is capped\n      pip:\n        name: virtualenv\u003d\u003d20.0.5\n        extra_args: --user --upgrade","commit_id":"85e48ff7e3bd159107733658f636f2ff481132b2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9f259e08162834f89d53db0df426b7a79925c4f6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: all"},{"line_number":2,"context_line":"  pre_tasks:"},{"line_number":3,"context_line":"    - name: Ensure virtualenv is capped"},{"line_number":4,"context_line":"      shell: |"},{"line_number":5,"context_line":"        set -euo pipefail"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1fa4df85_ea8f8c20","line":2,"in_reply_to":"1fa4df85_75d06220","updated":"2020-03-02 16:11:44.000000000","message":"Well, no specific reason, I just copied this code from the ensure-tox role. Not sure how you\u0027d tell the ansible module when to use pip vs. pip3.","commit_id":"85e48ff7e3bd159107733658f636f2ff481132b2"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"d1b2170803f9d9b002b685e8e2e185614d7b04a6","unresolved":false,"context_lines":[{"line_number":10,"context_line":"          PIP\u003dpip3"},{"line_number":11,"context_line":"        fi"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"        $PIP install --user virtualenv\u003d\u003d20.0.5"},{"line_number":14,"context_line":"      args:"},{"line_number":15,"context_line":"        executable: /bin/bash"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1fa4df85_963724e4","line":13,"updated":"2020-03-02 15:17:25.000000000","message":"I think this requires --upgrade","commit_id":"85e48ff7e3bd159107733658f636f2ff481132b2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9f259e08162834f89d53db0df426b7a79925c4f6","unresolved":false,"context_lines":[{"line_number":10,"context_line":"          PIP\u003dpip3"},{"line_number":11,"context_line":"        fi"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"        $PIP install --user virtualenv\u003d\u003d20.0.5"},{"line_number":14,"context_line":"      args:"},{"line_number":15,"context_line":"        executable: /bin/bash"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1fa4df85_2a9fe45f","line":13,"in_reply_to":"1fa4df85_963724e4","updated":"2020-03-02 16:11:44.000000000","message":"Not if one specifies an explicit version IIUC. It did work fine according to https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_cde/710729/4/check/cross-keystone-py27/cde8e6e/job-output.txt , just need to fix both py2 and py3, thus the new PS.","commit_id":"85e48ff7e3bd159107733658f636f2ff481132b2"}]}
