)]}'
{"/COMMIT_MSG":[{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for checking more daemons"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is changing way how keepalived is"},{"line_number":10,"context_line":"checking daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"6e4d265d_3df89921","line":9,"range":{"start_line":9,"start_character":23,"end_line":9,"end_character":27},"updated":"2022-04-19 09:59:49.000000000","message":"the way","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for checking more daemons"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is changing way how keepalived is"},{"line_number":10,"context_line":"checking daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"4604d2e3_5fcd73ff","line":9,"range":{"start_line":9,"start_character":23,"end_line":9,"end_character":27},"in_reply_to":"6e4d265d_3df89921","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for checking more daemons"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is changing way how keepalived is"},{"line_number":10,"context_line":"checking daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"},{"line_number":13,"context_line":"which are copied from kolla-ansible and return 0/1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"2c6d4736_f56425aa","line":10,"range":{"start_line":9,"start_character":42,"end_line":10,"end_character":8},"updated":"2022-04-19 09:59:49.000000000","message":"checks","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for checking more daemons"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is changing way how keepalived is"},{"line_number":10,"context_line":"checking daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"},{"line_number":13,"context_line":"which are copied from kolla-ansible and return 0/1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"800ae517_ae4f6453","line":10,"range":{"start_line":9,"start_character":42,"end_line":10,"end_character":8},"in_reply_to":"2c6d4736_f56425aa","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":10,"context_line":"checking daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"},{"line_number":13,"context_line":"which are copied from kolla-ansible and return 0/1"},{"line_number":14,"context_line":"if dead/live."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch is required for proxysql HA."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"05471416_54162c47","line":13,"range":{"start_line":13,"start_character":17,"end_line":13,"end_character":22},"updated":"2022-04-19 09:59:49.000000000","message":"by","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":10,"context_line":"checking daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"},{"line_number":13,"context_line":"which are copied from kolla-ansible and return 0/1"},{"line_number":14,"context_line":"if dead/live."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch is required for proxysql HA."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"b2af26e4_69d3a645","line":13,"range":{"start_line":13,"start_character":17,"end_line":13,"end_character":22},"in_reply_to":"05471416_54162c47","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"},{"line_number":13,"context_line":"which are copied from kolla-ansible and return 0/1"},{"line_number":14,"context_line":"if dead/live."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch is required for proxysql HA."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"c9a6092c_fc708800","line":14,"range":{"start_line":14,"start_character":3,"end_line":14,"end_character":12},"updated":"2022-04-19 09:59:49.000000000","message":"alive/down (to follow the order)","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"},{"line_number":13,"context_line":"which are copied from kolla-ansible and return 0/1"},{"line_number":14,"context_line":"if dead/live."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"This patch is required for proxysql HA."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"a26c7649_9c26cf8d","line":14,"range":{"start_line":14,"start_character":3,"end_line":14,"end_character":12},"in_reply_to":"c9a6092c_fc708800","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"6a654c5d685575d00bb041dc4c17b7a31839fecc","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for checking more daemons"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is changing the way how keepalived is"},{"line_number":10,"context_line":"checks daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":21,"id":"2983f3be_b29afac0","line":9,"range":{"start_line":9,"start_character":46,"end_line":9,"end_character":48},"updated":"2022-04-20 12:09:40.000000000","message":"also, drop this","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for checking more daemons"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is changing the way how keepalived is"},{"line_number":10,"context_line":"checks daemons liveness."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Check_alive.sh will find checks for daemons"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":21,"id":"5558bf9b_88652ca9","line":9,"range":{"start_line":9,"start_character":46,"end_line":9,"end_character":48},"in_reply_to":"2983f3be_b29afac0","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"}],"/PATCHSET_LEVEL":[{"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":"dc42e8d1f9ed6399d8252fe8f5c1c20d5cff4ff8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"04febc21_b5049d1c","updated":"2022-04-04 19:15:49.000000000","message":"Marcin going too fast...","commit_id":"f94efede95038d915255ddc7b0e1f575f7cf53ec"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"b92b76bf1e5c11f772edc01ecbf3934373d99cf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"8010a40a_f079c441","updated":"2022-04-08 10:03:24.000000000","message":"Michal, can you resolve my comment, and let\u0027s get over with this change?","commit_id":"f94efede95038d915255ddc7b0e1f575f7cf53ec"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"aef7102bdbc6ac4210a50f953911e0f31b9d1de0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"386880aa_5ad1a112","updated":"2022-04-04 18:42:21.000000000","message":"Nope, I really don\u0027t agree that this is duplicate of https://review.opendev.org/c/openstack/kolla/+/801231. This is simple way how to add service/container whose VIP address is backuped by keepalived, so k-a just provide check and that\u0027s it...\nThis is also dependency for my proxysql patches, how can I replace it with adjutant ?\n\nAdjutant is completly new openstack service ..this is simple change of one check - moreover if this will be merged and kolla-ansible will not provide another check file for another service ..it will work as it is working now ... So, where is the problem ? \n\n","commit_id":"f94efede95038d915255ddc7b0e1f575f7cf53ec"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"52d75c24_8147e444","updated":"2022-04-20 11:53:04.000000000","message":"thanks, let\u0027s do one last improvement round and merge","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"96e5d22629a177b95b613f41e5f0afc8ccbcc080","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"4ae53dd3_eb532846","updated":"2022-04-21 12:06:30.000000000","message":"recheck\n\npypi issue","commit_id":"1c63c9565840b526c41c0cf20bcc8b8f60a771e3"},{"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":"a662cc1afa2c40c52d506531524494f5302c0b35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"d084c584_7f6c8741","updated":"2022-04-21 10:46:39.000000000","message":"thanks","commit_id":"1c63c9565840b526c41c0cf20bcc8b8f60a771e3"}],"docker/keepalived/check_alive.sh":[{"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":"2cde95f7ad5500c1ca276fce52be2c2a051ae399","unresolved":true,"context_lines":[{"line_number":4,"context_line":"# - it successfully pass all checks for all daemons"},{"line_number":5,"context_line":"# Failures return 1"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ALL_CHECKS\u003d$(ls /checks | grep \u0027check_alive_\u0027)"},{"line_number":8,"context_line":"RESULT\u003d0"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"}],"source_content_type":"text/x-sh","patch_set":3,"id":"c5831be7_6cb145f3","line":7,"range":{"start_line":7,"start_character":26,"end_line":7,"end_character":45},"updated":"2020-12-18 19:33:52.000000000","message":"I would assume the dir to contain only check scripts and name them after the services.","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b1796f3739d19994b5396d41b8bc4da7614b709f","unresolved":true,"context_lines":[{"line_number":4,"context_line":"# - it successfully pass all checks for all daemons"},{"line_number":5,"context_line":"# Failures return 1"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ALL_CHECKS\u003d$(ls /checks | grep \u0027check_alive_\u0027)"},{"line_number":8,"context_line":"RESULT\u003d0"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"}],"source_content_type":"text/x-sh","patch_set":3,"id":"d3bddd68_5befb9bb","line":7,"range":{"start_line":7,"start_character":26,"end_line":7,"end_character":45},"in_reply_to":"c5831be7_6cb145f3","updated":"2020-12-18 19:46:38.000000000","message":"Yes, it is as you said (it\u0027s in other review), of course I can rework...","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"238a0dcb9e0b9f1a525c0c312e3211b1a0c68d84","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# - it successfully pass all checks for all daemons"},{"line_number":5,"context_line":"# Failures return 1"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ALL_CHECKS\u003d$(ls /checks | grep \u0027check_alive_\u0027)"},{"line_number":8,"context_line":"RESULT\u003d0"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"}],"source_content_type":"text/x-sh","patch_set":3,"id":"2f8c474e_63416747","line":7,"range":{"start_line":7,"start_character":26,"end_line":7,"end_character":45},"in_reply_to":"d3bddd68_5befb9bb","updated":"2020-12-19 19:39:34.000000000","message":"Done","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"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":"2cde95f7ad5500c1ca276fce52be2c2a051ae399","unresolved":true,"context_lines":[{"line_number":8,"context_line":"RESULT\u003d0"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"},{"line_number":11,"context_line":"    SERVICE_CHECK_RESULT\u003d$(/checks/${i} \u0026\u0026 echo 0 || echo 1)"},{"line_number":12,"context_line":"    if [[ \"${RESULT}\" -eq 0 \u0026\u0026 \"${SERVICE_CHECK_RESULT}\" -eq 0 ]]; then"},{"line_number":13,"context_line":"        RESULT\u003d0"},{"line_number":14,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":3,"id":"09b69170_7e68c8fd","line":11,"updated":"2020-12-18 19:33:52.000000000","message":"why not just capture the real rc?","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b1796f3739d19994b5396d41b8bc4da7614b709f","unresolved":true,"context_lines":[{"line_number":8,"context_line":"RESULT\u003d0"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"},{"line_number":11,"context_line":"    SERVICE_CHECK_RESULT\u003d$(/checks/${i} \u0026\u0026 echo 0 || echo 1)"},{"line_number":12,"context_line":"    if [[ \"${RESULT}\" -eq 0 \u0026\u0026 \"${SERVICE_CHECK_RESULT}\" -eq 0 ]]; then"},{"line_number":13,"context_line":"        RESULT\u003d0"},{"line_number":14,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":3,"id":"dc73ae66_c833376b","line":11,"in_reply_to":"09b69170_7e68c8fd","updated":"2020-12-18 19:46:38.000000000","message":"I can, i just wrote it as it is now, can rework...","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"238a0dcb9e0b9f1a525c0c312e3211b1a0c68d84","unresolved":false,"context_lines":[{"line_number":8,"context_line":"RESULT\u003d0"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"},{"line_number":11,"context_line":"    SERVICE_CHECK_RESULT\u003d$(/checks/${i} \u0026\u0026 echo 0 || echo 1)"},{"line_number":12,"context_line":"    if [[ \"${RESULT}\" -eq 0 \u0026\u0026 \"${SERVICE_CHECK_RESULT}\" -eq 0 ]]; then"},{"line_number":13,"context_line":"        RESULT\u003d0"},{"line_number":14,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":3,"id":"680163f0_fe3caf3a","line":11,"in_reply_to":"dc73ae66_c833376b","updated":"2020-12-19 19:39:34.000000000","message":"Done","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"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":"2cde95f7ad5500c1ca276fce52be2c2a051ae399","unresolved":true,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"},{"line_number":11,"context_line":"    SERVICE_CHECK_RESULT\u003d$(/checks/${i} \u0026\u0026 echo 0 || echo 1)"},{"line_number":12,"context_line":"    if [[ \"${RESULT}\" -eq 0 \u0026\u0026 \"${SERVICE_CHECK_RESULT}\" -eq 0 ]]; then"},{"line_number":13,"context_line":"        RESULT\u003d0"},{"line_number":14,"context_line":"    else"},{"line_number":15,"context_line":"        RESULT\u003d1"},{"line_number":16,"context_line":"    fi"},{"line_number":17,"context_line":"done"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"exit ${RESULT}"}],"source_content_type":"text/x-sh","patch_set":3,"id":"c89971a0_5efa8579","line":16,"range":{"start_line":12,"start_character":4,"end_line":16,"end_character":6},"updated":"2020-12-18 19:33:52.000000000","message":"we can fail fast on failure, otherwise not","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"238a0dcb9e0b9f1a525c0c312e3211b1a0c68d84","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"},{"line_number":11,"context_line":"    SERVICE_CHECK_RESULT\u003d$(/checks/${i} \u0026\u0026 echo 0 || echo 1)"},{"line_number":12,"context_line":"    if [[ \"${RESULT}\" -eq 0 \u0026\u0026 \"${SERVICE_CHECK_RESULT}\" -eq 0 ]]; then"},{"line_number":13,"context_line":"        RESULT\u003d0"},{"line_number":14,"context_line":"    else"},{"line_number":15,"context_line":"        RESULT\u003d1"},{"line_number":16,"context_line":"    fi"},{"line_number":17,"context_line":"done"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"exit ${RESULT}"}],"source_content_type":"text/x-sh","patch_set":3,"id":"96d837fb_b823f3ab","line":16,"range":{"start_line":12,"start_character":4,"end_line":16,"end_character":6},"in_reply_to":"4ab0ccf2_39b97f46","updated":"2020-12-19 19:39:34.000000000","message":"Done","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b1796f3739d19994b5396d41b8bc4da7614b709f","unresolved":true,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"for i in ${ALL_CHECKS}; do"},{"line_number":11,"context_line":"    SERVICE_CHECK_RESULT\u003d$(/checks/${i} \u0026\u0026 echo 0 || echo 1)"},{"line_number":12,"context_line":"    if [[ \"${RESULT}\" -eq 0 \u0026\u0026 \"${SERVICE_CHECK_RESULT}\" -eq 0 ]]; then"},{"line_number":13,"context_line":"        RESULT\u003d0"},{"line_number":14,"context_line":"    else"},{"line_number":15,"context_line":"        RESULT\u003d1"},{"line_number":16,"context_line":"    fi"},{"line_number":17,"context_line":"done"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"exit ${RESULT}"}],"source_content_type":"text/x-sh","patch_set":3,"id":"4ab0ccf2_39b97f46","line":16,"range":{"start_line":12,"start_character":4,"end_line":16,"end_character":6},"in_reply_to":"c89971a0_5efa8579","updated":"2020-12-18 19:46:38.000000000","message":"Agree","commit_id":"10195c6f6141bf4c7a352c92bfbbcd7246275868"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"9713be4125785783c7ca1f281116abeed3f47faa","unresolved":true,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"CHECKS\u003d$(find /checks -type f)"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"for check in ${CHECKS}; do"},{"line_number":9,"context_line":"    ${check} || exit 1"},{"line_number":10,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":19,"id":"2df42a5b_5d485998","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":22},"updated":"2021-09-08 13:43:10.000000000","message":"what if this ends up empty? no checks at all? should we default to previous behaviour?","commit_id":"f94efede95038d915255ddc7b0e1f575f7cf53ec"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"b92b76bf1e5c11f772edc01ecbf3934373d99cf4","unresolved":true,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"CHECKS\u003d$(find /checks -type f)"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"for check in ${CHECKS}; do"},{"line_number":9,"context_line":"    ${check} || exit 1"},{"line_number":10,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":19,"id":"9f78a22a_bdfcba8c","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":22},"in_reply_to":"2df42a5b_5d485998","updated":"2022-04-08 10:03:24.000000000","message":"Michal, can you add an if statement when ${CHECKS} is empty to do previous behaviour and let\u0027s finally merge this change?","commit_id":"f94efede95038d915255ddc7b0e1f575f7cf53ec"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b77d20e12d1bdade580fb14bca7700a90446f5f6","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"CHECKS\u003d$(find /checks -type f)"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"for check in ${CHECKS}; do"},{"line_number":9,"context_line":"    ${check} || exit 1"},{"line_number":10,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":19,"id":"5920ea3a_45dd8f8c","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":22},"in_reply_to":"9f78a22a_bdfcba8c","updated":"2022-04-13 12:24:30.000000000","message":"Done","commit_id":"f94efede95038d915255ddc7b0e1f575f7cf53ec"},{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# This will return 0 when it successfully pass all checks for all daemons"},{"line_number":4,"context_line":"# Failures return 1"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"if [ -d \"/checks\" ]; then"}],"source_content_type":"text/x-sh","patch_set":20,"id":"042105a6_33716451","line":3,"range":{"start_line":3,"start_character":42,"end_line":3,"end_character":47},"updated":"2022-04-19 09:59:49.000000000","message":"passes","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# This will return 0 when it successfully pass all checks for all daemons"},{"line_number":4,"context_line":"# Failures return 1"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"if [ -d \"/checks\" ]; then"}],"source_content_type":"text/x-sh","patch_set":20,"id":"de00a137_61aa8cdd","line":3,"range":{"start_line":3,"start_character":42,"end_line":3,"end_character":47},"in_reply_to":"042105a6_33716451","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"if [ \"${CHECKS}\" ]; then"},{"line_number":11,"context_line":"    for check in ${CHECKS}; do"},{"line_number":12,"context_line":"        ${check} || exit 1"},{"line_number":13,"context_line":"    done"},{"line_number":14,"context_line":"else"},{"line_number":15,"context_line":"    # Original single check of haproxy liveness"}],"source_content_type":"text/x-sh","patch_set":20,"id":"da0334a0_23435b8d","line":12,"range":{"start_line":12,"start_character":8,"end_line":12,"end_character":26},"updated":"2022-04-19 09:59:49.000000000","message":"FWIW, it would be better if we collected all outputs and then failed (so that the user can already see how many checks fail). Not sure how much value that gives though.","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"94fd0081886ee6d1b69ddfefd7dbffae497f3ae2","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"if [ \"${CHECKS}\" ]; then"},{"line_number":11,"context_line":"    for check in ${CHECKS}; do"},{"line_number":12,"context_line":"        ${check} || exit 1"},{"line_number":13,"context_line":"    done"},{"line_number":14,"context_line":"else"},{"line_number":15,"context_line":"    # Original single check of haproxy liveness"}],"source_content_type":"text/x-sh","patch_set":20,"id":"4793aec0_47cecd1d","line":12,"range":{"start_line":12,"start_character":8,"end_line":12,"end_character":26},"in_reply_to":"b0b58053_f4cb1005","updated":"2022-04-20 10:13:23.000000000","message":"Implemented, this is output from keepalived when i run it and stop proxysql container.\n\n(keepalived)[root@controller0 /]# /check_alive.sh\nKeepalived check script /checks/check_alive_haproxy.sh passed.\nKeepalived check script /checks/check_alive_proxysql.sh failed.","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"if [ \"${CHECKS}\" ]; then"},{"line_number":11,"context_line":"    for check in ${CHECKS}; do"},{"line_number":12,"context_line":"        ${check} || exit 1"},{"line_number":13,"context_line":"    done"},{"line_number":14,"context_line":"else"},{"line_number":15,"context_line":"    # Original single check of haproxy liveness"}],"source_content_type":"text/x-sh","patch_set":20,"id":"b0b58053_f4cb1005","line":12,"range":{"start_line":12,"start_character":8,"end_line":12,"end_character":26},"in_reply_to":"da0334a0_23435b8d","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"158ad30bbb2b89670619b546e9ce1ee1cfb89229","unresolved":true,"context_lines":[{"line_number":12,"context_line":"        ${check} || exit 1"},{"line_number":13,"context_line":"    done"},{"line_number":14,"context_line":"else"},{"line_number":15,"context_line":"    # Original single check of haproxy liveness"},{"line_number":16,"context_line":"    echo \"show info\" | socat unix-connect:/var/lib/kolla/haproxy/haproxy.sock stdio \u003e /dev/null"},{"line_number":17,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":20,"id":"94c52973_e23cc7a1","line":15,"range":{"start_line":15,"start_character":6,"end_line":15,"end_character":15},"updated":"2022-04-19 09:59:49.000000000","message":"Legacy;\n\nwe can also mark this with a TODO for removal after k-a patches merge","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4ae7367778fe8c21014f52cc22ae42b386f0735a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"        ${check} || exit 1"},{"line_number":13,"context_line":"    done"},{"line_number":14,"context_line":"else"},{"line_number":15,"context_line":"    # Original single check of haproxy liveness"},{"line_number":16,"context_line":"    echo \"show info\" | socat unix-connect:/var/lib/kolla/haproxy/haproxy.sock stdio \u003e /dev/null"},{"line_number":17,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":20,"id":"f8392a8b_1aa67947","line":15,"range":{"start_line":15,"start_character":6,"end_line":15,"end_character":15},"in_reply_to":"94c52973_e23cc7a1","updated":"2022-04-20 10:12:42.000000000","message":"Done","commit_id":"58a4276eddffbb477e2dbc203ba7d7d41ee3614c"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":14,"context_line":"    # Initialize array"},{"line_number":15,"context_line":"    for check in ${CHECKS}; do"},{"line_number":16,"context_line":"        check_results[${check}]\u003d0"},{"line_number":17,"context_line":"    done"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    # Store results"},{"line_number":20,"context_line":"    for check in ${CHECKS}; do"}],"source_content_type":"text/x-sh","patch_set":21,"id":"b9870a47_fae40b27","line":17,"updated":"2022-04-20 11:53:04.000000000","message":"you really don\u0027t need to initialise it","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":14,"context_line":"    # Initialize array"},{"line_number":15,"context_line":"    for check in ${CHECKS}; do"},{"line_number":16,"context_line":"        check_results[${check}]\u003d0"},{"line_number":17,"context_line":"    done"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    # Store results"},{"line_number":20,"context_line":"    for check in ${CHECKS}; do"}],"source_content_type":"text/x-sh","patch_set":21,"id":"6ba1ce63_fb13ddbc","line":17,"in_reply_to":"b9870a47_fae40b27","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    # Store results"},{"line_number":20,"context_line":"    for check in ${CHECKS}; do"},{"line_number":21,"context_line":"        # Run check but do not print stderr to stdout"},{"line_number":22,"context_line":"        # as single check can be executed manually and see result"},{"line_number":23,"context_line":"        ${check} 2\u003e/dev/null"},{"line_number":24,"context_line":"        check_results[${check}]\u003d$?"}],"source_content_type":"text/x-sh","patch_set":21,"id":"66b9c70c_105e2a30","line":21,"range":{"start_line":21,"start_character":44,"end_line":21,"end_character":53},"updated":"2022-04-20 11:53:04.000000000","message":"(skip this)","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    # Store results"},{"line_number":20,"context_line":"    for check in ${CHECKS}; do"},{"line_number":21,"context_line":"        # Run check but do not print stderr to stdout"},{"line_number":22,"context_line":"        # as single check can be executed manually and see result"},{"line_number":23,"context_line":"        ${check} 2\u003e/dev/null"},{"line_number":24,"context_line":"        check_results[${check}]\u003d$?"}],"source_content_type":"text/x-sh","patch_set":21,"id":"bfce11ac_f0501085","line":21,"range":{"start_line":21,"start_character":44,"end_line":21,"end_character":53},"in_reply_to":"66b9c70c_105e2a30","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":19,"context_line":"    # Store results"},{"line_number":20,"context_line":"    for check in ${CHECKS}; do"},{"line_number":21,"context_line":"        # Run check but do not print stderr to stdout"},{"line_number":22,"context_line":"        # as single check can be executed manually and see result"},{"line_number":23,"context_line":"        ${check} 2\u003e/dev/null"},{"line_number":24,"context_line":"        check_results[${check}]\u003d$?"},{"line_number":25,"context_line":"    done"}],"source_content_type":"text/x-sh","patch_set":21,"id":"dbee23a1_f5d793fa","line":22,"range":{"start_line":22,"start_character":51,"end_line":22,"end_character":59},"updated":"2022-04-20 11:53:04.000000000","message":"to see the","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    # Store results"},{"line_number":20,"context_line":"    for check in ${CHECKS}; do"},{"line_number":21,"context_line":"        # Run check but do not print stderr to stdout"},{"line_number":22,"context_line":"        # as single check can be executed manually and see result"},{"line_number":23,"context_line":"        ${check} 2\u003e/dev/null"},{"line_number":24,"context_line":"        check_results[${check}]\u003d$?"},{"line_number":25,"context_line":"    done"}],"source_content_type":"text/x-sh","patch_set":21,"id":"80ca00ff_62de2a76","line":22,"range":{"start_line":22,"start_character":51,"end_line":22,"end_character":59},"in_reply_to":"dbee23a1_f5d793fa","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":24,"context_line":"        check_results[${check}]\u003d$?"},{"line_number":25,"context_line":"    done"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    # Print results and save final result"},{"line_number":28,"context_line":"    for i in \"${!check_results[@]}\"; do"},{"line_number":29,"context_line":"        if [ \"${check_results[$i]}\" \u003d\u003d \"0\" ]; then"},{"line_number":30,"context_line":"            echo \"Keepalived check script ${i} passed.\""}],"source_content_type":"text/x-sh","patch_set":21,"id":"aa87fa2c_9bdb11d2","line":27,"range":{"start_line":27,"start_character":29,"end_line":27,"end_character":35},"updated":"2022-04-20 11:53:04.000000000","message":"the final","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        check_results[${check}]\u003d$?"},{"line_number":25,"context_line":"    done"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    # Print results and save final result"},{"line_number":28,"context_line":"    for i in \"${!check_results[@]}\"; do"},{"line_number":29,"context_line":"        if [ \"${check_results[$i]}\" \u003d\u003d \"0\" ]; then"},{"line_number":30,"context_line":"            echo \"Keepalived check script ${i} passed.\""}],"source_content_type":"text/x-sh","patch_set":21,"id":"b597410d_7396b932","line":27,"range":{"start_line":27,"start_character":29,"end_line":27,"end_character":35},"in_reply_to":"aa87fa2c_9bdb11d2","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":27,"context_line":"    # Print results and save final result"},{"line_number":28,"context_line":"    for i in \"${!check_results[@]}\"; do"},{"line_number":29,"context_line":"        if [ \"${check_results[$i]}\" \u003d\u003d \"0\" ]; then"},{"line_number":30,"context_line":"            echo \"Keepalived check script ${i} passed.\""},{"line_number":31,"context_line":"        else"},{"line_number":32,"context_line":"            final_result\u003d1"},{"line_number":33,"context_line":"            echo \"Keepalived check script ${i} failed.\""}],"source_content_type":"text/x-sh","patch_set":21,"id":"415177d3_81c73aba","line":30,"range":{"start_line":30,"start_character":47,"end_line":30,"end_character":53},"updated":"2022-04-20 11:53:04.000000000","message":"succeeded","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    # Print results and save final result"},{"line_number":28,"context_line":"    for i in \"${!check_results[@]}\"; do"},{"line_number":29,"context_line":"        if [ \"${check_results[$i]}\" \u003d\u003d \"0\" ]; then"},{"line_number":30,"context_line":"            echo \"Keepalived check script ${i} passed.\""},{"line_number":31,"context_line":"        else"},{"line_number":32,"context_line":"            final_result\u003d1"},{"line_number":33,"context_line":"            echo \"Keepalived check script ${i} failed.\""}],"source_content_type":"text/x-sh","patch_set":21,"id":"fe79998d_5bdcf3fb","line":30,"range":{"start_line":30,"start_character":47,"end_line":30,"end_character":53},"in_reply_to":"415177d3_81c73aba","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":33,"context_line":"            echo \"Keepalived check script ${i} failed.\""},{"line_number":34,"context_line":"        fi"},{"line_number":35,"context_line":"    done"},{"line_number":36,"context_line":"    exit ${final_result}"},{"line_number":37,"context_line":"else"},{"line_number":38,"context_line":"    # Legacy single check of haproxy liveness"},{"line_number":39,"context_line":"    # TODO(kevko): This can be removed after k-a part will be merged"}],"source_content_type":"text/x-sh","patch_set":21,"id":"fd2850f0_875ddb72","line":36,"updated":"2022-04-20 11:53:04.000000000","message":"could use a newline","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            echo \"Keepalived check script ${i} failed.\""},{"line_number":34,"context_line":"        fi"},{"line_number":35,"context_line":"    done"},{"line_number":36,"context_line":"    exit ${final_result}"},{"line_number":37,"context_line":"else"},{"line_number":38,"context_line":"    # Legacy single check of haproxy liveness"},{"line_number":39,"context_line":"    # TODO(kevko): This can be removed after k-a part will be merged"}],"source_content_type":"text/x-sh","patch_set":21,"id":"8d025058_75b91756","line":36,"in_reply_to":"fd2850f0_875ddb72","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"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":"80c28e8ebd65c619d1de95df489b47298e9124d0","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    exit ${final_result}"},{"line_number":37,"context_line":"else"},{"line_number":38,"context_line":"    # Legacy single check of haproxy liveness"},{"line_number":39,"context_line":"    # TODO(kevko): This can be removed after k-a part will be merged"},{"line_number":40,"context_line":"    # https://review.opendev.org/c/openstack/kolla-ansible/+/770215"},{"line_number":41,"context_line":"    echo \"show info\" | socat unix-connect:/var/lib/kolla/haproxy/haproxy.sock stdio \u003e /dev/null"},{"line_number":42,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":21,"id":"b7022366_e2ed9957","line":39,"range":{"start_line":39,"start_character":54,"end_line":39,"end_character":68},"updated":"2022-04-20 11:53:04.000000000","message":"merges","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"40ec8f8f063108dc947127866ffb97eb19e8f000","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    exit ${final_result}"},{"line_number":37,"context_line":"else"},{"line_number":38,"context_line":"    # Legacy single check of haproxy liveness"},{"line_number":39,"context_line":"    # TODO(kevko): This can be removed after k-a part will be merged"},{"line_number":40,"context_line":"    # https://review.opendev.org/c/openstack/kolla-ansible/+/770215"},{"line_number":41,"context_line":"    echo \"show info\" | socat unix-connect:/var/lib/kolla/haproxy/haproxy.sock stdio \u003e /dev/null"},{"line_number":42,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":21,"id":"2740c682_5feecd9e","line":39,"range":{"start_line":39,"start_character":54,"end_line":39,"end_character":68},"in_reply_to":"b7022366_e2ed9957","updated":"2022-04-21 10:29:21.000000000","message":"Done","commit_id":"76603ffec30167ff24595881c1b23a8817a995eb"}]}