)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"14eb7452b4f08d4bef3ea6a92440e0abce1b119d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f06e1ed2_1885ea69","updated":"2024-05-26 06:20:06.000000000","message":"recheck (upgrade job failed on dbus failure)","commit_id":"fd1b8153fff2a88b89e47f2a242e02e1b7fb62ac"}],"tools/kolla-ansible":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"25e65e18f52d338d00a9fc95db61c30dd76a3fdc","unresolved":true,"context_lines":[{"line_number":130,"context_line":""},{"line_number":131,"context_line":"function install_deps {"},{"line_number":132,"context_line":"    attempts\u003d10"},{"line_number":133,"context_line":"    interval\u003d5"},{"line_number":134,"context_line":"    echo \"Installing Ansible Galaxy dependencies\""},{"line_number":135,"context_line":"    if pip show ansible 2\u003e/dev/null; then"},{"line_number":136,"context_line":"        for i in $(seq 1 $attempts); do"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"3411cbfd_02767476","line":133,"updated":"2024-05-24 08:56:01.000000000","message":"+ success\u003d0 for sanity?","commit_id":"d2a372550ff9214c9a85e1e39b6ae2bfc54f8ccc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"bbd7c8334a829d6e93dd715f2da7c07beb948639","unresolved":false,"context_lines":[{"line_number":130,"context_line":""},{"line_number":131,"context_line":"function install_deps {"},{"line_number":132,"context_line":"    attempts\u003d10"},{"line_number":133,"context_line":"    interval\u003d5"},{"line_number":134,"context_line":"    echo \"Installing Ansible Galaxy dependencies\""},{"line_number":135,"context_line":"    if pip show ansible 2\u003e/dev/null; then"},{"line_number":136,"context_line":"        for i in $(seq 1 $attempts); do"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"5b7bf5dd_b73206ea","line":133,"in_reply_to":"3411cbfd_02767476","updated":"2024-05-24 12:29:55.000000000","message":"Done","commit_id":"d2a372550ff9214c9a85e1e39b6ae2bfc54f8ccc"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"25e65e18f52d338d00a9fc95db61c30dd76a3fdc","unresolved":true,"context_lines":[{"line_number":144,"context_line":"    else"},{"line_number":145,"context_line":"        for i in $(seq 1 $attempts); do"},{"line_number":146,"context_line":"            if ansible-galaxy collection install -r ${BASEDIR}/requirements.yml --force; then"},{"line_number":147,"context_line":"                success\u003d1"},{"line_number":148,"context_line":"            if ansible-galaxy collection install -r ${BASEDIR}/requirements-core.yml --force; then"},{"line_number":149,"context_line":"                success\u003d1"},{"line_number":150,"context_line":"                break"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7dfe2fa2_05664f1b","line":147,"updated":"2024-05-24 08:56:01.000000000","message":"missing \"fi\"? though I think it would be better to have two seperate loops?","commit_id":"d2a372550ff9214c9a85e1e39b6ae2bfc54f8ccc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"bbd7c8334a829d6e93dd715f2da7c07beb948639","unresolved":false,"context_lines":[{"line_number":144,"context_line":"    else"},{"line_number":145,"context_line":"        for i in $(seq 1 $attempts); do"},{"line_number":146,"context_line":"            if ansible-galaxy collection install -r ${BASEDIR}/requirements.yml --force; then"},{"line_number":147,"context_line":"                success\u003d1"},{"line_number":148,"context_line":"            if ansible-galaxy collection install -r ${BASEDIR}/requirements-core.yml --force; then"},{"line_number":149,"context_line":"                success\u003d1"},{"line_number":150,"context_line":"                break"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7dd46cd9_1b7c6bb7","line":147,"in_reply_to":"7dfe2fa2_05664f1b","updated":"2024-05-24 12:29:55.000000000","message":"Done","commit_id":"d2a372550ff9214c9a85e1e39b6ae2bfc54f8ccc"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"25e65e18f52d338d00a9fc95db61c30dd76a3fdc","unresolved":true,"context_lines":[{"line_number":154,"context_line":"        done"},{"line_number":155,"context_line":"    fi"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    if [[ $? -ne 0 ]]; then"},{"line_number":158,"context_line":"        echo \"ERROR: Failed to install Ansible Galaxy dependencies\" \u003e\u00262"},{"line_number":159,"context_line":"        exit 1"},{"line_number":160,"context_line":"    fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"50b55bb8_682e376e","line":157,"range":{"start_line":157,"start_character":10,"end_line":157,"end_character":12},"updated":"2024-05-24 08:56:01.000000000","message":"$success ?","commit_id":"d2a372550ff9214c9a85e1e39b6ae2bfc54f8ccc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"bbd7c8334a829d6e93dd715f2da7c07beb948639","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        done"},{"line_number":155,"context_line":"    fi"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    if [[ $? -ne 0 ]]; then"},{"line_number":158,"context_line":"        echo \"ERROR: Failed to install Ansible Galaxy dependencies\" \u003e\u00262"},{"line_number":159,"context_line":"        exit 1"},{"line_number":160,"context_line":"    fi"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"bcb6f37e_ee7e38ef","line":157,"range":{"start_line":157,"start_character":10,"end_line":157,"end_character":12},"in_reply_to":"50b55bb8_682e376e","updated":"2024-05-24 12:29:55.000000000","message":"Done","commit_id":"d2a372550ff9214c9a85e1e39b6ae2bfc54f8ccc"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2f13b252b04c806b3ea34dc1fa830e94588ca5ce","unresolved":true,"context_lines":[{"line_number":158,"context_line":"            fi"},{"line_number":159,"context_line":"            echo \"Galaxy collection download failed, retrying\""},{"line_number":160,"context_line":"            sleep $interval"},{"line_number":161,"context_line":"        done"},{"line_number":162,"context_line":"    fi"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"    if [[ $success -ne 1 ]]; then"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"0f8726a9_490b6c80","line":161,"updated":"2024-08-08 19:27:20.000000000","message":"I think we need some more refined logic here. currently we will claim success if one of the install commands worked, but don\u0027t we actually need both of them? so we\u0027d need success1 and success2 and then `success\u003dsuccess1 \u0026\u0026 success2`, not sure how to best express this in bash","commit_id":"fd1b8153fff2a88b89e47f2a242e02e1b7fb62ac"}]}