)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"2f16c332cf66b7576652fec69c962112134e91d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"05558a1d_22033ce6","updated":"2022-02-01 16:02:45.000000000","message":"Ew, it doesn\u0027t. ;)","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"25e294ec89bc37d70d4eeb53564e943572dd8deb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"199e3ef3_84faf166","updated":"2022-02-01 16:02:11.000000000","message":"Looks good, let\u0027s see if CI likes it.","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"81465bc1b56d7a9b79b19c87d7847ece7d6b375a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b778ea38_f8fbb551","updated":"2022-02-01 16:06:52.000000000","message":"kuryr-daemon:\n /usr/bin/cni_ds_init: /usr/local/bin/kuryr-daemon: /usr/bin/python3: bad interpreter: No such file or directory\n /usr/bin/cni_ds_init: line 22: /usr/local/bin/kuryr-daemon: Success\n\nkuryr-controller:\n standard_init_linux.go:228: exec user process caused: no such file or directory\n standard_init_linux.go:228: exec user process caused: no such file or directory\n","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"a0c587218953d50a24d169a81cf1f872c2afbbb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"92d98811_9e3c543d","updated":"2022-02-02 08:40:27.000000000","message":"Looks okay, let\u0027s wait for Zuul.","commit_id":"c3e863b993bc247f9dfe87b859b7142311313bae"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"7313757cdaac437652cdf5fd3f269a462dbfc878","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ac86f22a_ec990707","updated":"2022-02-03 14:12:56.000000000","message":"This needs a backport to stable/xena to unblock gate there.","commit_id":"c62488710389040313dfa287c7c746a327a1759c"}],"cni.Dockerfile":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"162e29b0102bb7a69f706de10a639858b82f09c3","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"COPY . /opt/kuryr-kubernetes"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# Issuing \u0027yum -y history undo last\u0027 command will complain about removing "},{"line_number":28,"context_line":"# sudo/systemd/dnf packages"},{"line_number":29,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"},{"line_number":30,"context_line":"    \u0026\u0026 python3 -m pip --no-cache-dir install -c $UPPER_CONSTRAINTS_FILE /opt/kuryr-kubernetes \\"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"aae24a90_15480aa0","line":27,"range":{"start_line":27,"start_character":73,"end_line":27,"end_character":74},"updated":"2022-02-01 13:12:03.000000000","message":"Trailing whitespace","commit_id":"91274764712c217e9df4e6ec0f4abc81c284f4b9"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"54faf70d738c8446c81d8b47e59b84f8e617bb1f","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"COPY . /opt/kuryr-kubernetes"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# Issuing \u0027yum -y history undo last\u0027 command will complain about removing "},{"line_number":28,"context_line":"# sudo/systemd/dnf packages"},{"line_number":29,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"},{"line_number":30,"context_line":"    \u0026\u0026 python3 -m pip --no-cache-dir install -c $UPPER_CONSTRAINTS_FILE /opt/kuryr-kubernetes \\"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"780d2b93_541c896e","line":27,"range":{"start_line":27,"start_character":73,"end_line":27,"end_character":74},"in_reply_to":"aae24a90_15480aa0","updated":"2022-02-01 14:50:45.000000000","message":"Done","commit_id":"91274764712c217e9df4e6ec0f4abc81c284f4b9"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"162e29b0102bb7a69f706de10a639858b82f09c3","unresolved":true,"context_lines":[{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 yum remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"d3f2e95f_6fe6e540","line":36,"range":{"start_line":36,"start_character":7,"end_line":36,"end_character":10},"updated":"2022-02-01 13:12:03.000000000","message":"Why do we start mixing dnf here now?","commit_id":"91274764712c217e9df4e6ec0f4abc81c284f4b9"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"54faf70d738c8446c81d8b47e59b84f8e617bb1f","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 yum remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"3d4b36f6_dfd55013","line":36,"range":{"start_line":36,"start_character":7,"end_line":36,"end_character":10},"in_reply_to":"ce33a727_a1c36873","updated":"2022-02-01 14:50:45.000000000","message":"Done","commit_id":"91274764712c217e9df4e6ec0f4abc81c284f4b9"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"480071d5c20c2b256ea6c12711428fdc201772d8","unresolved":true,"context_lines":[{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 yum remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"ce33a727_a1c36873","line":36,"range":{"start_line":36,"start_character":7,"end_line":36,"end_character":10},"in_reply_to":"d3f2e95f_6fe6e540","updated":"2022-02-01 13:49:40.000000000","message":"actually, I\u0027ll change every yum occurrence to dnf everywhere, since yum is long time deprecated.","commit_id":"91274764712c217e9df4e6ec0f4abc81c284f4b9"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"ad29037dd9de4f468c5885ebc0a00881d428d702","unresolved":true,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"RUN GO111MODULE\u003dauto go build -o /go/bin/kuryr-cni ./kuryr_cni/pkg/*"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"FROM quay.io/centos/centos:stream8"},{"line_number":9,"context_line":"LABEL authors\u003d\"Antoni Segura Puimedon\u003ctoni@kuryr.org\u003e, Michał Dulko\u003cmdulko@redhat.com\u003e\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"d9a32bfe_648c1eb0","line":8,"updated":"2022-02-02 01:54:49.000000000","message":"did you hit issues with centos9?","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"1eaeeb3bcd8a0a98c8c695e4bfe5af8102983982","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"RUN GO111MODULE\u003dauto go build -o /go/bin/kuryr-cni ./kuryr_cni/pkg/*"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"FROM quay.io/centos/centos:stream8"},{"line_number":9,"context_line":"LABEL authors\u003d\"Antoni Segura Puimedon\u003ctoni@kuryr.org\u003e, Michał Dulko\u003cmdulko@redhat.com\u003e\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"a0cacff8_e0872588","line":8,"in_reply_to":"2acd5b98_fc3c65d7","updated":"2022-02-02 10:27:22.000000000","message":"Ack","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"a0c587218953d50a24d169a81cf1f872c2afbbb5","unresolved":true,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"RUN GO111MODULE\u003dauto go build -o /go/bin/kuryr-cni ./kuryr_cni/pkg/*"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"FROM quay.io/centos/centos:stream8"},{"line_number":9,"context_line":"LABEL authors\u003d\"Antoni Segura Puimedon\u003ctoni@kuryr.org\u003e, Michał Dulko\u003cmdulko@redhat.com\u003e\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"2acd5b98_fc3c65d7","line":8,"in_reply_to":"b84605b7_f8a103f2","updated":"2022-02-02 08:40:27.000000000","message":"Yeah, this seems expected, there was an RDO delay with 7-\u003e8 transition too.","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"fbb271b0ccff36b1cb757901e1e1668aac0f592f","unresolved":true,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"RUN GO111MODULE\u003dauto go build -o /go/bin/kuryr-cni ./kuryr_cni/pkg/*"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"FROM quay.io/centos/centos:stream8"},{"line_number":9,"context_line":"LABEL authors\u003d\"Antoni Segura Puimedon\u003ctoni@kuryr.org\u003e, Michał Dulko\u003cmdulko@redhat.com\u003e\""},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"b84605b7_f8a103f2","line":8,"in_reply_to":"d9a32bfe_648c1eb0","updated":"2022-02-02 06:00:54.000000000","message":"Yes I did. There were incompatibilities with RDO repository, which seems to be built against RHEL8 packages. while stream9 have too new packages.","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"901a7f3f0440e8d9f0c1680abf90cc905fc1c049","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/cni_ds_init /usr/bin/cni_ds_init \\"},{"line_number":32,"context_line":"    \u0026\u0026 mkdir -p /etc/kuryr-cni \\"},{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 dnf remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"fe27e2f8_da92097f","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":27},"updated":"2022-02-01 16:23:17.000000000","message":"Looks like one of these deletes Python. :/","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"a0c587218953d50a24d169a81cf1f872c2afbbb5","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/cni_ds_init /usr/bin/cni_ds_init \\"},{"line_number":32,"context_line":"    \u0026\u0026 mkdir -p /etc/kuryr-cni \\"},{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 dnf remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"83b5cde1_48596c90","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":27},"in_reply_to":"61f899b3_55f40e57","updated":"2022-02-02 08:40:27.000000000","message":"Well, I certainly don\u0027t have a strong argument against switching to Ubuntu.","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"1eaeeb3bcd8a0a98c8c695e4bfe5af8102983982","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/cni_ds_init /usr/bin/cni_ds_init \\"},{"line_number":32,"context_line":"    \u0026\u0026 mkdir -p /etc/kuryr-cni \\"},{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 dnf remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"855722c0_e53cd9f9","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":27},"in_reply_to":"83b5cde1_48596c90","updated":"2022-02-02 10:27:22.000000000","message":"All our OpenStack CI are based on ubuntu, so it\u0027s additional headache for us if something go south with centos like this case. I\u0027d just use single distribution as a common denominator to reduce things like that.","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"b13514b412abde462f75969cd81424a55698d47b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/cni_ds_init /usr/bin/cni_ds_init \\"},{"line_number":32,"context_line":"    \u0026\u0026 mkdir -p /etc/kuryr-cni \\"},{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 dnf remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"a687c92e_87aa889d","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":27},"in_reply_to":"855722c0_e53cd9f9","updated":"2022-02-02 15:03:17.000000000","message":"Ack","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"5d48fedbe0b7de8952fb47e7fd8693f558f0547f","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/cni_ds_init /usr/bin/cni_ds_init \\"},{"line_number":32,"context_line":"    \u0026\u0026 mkdir -p /etc/kuryr-cni \\"},{"line_number":33,"context_line":"    \u0026\u0026 cp /opt/kuryr-kubernetes/etc/cni/net.d/* /etc/kuryr-cni \\"},{"line_number":34,"context_line":"    \u0026\u0026 dnf remove -y python3-pip openvswitch iproute pciutils gcc gcc-c++ \\"},{"line_number":35,"context_line":"        python3-devel git \\"},{"line_number":36,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":37,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":38,"context_line":"    \u0026\u0026 mkdir ${OSLO_LOCK_PATH}"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"61f899b3_55f40e57","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":27},"in_reply_to":"fe27e2f8_da92097f","updated":"2022-02-02 05:57:34.000000000","message":"Can we just use ubuntu as container image?","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"37d215395e43e95dd61d09c146a9baaa09c7f248","unresolved":true,"context_lines":[{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""},{"line_number":12,"context_line":"ARG OSLO_LOCK_PATH\u003d/var/kuryr-lock"},{"line_number":13,"context_line":"ARG RDO_REPO\u003dhttps://repos.fedorapeople.org/repos/openstack/openstack-xena/rdo-release-xena-1.el8.noarch.rpm"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN dnf upgrade -y \u0026\u0026 dnf install -y epel-release $RDO_REPO"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# NOTE(gryf): hack to make package manager to cooperate - it might be a"},{"line_number":18,"context_line":"# subject to change in the future, either when yum/dnf starts to respect"},{"line_number":19,"context_line":"# yum.conf variables, or mirror location would change."},{"line_number":20,"context_line":"RUN sed -e \u0027s/$releasever/8-stream/\u0027 -i /etc/yum.repos.d/messaging.repo \\"},{"line_number":21,"context_line":"    \u0026\u0026 sed -e \u0027s/$basearch/x86_64/\u0027 -i /etc/yum.repos.d/messaging.repo"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"RUN dnf install -y --setopt\u003dtsflags\u003dnodocs python3-pip openvswitch sudo iproute libstdc++ pciutils kmod-libs \\"},{"line_number":24,"context_line":"    \u0026\u0026 dnf install -y --setopt\u003dtsflags\u003dnodocs gcc gcc-c++ python3-devel git"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"COPY . /opt/kuryr-kubernetes"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"}],"source_content_type":"text/x-dockerfile","patch_set":5,"id":"584afa37_60af9bd3","line":25,"range":{"start_line":14,"start_character":0,"end_line":25,"end_character":0},"updated":"2022-02-02 13:41:41.000000000","message":"Splitting this into 3 RUN statements will create 2 more layers in the container image. I\u0027m not sure it doesn\u0027t carry any consequences.","commit_id":"3b2b78bbc9b8a99465ef6c9745f47491d199f9b4"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"b13514b412abde462f75969cd81424a55698d47b","unresolved":false,"context_lines":[{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""},{"line_number":12,"context_line":"ARG OSLO_LOCK_PATH\u003d/var/kuryr-lock"},{"line_number":13,"context_line":"ARG RDO_REPO\u003dhttps://repos.fedorapeople.org/repos/openstack/openstack-xena/rdo-release-xena-1.el8.noarch.rpm"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN dnf upgrade -y \u0026\u0026 dnf install -y epel-release $RDO_REPO"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# NOTE(gryf): hack to make package manager to cooperate - it might be a"},{"line_number":18,"context_line":"# subject to change in the future, either when yum/dnf starts to respect"},{"line_number":19,"context_line":"# yum.conf variables, or mirror location would change."},{"line_number":20,"context_line":"RUN sed -e \u0027s/$releasever/8-stream/\u0027 -i /etc/yum.repos.d/messaging.repo \\"},{"line_number":21,"context_line":"    \u0026\u0026 sed -e \u0027s/$basearch/x86_64/\u0027 -i /etc/yum.repos.d/messaging.repo"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"RUN dnf install -y --setopt\u003dtsflags\u003dnodocs python3-pip openvswitch sudo iproute libstdc++ pciutils kmod-libs \\"},{"line_number":24,"context_line":"    \u0026\u0026 dnf install -y --setopt\u003dtsflags\u003dnodocs gcc gcc-c++ python3-devel git"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"COPY . /opt/kuryr-kubernetes"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"}],"source_content_type":"text/x-dockerfile","patch_set":5,"id":"4fa6b95c_e200dad4","line":25,"range":{"start_line":14,"start_character":0,"end_line":25,"end_character":0},"in_reply_to":"39c252d8_16ea59dc","updated":"2022-02-02 15:03:17.000000000","message":"Ack","commit_id":"3b2b78bbc9b8a99465ef6c9745f47491d199f9b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"d7fad2ad56d1e20342d84c28014c7cd06645522a","unresolved":true,"context_lines":[{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""},{"line_number":12,"context_line":"ARG OSLO_LOCK_PATH\u003d/var/kuryr-lock"},{"line_number":13,"context_line":"ARG RDO_REPO\u003dhttps://repos.fedorapeople.org/repos/openstack/openstack-xena/rdo-release-xena-1.el8.noarch.rpm"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN dnf upgrade -y \u0026\u0026 dnf install -y epel-release $RDO_REPO"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# NOTE(gryf): hack to make package manager to cooperate - it might be a"},{"line_number":18,"context_line":"# subject to change in the future, either when yum/dnf starts to respect"},{"line_number":19,"context_line":"# yum.conf variables, or mirror location would change."},{"line_number":20,"context_line":"RUN sed -e \u0027s/$releasever/8-stream/\u0027 -i /etc/yum.repos.d/messaging.repo \\"},{"line_number":21,"context_line":"    \u0026\u0026 sed -e \u0027s/$basearch/x86_64/\u0027 -i /etc/yum.repos.d/messaging.repo"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"RUN dnf install -y --setopt\u003dtsflags\u003dnodocs python3-pip openvswitch sudo iproute libstdc++ pciutils kmod-libs \\"},{"line_number":24,"context_line":"    \u0026\u0026 dnf install -y --setopt\u003dtsflags\u003dnodocs gcc gcc-c++ python3-devel git"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"COPY . /opt/kuryr-kubernetes"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"}],"source_content_type":"text/x-dockerfile","patch_set":5,"id":"7d74f098_44fa47f2","line":25,"range":{"start_line":14,"start_character":0,"end_line":25,"end_character":0},"in_reply_to":"584afa37_60af9bd3","updated":"2022-02-02 14:52:53.000000000","message":"I can merge them to see if the container image size will change.","commit_id":"3b2b78bbc9b8a99465ef6c9745f47491d199f9b4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"883953c8e1dc9ddbb5501937f6b8757f5b00dc89","unresolved":true,"context_lines":[{"line_number":11,"context_line":"ARG UPPER_CONSTRAINTS_FILE\u003d\"https://releases.openstack.org/constraints/upper/master\""},{"line_number":12,"context_line":"ARG OSLO_LOCK_PATH\u003d/var/kuryr-lock"},{"line_number":13,"context_line":"ARG RDO_REPO\u003dhttps://repos.fedorapeople.org/repos/openstack/openstack-xena/rdo-release-xena-1.el8.noarch.rpm"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"RUN dnf upgrade -y \u0026\u0026 dnf install -y epel-release $RDO_REPO"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# NOTE(gryf): hack to make package manager to cooperate - it might be a"},{"line_number":18,"context_line":"# subject to change in the future, either when yum/dnf starts to respect"},{"line_number":19,"context_line":"# yum.conf variables, or mirror location would change."},{"line_number":20,"context_line":"RUN sed -e \u0027s/$releasever/8-stream/\u0027 -i /etc/yum.repos.d/messaging.repo \\"},{"line_number":21,"context_line":"    \u0026\u0026 sed -e \u0027s/$basearch/x86_64/\u0027 -i /etc/yum.repos.d/messaging.repo"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"RUN dnf install -y --setopt\u003dtsflags\u003dnodocs python3-pip openvswitch sudo iproute libstdc++ pciutils kmod-libs \\"},{"line_number":24,"context_line":"    \u0026\u0026 dnf install -y --setopt\u003dtsflags\u003dnodocs gcc gcc-c++ python3-devel git"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"COPY . /opt/kuryr-kubernetes"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"}],"source_content_type":"text/x-dockerfile","patch_set":5,"id":"39c252d8_16ea59dc","line":25,"range":{"start_line":14,"start_character":0,"end_line":25,"end_character":0},"in_reply_to":"7d74f098_44fa47f2","updated":"2022-02-02 14:55:33.000000000","message":"Yup, it is smaller. Let\u0027s amend it.","commit_id":"3b2b78bbc9b8a99465ef6c9745f47491d199f9b4"}],"controller.Dockerfile":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"901a7f3f0440e8d9f0c1680abf90cc905fc1c049","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"RUN pip3 --no-cache-dir install -U pip \\"},{"line_number":14,"context_line":"    \u0026\u0026 python3 -m pip install -c $UPPER_CONSTRAINTS_FILE --no-cache-dir /opt/kuryr-kubernetes \\"},{"line_number":15,"context_line":"    \u0026\u0026 dnf remove -y python3-pip gcc gcc-c++ python3-devel git \\"},{"line_number":16,"context_line":"    \u0026\u0026 dnf clean all \\"},{"line_number":17,"context_line":"    \u0026\u0026 rm -rf /opt/kuryr-kubernetes \\"},{"line_number":18,"context_line":"    \u0026\u0026 groupadd -r kuryr -g 711 \\"}],"source_content_type":"text/x-dockerfile","patch_set":2,"id":"4b085025_5c1cd474","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":64},"updated":"2022-02-01 16:23:17.000000000","message":"Same here, one of these seems to delete Python.","commit_id":"6f5ed6301ab3491e39f81a71bc36d2cc2b69fb2e"}]}