)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"ae9c8d9d4b211a325a843d5e6ff557e7cae3a3bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"19213ca4_aef7589b","updated":"2023-02-07 06:19:15.000000000","message":"the parent changes have merged and builds over the last few days have been OK, and no runtime problems reported.  Merging this now","commit_id":"75a6a641b11a064f85e5620f9ad6bc35106139ca"}],"nodepool/elements/infra-package-needs/install.d/10-pip-packages":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b676fbc0556bf6ed9231a1fe9df3e2a777017691","unresolved":true,"context_lines":[{"line_number":25,"context_line":"# these, skip this."},{"line_number":26,"context_line":"if [ -z ${DIB_PYTHON_PIP+x} ]; then"},{"line_number":27,"context_line":"    exit 0"},{"line_number":28,"context_line":"fi"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"packages\u003d\u0027tox\u0027"},{"line_number":31,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"d3c6696f_5942e98d","side":"PARENT","line":28,"updated":"2023-02-02 20:57:20.000000000","message":"From a jammy image build log:\n\n 2023-02-02 01:49:57.848 | dib-run-parts Running /tmp/in_target.d/install.d/10-pip-packages\n 2023-02-02 01:49:57.855 | + set -e\n 2023-02-02 01:49:57.855 | + \u0027[\u0027 -z \u0027]\u0027\n 2023-02-02 01:49:57.855 | + exit 0\n 2023-02-02 01:49:57.857 | dib-run-parts 10-pip-packages completed\n \nThis file can safely be removed.","commit_id":"5a6b14875f4ed20d422ba208c5134f18d9b796da"}],"nodepool/elements/infra-package-needs/install.d/40-install-tox":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b676fbc0556bf6ed9231a1fe9df3e2a777017691","unresolved":true,"context_lines":[{"line_number":30,"context_line":"else"},{"line_number":31,"context_line":"    /usr/tox-env/bin/pip install --upgrade pip"},{"line_number":32,"context_line":"fi"},{"line_number":33,"context_line":"/usr/tox-env/bin/pip install tox"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"a58a6feb_1d261e0d","side":"PARENT","line":33,"updated":"2023-02-02 20:57:20.000000000","message":"We do appear to actually install this tox-env and put tox in it looking at a jammy image build. I suspect that we were thinking we could set the ensure-tox var to find tox at this path and avoid some job runtime overhead. Considering we haven\u0027t been doing that removing this file isn\u0027t a regression.\n\nWe might want to consider doing that to speed up jobs. However, with all the tox upgrade madness I kinda like managing tox at runtime as it is more flexible.","commit_id":"5a6b14875f4ed20d422ba208c5134f18d9b796da"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"ae9c8d9d4b211a325a843d5e6ff557e7cae3a3bc","unresolved":false,"context_lines":[{"line_number":30,"context_line":"else"},{"line_number":31,"context_line":"    /usr/tox-env/bin/pip install --upgrade pip"},{"line_number":32,"context_line":"fi"},{"line_number":33,"context_line":"/usr/tox-env/bin/pip install tox"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"8d23b486_5a73669e","side":"PARENT","line":33,"in_reply_to":"a58a6feb_1d261e0d","updated":"2023-02-07 06:19:15.000000000","message":"Yeah, I think this was the idea.  But I can not see that a global tox install is going to be good for anything now, given version skews, jobs not even using tox, etc.","commit_id":"5a6b14875f4ed20d422ba208c5134f18d9b796da"}]}
