)]}'
{"/COMMIT_MSG":[{"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":"4dd4f4d41c42cf9588a60d7fa35780e6c7b831e8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rafael Folco \u003crfolco@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-07-03 17:42:32 -0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Enable py3 on dib release 7"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change installs python3 on a dib7 base image to allow dib"},{"line_number":10,"context_line":"to build images on older branches and distros like CentOS7/RHEL7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"bf51134e_f67dd56e","line":7,"range":{"start_line":7,"start_character":14,"end_line":7,"end_character":27},"updated":"2020-07-06 10:14:50.000000000","message":"centos/rhel 7","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"09521de09dc9a9dac4bb37a069f9560281ce3964","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rafael Folco \u003crfolco@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-07-03 17:42:32 -0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Enable py3 on dib release 7"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change installs python3 on a dib7 base image to allow dib"},{"line_number":10,"context_line":"to build images on older branches and distros like CentOS7/RHEL7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"bf51134e_0ccb1c4a","line":7,"range":{"start_line":7,"start_character":14,"end_line":7,"end_character":27},"in_reply_to":"bf51134e_f67dd56e","updated":"2020-07-06 12:12:12.000000000","message":"ack","commit_id":"e465bde14180973d05de3037f726cce7be249047"}],"diskimage_builder/elements/dib-python/pre-install.d/01-dib-python":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"19ca5e25a9352a6d0357f6a38a8991b7b25116d2","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    # a good idea, abstracting the python binary for system scripts!"},{"line_number":12,"context_line":"    # :)  Use it for dib-python."},{"line_number":13,"context_line":"    python_path\u003d/usr/libexec/platform-python"},{"line_number":14,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003d\u003d 7 ]]; then"},{"line_number":15,"context_line":"    yum install -y python3"},{"line_number":16,"context_line":"    pip3 install pyyaml"},{"line_number":17,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_997325b2","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":71},"updated":"2020-07-01 05:03:01.000000000","message":"this should probably have an \u0026\u0026 DIB_PYTHON_VERSION\u003d\u003d3 with it too.","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"2267e302b75ce19aeddce2e5815ffac5307dd6f0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    # a good idea, abstracting the python binary for system scripts!"},{"line_number":12,"context_line":"    # :)  Use it for dib-python."},{"line_number":13,"context_line":"    python_path\u003d/usr/libexec/platform-python"},{"line_number":14,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003d\u003d 7 ]]; then"},{"line_number":15,"context_line":"    yum install -y python3"},{"line_number":16,"context_line":"    pip3 install pyyaml"},{"line_number":17,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_f2d16f31","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":71},"in_reply_to":"bf51134e_997325b2","updated":"2020-07-03 14:44:18.000000000","message":"I can include that condition if it makes more clear. Even though py3 is the only option w/ py2 support dropped.","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"425d6ee720d09a06d42095d1a0ac2eac47c3278a","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    python_path\u003d/usr/libexec/platform-python"},{"line_number":14,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003d\u003d 7 ]]; then"},{"line_number":15,"context_line":"    yum install -y python3"},{"line_number":16,"context_line":"    pip3 install pyyaml"},{"line_number":17,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":18,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":19,"context_line":"    apt-get install -y python3"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_04e07025","line":16,"updated":"2020-06-29 06:40:26.000000000","message":"we should not install this via pip outside packages.  pip might not even be on the host.","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"58ab94bff5680050580c7f75c168a534ee759534","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    python_path\u003d/usr/libexec/platform-python"},{"line_number":14,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003d\u003d 7 ]]; then"},{"line_number":15,"context_line":"    yum install -y python3"},{"line_number":16,"context_line":"    pip3 install pyyaml"},{"line_number":17,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":18,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":19,"context_line":"    apt-get install -y python3"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_85b0f7de","line":16,"in_reply_to":"bf51134e_04e07025","updated":"2020-06-29 13:58:04.000000000","message":"context: we still build images for older branches. Now that python2 support has been dropped, and python3 is available on centos7.8.\n\nthe problem: python3-pyyaml is not available as package for centos7, so I am proposing installing it from pip. And python3 is available on the system, but not in chroot base image, so I need to install it, even though w/ DIB_PYTHON_VERSION\u003d3.\n\n\nThe original failure is:\n2020-06-10 21:01:56.693 | dib-run-parts Running /tmp/in_target.d/pre-install.d/01-dib-python\n2020-06-10 21:01:56.696 | + set -eu\n2020-06-10 21:01:56.697 | + set -o pipefail\n2020-06-10 21:01:56.697 | + [[ centos7 \u003d~ (centos|rhel) ]]\n2020-06-10 21:01:56.697 | + [[ 7 \u003d\u003d 8 ]]\n2020-06-10 21:01:56.697 | + [[ centos7 \u003d~ (debian) ]]\n2020-06-10 21:01:56.697 | ++ command -v python3\n2020-06-10 21:01:56.698 | + python_path\u003d\n\nhttps://225f899c106a113608d1-616db0a32cd5a8455c371f4b45f3133e.ssl.cf2.rackcdn.com/733659/13/check/tripleo-buildimage-overcloud-full-centos-7-train/e5d64f4/build.log\n\n\nPerhaps I could set it somewhere else ?","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"425d6ee720d09a06d42095d1a0ac2eac47c3278a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003d\u003d 7 ]]; then"},{"line_number":15,"context_line":"    yum install -y python3"},{"line_number":16,"context_line":"    pip3 install pyyaml"},{"line_number":17,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":18,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":19,"context_line":"    apt-get install -y python3"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_24be54fe","line":17,"updated":"2020-06-29 06:40:26.000000000","message":"centos7/rhel7 is going to have DIB_PYTHON_VERSION set to 2?  so i\u0027m not sure what this is trying to do installing python3.","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"58ab94bff5680050580c7f75c168a534ee759534","unresolved":false,"context_lines":[{"line_number":14,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003d\u003d 7 ]]; then"},{"line_number":15,"context_line":"    yum install -y python3"},{"line_number":16,"context_line":"    pip3 install pyyaml"},{"line_number":17,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":18,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":19,"context_line":"    apt-get install -y python3"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_457b7ff3","line":17,"in_reply_to":"bf51134e_24be54fe","updated":"2020-06-29 13:58:04.000000000","message":"We are currently pinning diskimage-builder to the commit before python2 drop, as a temporary workaround. But I\u0027d like to continue building images on older branches w/ python3 and use the latest dib code.","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"425d6ee720d09a06d42095d1a0ac2eac47c3278a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    apt-get install -y python3"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"else"},{"line_number":22,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":23,"context_line":"fi"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"if [ -z \"$python_path\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_44d7084c","line":22,"updated":"2020-06-29 06:40:26.000000000","message":"why are you adding this echo?  won\u0027t it fool the check below?","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"58ab94bff5680050580c7f75c168a534ee759534","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    apt-get install -y python3"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"else"},{"line_number":22,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":23,"context_line":"fi"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"if [ -z \"$python_path\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"bf51134e_e5dd73dd","line":22,"in_reply_to":"bf51134e_44d7084c","updated":"2020-06-29 13:58:04.000000000","message":"the error below is never reached, because when command -v returns 1, the script exits and the error is not handled.\nI just added || echo \"\" to ignore the return code from command -v and handle the error.","commit_id":"a4ec179734dc76083c48e8aee5944e69c081743b"},{"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":"4dd4f4d41c42cf9588a60d7fa35780e6c7b831e8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    # CentOS7 cloud base image does not have python3"},{"line_number":17,"context_line":"    yum install -y python3 python3-pip"},{"line_number":18,"context_line":"    # Python3-pyyaml is not available as a package, install w/ pip"},{"line_number":19,"context_line":"    pip3 install pyyaml"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_16d7a951","line":19,"range":{"start_line":19,"start_character":4,"end_line":19,"end_character":8},"updated":"2020-07-06 10:14:50.000000000","message":"python3 -m pip","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"09521de09dc9a9dac4bb37a069f9560281ce3964","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    # CentOS7 cloud base image does not have python3"},{"line_number":17,"context_line":"    yum install -y python3 python3-pip"},{"line_number":18,"context_line":"    # Python3-pyyaml is not available as a package, install w/ pip"},{"line_number":19,"context_line":"    pip3 install pyyaml"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_6cda9817","line":19,"range":{"start_line":19,"start_character":4,"end_line":19,"end_character":8},"in_reply_to":"bf51134e_16d7a951","updated":"2020-07-06 12:12:12.000000000","message":"yup","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"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":"4dd4f4d41c42cf9588a60d7fa35780e6c7b831e8","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    yum install -y python3 python3-pip"},{"line_number":18,"context_line":"    # Python3-pyyaml is not available as a package, install w/ pip"},{"line_number":19,"context_line":"    pip3 install pyyaml"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"},{"line_number":23,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_d6f531ef","line":20,"range":{"start_line":20,"start_character":60,"end_line":20,"end_character":68},"updated":"2020-07-06 10:14:50.000000000","message":"true","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"09521de09dc9a9dac4bb37a069f9560281ce3964","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    yum install -y python3 python3-pip"},{"line_number":18,"context_line":"    # Python3-pyyaml is not available as a package, install w/ pip"},{"line_number":19,"context_line":"    pip3 install pyyaml"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"},{"line_number":23,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_4cd514e5","line":20,"range":{"start_line":20,"start_character":60,"end_line":20,"end_character":68},"in_reply_to":"bf51134e_d6f531ef","updated":"2020-07-06 12:12:12.000000000","message":"+1","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"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":"4dd4f4d41c42cf9588a60d7fa35780e6c7b831e8","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"},{"line_number":23,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":24,"context_line":"else"},{"line_number":25,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":26,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_b6e87d16","line":23,"range":{"start_line":23,"start_character":60,"end_line":23,"end_character":68},"updated":"2020-07-06 10:14:50.000000000","message":"true","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"09521de09dc9a9dac4bb37a069f9560281ce3964","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"},{"line_number":23,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":24,"context_line":"else"},{"line_number":25,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":26,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_ac05d07c","line":23,"range":{"start_line":23,"start_character":60,"end_line":23,"end_character":68},"in_reply_to":"bf51134e_b6e87d16","updated":"2020-07-06 12:12:12.000000000","message":"+1","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"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":"4dd4f4d41c42cf9588a60d7fa35780e6c7b831e8","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    apt-get install -y python3"},{"line_number":23,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":24,"context_line":"else"},{"line_number":25,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":26,"context_line":"fi"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"if [ -z \"$python_path\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_96ebb910","line":25,"range":{"start_line":25,"start_character":60,"end_line":25,"end_character":68},"updated":"2020-07-06 10:14:50.000000000","message":"true","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"09521de09dc9a9dac4bb37a069f9560281ce3964","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    apt-get install -y python3"},{"line_number":23,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":24,"context_line":"else"},{"line_number":25,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || echo \"\")"},{"line_number":26,"context_line":"fi"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"if [ -z \"$python_path\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"bf51134e_8c08cc91","line":25,"range":{"start_line":25,"start_character":60,"end_line":25,"end_character":68},"in_reply_to":"bf51134e_96ebb910","updated":"2020-07-06 12:12:12.000000000","message":"+1","commit_id":"e465bde14180973d05de3037f726cce7be249047"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e1b76cbf34b13d87454b0acb0a34808523eb14f1","unresolved":false,"context_lines":[{"line_number":16,"context_line":"    # CentOS7/RHEL7 cloud base image does not have python3"},{"line_number":17,"context_line":"    yum install -y python3 python3-pip"},{"line_number":18,"context_line":"    # Python3-pyyaml is not available as a package, install w/ pip"},{"line_number":19,"context_line":"    python3 -m pip install pyyaml"},{"line_number":20,"context_line":"    python_path\u003d$(command -v python${DIB_PYTHON_VERSION} || true)"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (debian) \u0026\u0026 ${DIB_PYTHON_VERSION} \u003d\u003d 3 ]]; then"},{"line_number":22,"context_line":"    apt-get install -y python3"}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"bf51134e_2cbe2bd4","line":19,"updated":"2020-07-17 06:28:04.000000000","message":"OK, so probably some of the package map type stuff is using pyyaml inside the chroot i guess.  I really don\u0027t like pip installing this -- we just went to *a lot* of effort to have things not pip installed in the base images.\n\nWhere is it that fails without pyyaml?\n\nWhat I\u0027m thinking is that if that element has a element-deps on the epel element, then we can do a somewhat hacky \"yum --enablerepo\u003depel install pyyaml3\" in there and not have to have pip installed packages.","commit_id":"6c05cb288e88d0e1adefc4ab1c092d81195cee02"}]}