)]}'
{"inc/python":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"504814a643839a303e1d2b4a1593f0c379052205","unresolved":false,"context_lines":[{"line_number":163,"context_line":"                # support for python3 in progress, but don\u0027t claim support"},{"line_number":164,"context_line":"                # in their classifier"},{"line_number":165,"context_line":"                echo \"Check python version for : $package_dir\""},{"line_number":166,"context_line":"                if [[ ${package_dir##*/} \u003d\u003d \"nova\" || ${package_dir##*/} \u003d\u003d \"glance\" || \\"},{"line_number":167,"context_line":"                        ${package_dir##*/} \u003d\u003d \"cinder\" || ${package_dir##*/} \u003d\u003d \"swift\" || \\"},{"line_number":168,"context_line":"                        ${package_dir##*/} \u003d\u003d \"uwsgi\" ]]; then"},{"line_number":169,"context_line":"                    echo \"Using $PYTHON3_VERSION version to install $package_dir\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da36d5c6_0f6749eb","side":"PARENT","line":166,"updated":"2017-02-27 14:19:29.000000000","message":"It should be safe to remove this section.","commit_id":"0072abbf11939a9ab4b5be9f5417b1f463dd4e3d"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"504814a643839a303e1d2b4a1593f0c379052205","unresolved":false,"context_lines":[{"line_number":170,"context_line":"                    sudo_pip\u003d\"$sudo_pip LC_ALL\u003den_US.UTF-8\""},{"line_number":171,"context_line":"                    cmd_pip\u003d$(get_pip_command $PYTHON3_VERSION)"},{"line_number":172,"context_line":"                elif [[ -d \"$package_dir\" ]]; then"},{"line_number":173,"context_line":"                    python_versions\u003d$(get_python_versions_for_package $package_dir)"},{"line_number":174,"context_line":"                    if [[ $python_versions \u003d~ $PYTHON3_VERSION ]]; then"},{"line_number":175,"context_line":"                        echo \"Using $PYTHON3_VERSION version to install $package_dir\""},{"line_number":176,"context_line":"                        sudo_pip\u003d\"$sudo_pip LC_ALL\u003den_US.UTF-8\""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da36d5c6_6fffe5aa","side":"PARENT","line":173,"updated":"2017-02-27 14:19:29.000000000","message":"This section was slated for removal after https://review.openstack.org/#/c/418125/.","commit_id":"0072abbf11939a9ab4b5be9f5417b1f463dd4e3d"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"504814a643839a303e1d2b4a1593f0c379052205","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                        fi"},{"line_number":188,"context_line":"                    fi"},{"line_number":189,"context_line":"                else"},{"line_number":190,"context_line":"                    # Check pypi as we don\u0027t have the package on disk"},{"line_number":191,"context_line":"                    package\u003d$(echo $package_dir | grep -o \u0027^[.a-zA-Z0-9_-]*\u0027)"},{"line_number":192,"context_line":"                    python3_classifier\u003d$(check_python3_support_for_package_remote $package)"},{"line_number":193,"context_line":"                    if [[ ! -z \"$python3_classifier\" ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da36d5c6_8f4b395f","side":"PARENT","line":190,"updated":"2017-02-27 14:19:29.000000000","message":"I don\u0027t understand why this section was here to start.","commit_id":"0072abbf11939a9ab4b5be9f5417b1f463dd4e3d"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"504814a643839a303e1d2b4a1593f0c379052205","unresolved":false,"context_lines":[{"line_number":152,"context_line":"            cmd_pip\u003d$(get_pip_command $PYTHON2_VERSION)"},{"line_number":153,"context_line":"            local sudo_pip\u003d\"sudo -H\""},{"line_number":154,"context_line":"            if python3_enabled; then"},{"line_number":155,"context_line":"                # If running under python3, use that for all pip commands"},{"line_number":156,"context_line":"                local package_dir\u003d${!#}"},{"line_number":157,"context_line":"                local python_versions"},{"line_number":158,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"da36d5c6_8fced9b5","line":155,"updated":"2017-02-27 14:19:29.000000000","message":"The explicit switch added in https://review.openstack.org/#/c/418125/ solves the problem of projects that don\u0027t support python 3, yet. We can set DISABLED_PYTHON3_PACKAGES to default to the list of projects that are commonly included in co-gating test jobs. Maybe that patch should be updated to only have the disabled list, and default everything else to enabled (like this patch is doing)?","commit_id":"5fac4ef862e44864a2d627bc3fbd3763882ec8b2"}]}