)]}'
{"diskimage_builder/elements/yum-minimal/package-installs.yaml":[{"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":"f68d6af835fbe01bf2d924137306499564f444b0","unresolved":false,"context_lines":[{"line_number":18,"context_line":"# mirrors the default packages installed in upstream cloud images"},{"line_number":19,"context_line":"# to facilitate a basic network."},{"line_number":20,"context_line":"NetworkManager:"},{"line_number":21,"context_line":"dhcp-client:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"bf51134e_672d99a7","line":21,"updated":"2020-07-22 10:57:09.000000000","message":"I do wonder what changed here","commit_id":"808ddf486112cb8f3f2e8393de79f9ca97c00111"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"28938875d17575883ede71e8af6dbe1fcb5e7c9f","unresolved":false,"context_lines":[{"line_number":18,"context_line":"# mirrors the default packages installed in upstream cloud images"},{"line_number":19,"context_line":"# to facilitate a basic network."},{"line_number":20,"context_line":"NetworkManager:"},{"line_number":21,"context_line":"dhcp-client:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"bf51134e_a6c4c842","line":21,"in_reply_to":"bf51134e_672d99a7","updated":"2020-07-22 16:31:09.000000000","message":"spare line","commit_id":"808ddf486112cb8f3f2e8393de79f9ca97c00111"}],"diskimage_builder/elements/yum/pre-install.d/00-1-centos-python3":[{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"4a9aff6447ec81f9b095e72507810e1544a27248","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    yum install -y python3 epel-release"},{"line_number":19,"context_line":"    yum install -y python36-PyYAML"},{"line_number":20,"context_line":"    yum remove -y epel-release"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003e 7 ]]; then"},{"line_number":22,"context_line":"    # For 8 and above ensure the \"user\" python3 package is installed"},{"line_number":23,"context_line":"    # so we have /usr/bin/python3 and pyyaml."},{"line_number":24,"context_line":"    dnf install -y python3 python3-pyyaml"},{"line_number":25,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"9f560f44_7e839357","line":25,"range":{"start_line":21,"start_character":0,"end_line":25,"end_character":2},"updated":"2020-07-27 12:54:22.000000000","message":"Until we get rid of dib-python I think this is not ok. Can we leave this out of this patch and do this atomically with the dib-python removal?\n\nIn dib-python we have:\npython_path\u003d/usr/libexec/platform-python\n\nhttps://github.com/openstack/diskimage-builder/blob/master/diskimage_builder/elements/dib-python/pre-install.d/01-dib-python#L13\n\nThis is probably the reason why my dib-centos8 job failed at https://review.opendev.org/#/c/733659\n\n2020-07-24 18:23:28.868 | dib-run-parts Running /tmp/in_target.d/install.d/74-openvswitch\n2020-07-24 18:23:28.870 | + \u0027[\u0027 systemd \u003d\u003d systemd \u0027]\u0027\n2020-07-24 18:23:28.870 | + os-svc-enable -n openvswitch-switch\n2020-07-24 18:23:28.874 | /usr/bin/env: \u0027python\u0027: No such file or directory\n2020-07-24 18:23:28.879 | Failed to enable unit, unit .service.service does not exist.\n\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_cf6/733659/49/check/tripleo-buildimage-overcloud-full-centos-8/cf63e23/build.log","commit_id":"426b1089138999a0c1ef004a9b507c71796d78fd"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"f86837971ce18ec89bd0a96deb6fc9a0d9fe424b","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    yum install -y python3 epel-release"},{"line_number":19,"context_line":"    yum install -y python36-PyYAML"},{"line_number":20,"context_line":"    yum remove -y epel-release"},{"line_number":21,"context_line":"elif [[ ${DISTRO_NAME} \u003d~ (centos|rhel) \u0026\u0026 ${DIB_RELEASE} \u003e 7 ]]; then"},{"line_number":22,"context_line":"    # For 8 and above ensure the \"user\" python3 package is installed"},{"line_number":23,"context_line":"    # so we have /usr/bin/python3 and pyyaml."},{"line_number":24,"context_line":"    dnf install -y python3 python3-pyyaml"},{"line_number":25,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"9f560f44_8985f75c","line":25,"range":{"start_line":21,"start_character":0,"end_line":25,"end_character":2},"in_reply_to":"9f560f44_7e839357","updated":"2020-08-05 14:31:04.000000000","message":"Disregard comment above ^ ...\nFor the record, it\u0027s working fine for tripleo image builds: \nhttps://review.opendev.org/#/c/733659","commit_id":"426b1089138999a0c1ef004a9b507c71796d78fd"}],"diskimage_builder/elements/yum/pre-install.d/03-centos7-python3":[{"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":"f68d6af835fbe01bf2d924137306499564f444b0","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    # a way to say \"install this from EPEL\".  So we hack in an install"},{"line_number":19,"context_line":"    # of it here from EPEL."},{"line_number":20,"context_line":"    yum install -y python3 epel-release"},{"line_number":21,"context_line":"    yum install -y --enablerepo\u003depel python36-PyYAML"},{"line_number":22,"context_line":"    yum remove -y epel-release"},{"line_number":23,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"bf51134e_e7124962","line":21,"range":{"start_line":21,"start_character":19,"end_line":21,"end_character":36},"updated":"2020-07-22 10:57:09.000000000","message":"it\u0027s by default","commit_id":"808ddf486112cb8f3f2e8393de79f9ca97c00111"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"28938875d17575883ede71e8af6dbe1fcb5e7c9f","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    # a way to say \"install this from EPEL\".  So we hack in an install"},{"line_number":19,"context_line":"    # of it here from EPEL."},{"line_number":20,"context_line":"    yum install -y python3 epel-release"},{"line_number":21,"context_line":"    yum install -y --enablerepo\u003depel python36-PyYAML"},{"line_number":22,"context_line":"    yum remove -y epel-release"},{"line_number":23,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"bf51134e_c6ce1c5a","line":21,"range":{"start_line":21,"start_character":19,"end_line":21,"end_character":36},"in_reply_to":"bf51134e_e7124962","updated":"2020-07-22 16:31:09.000000000","message":"agreed","commit_id":"808ddf486112cb8f3f2e8393de79f9ca97c00111"}]}
