)]}'
{"tools/kolla-ansible":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"01309a1524d85f3d9191b3b26b9f64175b49b902","unresolved":false,"context_lines":[{"line_number":124,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"# Detect which python interpreter to use."},{"line_number":127,"context_line":"for python in python python3; do"},{"line_number":128,"context_line":"    if ! $python -h \u0026\u003e/dev/null; then"},{"line_number":129,"context_line":"        continue"},{"line_number":130,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3fa7e38b_40c14181","line":127,"range":{"start_line":127,"start_character":0,"end_line":127,"end_character":32},"updated":"2019-11-27 14:36:32.000000000","message":"can we default to python3?","commit_id":"8212af5e584022bd6015783444402e8940555c8e"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"389633523f7bd5f0de523dff5682ce9583fab6a4","unresolved":false,"context_lines":[{"line_number":124,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"# Detect which python interpreter to use."},{"line_number":127,"context_line":"for python in python python3; do"},{"line_number":128,"context_line":"    if ! $python -h \u0026\u003e/dev/null; then"},{"line_number":129,"context_line":"        continue"},{"line_number":130,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3fa7e38b_54d32660","line":127,"range":{"start_line":127,"start_character":0,"end_line":127,"end_character":32},"in_reply_to":"3fa7e38b_00d6e9f3","updated":"2019-11-29 12:45:09.000000000","message":"for python in python3 python2 (or python2 python3 if you want).\n\nPEP-394 allows to have /usr/bin/python -\u003e /usr/bin/python3 symlink. So we should not use unversioned Python.","commit_id":"8212af5e584022bd6015783444402e8940555c8e"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6e25cea78dfe48bd22163e9ea31f12c72a8af059","unresolved":false,"context_lines":[{"line_number":124,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"# Detect which python interpreter to use."},{"line_number":127,"context_line":"for python in python python3; do"},{"line_number":128,"context_line":"    if ! $python -h \u0026\u003e/dev/null; then"},{"line_number":129,"context_line":"        continue"},{"line_number":130,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3fa7e38b_00d6e9f3","line":127,"range":{"start_line":127,"start_character":0,"end_line":127,"end_character":32},"in_reply_to":"3fa7e38b_40c14181","updated":"2019-11-27 14:56:35.000000000","message":"We could do. You\u0027d need to have kolla_ansible installed in both py2 and py3 to hit it though. And if we remove py2 support then it won\u0027t matter anyway...","commit_id":"8212af5e584022bd6015783444402e8940555c8e"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"abc5f6964cdee6921d81c4df1fcfa26b799daaba","unresolved":false,"context_lines":[{"line_number":125,"context_line":""},{"line_number":126,"context_line":"# Detect which python interpreter to use."},{"line_number":127,"context_line":"for python in python3 python2; do"},{"line_number":128,"context_line":"    if ! $python -h \u0026\u003e/dev/null; then"},{"line_number":129,"context_line":"        continue"},{"line_number":130,"context_line":"    fi"},{"line_number":131,"context_line":"    if $python -c \u0027import kolla_ansible\u0027 \u0026\u003e/dev/null; then"},{"line_number":132,"context_line":"        PYTHON\u003d$python"},{"line_number":133,"context_line":"        break"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"3fa7e38b_1350ef5d","line":130,"range":{"start_line":128,"start_character":0,"end_line":130,"end_character":6},"updated":"2019-12-09 12:25:08.000000000","message":"not necessary, you don\u0027t differentiate between lack of python and lack of k-a anyway","commit_id":"a6a7eb170ecb796f2955ebfe0332e0baf5679d5b"}]}