)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"ec0864f6eb34de78aadafd25962bb975fa4e6aeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff1fd000_1575e955","updated":"2026-08-31 07:54:19.000000000","message":"LGTM","commit_id":"21d705ef07590b562f56457f120ebe1459738870"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"a2be52e08dcfe0541584cf5579ebb384e10169aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dbbff591_07ba4517","updated":"2026-08-26 08:24:45.000000000","message":"With this change:\n```\n2026-08-25 14:00:07.385 | + /opt/amphora-agent-venv/bin/pip install -U -c /opt/upper-constraints.txt /opt/octavia-lib\n2026-08-25 14:00:07.862 | Looking in indexes: http://mirror.ca-ymq-1.vexxhost.opendev.org/pypi/simple, http://mirror.ca-ymq-1.vexxhost.opendev.org/wheel/ubuntu-24.04-x86_64\n2026-08-25 14:00:07.864 | Processing ./opt/octavia-lib\n```","commit_id":"21d705ef07590b562f56457f120ebe1459738870"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9e34bb0f2dc3d8bbe96ed9833f8643de365e4d64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"371e4eaf_469b5582","updated":"2026-08-25 13:36:58.000000000","message":"recheck post_failure","commit_id":"21d705ef07590b562f56457f120ebe1459738870"}],"devstack/plugin.sh":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"bfc7478ce3fd4a5804f259658720fe7364c0eae9","unresolved":true,"context_lines":[{"line_number":113,"context_line":"        export DIB_PYPI_MIRROR_URL\u003d$NODEPOOL_PYPI_MIRROR"},{"line_number":114,"context_line":"        export DIB_PYPI_MIRROR_URL_1\u003d$NODEPOOL_WHEEL_MIRROR"},{"line_number":115,"context_line":"        export DIB_PIP_RETRIES\u003d0"},{"line_number":116,"context_line":"        export DIB_NO_PYPI_PIP\u003d1"},{"line_number":117,"context_line":"    fi"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    if ! [ -f $OCTAVIA_AMP_IMAGE_FILE ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"bed4d772_f8deb807","line":116,"range":{"start_line":116,"start_character":8,"end_line":116,"end_character":32},"updated":"2026-08-25 08:34:40.000000000","message":"we should check that DIB_LOCAL_ELEMETNS contains openstack-ci-mirror before setting no_pypi_pip","commit_id":"21d705ef07590b562f56457f120ebe1459738870"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9e34bb0f2dc3d8bbe96ed9833f8643de365e4d64","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        export DIB_PYPI_MIRROR_URL\u003d$NODEPOOL_PYPI_MIRROR"},{"line_number":114,"context_line":"        export DIB_PYPI_MIRROR_URL_1\u003d$NODEPOOL_WHEEL_MIRROR"},{"line_number":115,"context_line":"        export DIB_PIP_RETRIES\u003d0"},{"line_number":116,"context_line":"        export DIB_NO_PYPI_PIP\u003d1"},{"line_number":117,"context_line":"    fi"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    if ! [ -f $OCTAVIA_AMP_IMAGE_FILE ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"c2e285cb_f1fc019e","line":116,"range":{"start_line":116,"start_character":8,"end_line":116,"end_character":32},"in_reply_to":"0298c0df_c984cc40","updated":"2026-08-25 13:36:58.000000000","message":"the if block on L107 uses the results of sourcing mirror_info.sh, so it\u0027s basically the same thing\nI think I won\u0027t check the content of DIB_LOCAL_ELEMENTS becuase in this block, we know that mirrors have been defined, so let\u0027s skip pypi","commit_id":"21d705ef07590b562f56457f120ebe1459738870"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"d3bebd60615a9994761552129aa7f1622bdb6d9c","unresolved":true,"context_lines":[{"line_number":113,"context_line":"        export DIB_PYPI_MIRROR_URL\u003d$NODEPOOL_PYPI_MIRROR"},{"line_number":114,"context_line":"        export DIB_PYPI_MIRROR_URL_1\u003d$NODEPOOL_WHEEL_MIRROR"},{"line_number":115,"context_line":"        export DIB_PIP_RETRIES\u003d0"},{"line_number":116,"context_line":"        export DIB_NO_PYPI_PIP\u003d1"},{"line_number":117,"context_line":"    fi"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    if ! [ -f $OCTAVIA_AMP_IMAGE_FILE ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"0298c0df_c984cc40","line":116,"range":{"start_line":116,"start_character":8,"end_line":116,"end_character":32},"in_reply_to":"bed4d772_f8deb807","updated":"2026-08-25 10:16:43.000000000","message":"Would it make sense to combine it with the check for the mirrors on L104? You would presumably not want to use PyPi if you\u0027ve gone to the trouble of setting a local mirror","commit_id":"21d705ef07590b562f56457f120ebe1459738870"}]}
