)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"efaa36ddecace5c7df85d7e8f6789be4bf605866","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a6bb27e0_b80b9fe0","updated":"2022-03-08 23:23:43.000000000","message":"recheck","commit_id":"7a61ddea034f20b3f6e0be9a344294744126a975"}],"container_config_scripts/pacemaker_restart_bundle.sh":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"123ac9a31bbe65b7b35a0ec09bde81a0d92b5fae","unresolved":true,"context_lines":[{"line_number":76,"context_line":"    HOST\u003d$(facter hostname)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    if bundle_can_be_restarted ${BUNDLE_NAME}; then"},{"line_number":79,"context_line":"    # if the resource is running locally, restart it"},{"line_number":80,"context_line":"    if crm_resource -r $BUNDLE_NAME --locate 2\u003e\u00261 | grep -w -q \"${HOST}\"; then"},{"line_number":81,"context_line":"        log \"Restarting ${BUNDLE_NAME} locally on \u0027${HOST}\u0027\""},{"line_number":82,"context_line":"        /sbin/pcs resource restart $BUNDLE_NAME \"${HOST}\""},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    else"},{"line_number":85,"context_line":"        # At this point, if no resource is running locally, it\u0027s"},{"line_number":86,"context_line":"        # either because a) it has failed previously, or b) because"},{"line_number":87,"context_line":"        # it\u0027s an A/P resource running elsewhere."},{"line_number":88,"context_line":"        # By cleaning up resource, we ensure that a) it will try to"},{"line_number":89,"context_line":"        # restart, or b) it won\u0027t do anything if the resource is"},{"line_number":90,"context_line":"        # already running elsewhere."},{"line_number":91,"context_line":"        log \"${BUNDLE_NAME} is currently not running on \u0027${HOST}\u0027,\" \\"},{"line_number":92,"context_line":"             \"cleaning up its state to restart it if necessary\""},{"line_number":93,"context_line":"        /sbin/pcs resource cleanup $BUNDLE_NAME node\u003d\"${HOST}\""},{"line_number":94,"context_line":"    fi"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # Wait until the resource is in the expected target state"},{"line_number":97,"context_line":"    $RESTART_SCRIPTS_DIR/pacemaker_wait_bundle.sh \\"},{"line_number":98,"context_line":"            $RESOURCE_NAME $BUNDLE_NAME \\"},{"line_number":99,"context_line":"            \"$WAIT_TARGET_LOCAL\" \"$WAIT_TARGET_ANYWHERE\" \\"},{"line_number":100,"context_line":"            \"${HOST}\" __PCMKTIMEOUT__"},{"line_number":101,"context_line":"    else"},{"line_number":102,"context_line":"        log \"No restart needed for ${BUNDLE_NAME}.\""},{"line_number":103,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"541e127a_9435d6d3","line":100,"range":{"start_line":79,"start_character":0,"end_line":100,"end_character":37},"updated":"2022-02-28 15:14:46.000000000","message":"This block requirs additional indent, as it is nested in the if-block L78-101.","commit_id":"cc61a9235a62f59dc50c5f31287cce918f805910"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1e4b90eb65dc70aa7eeab6315762ea8e4d07f159","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    HOST\u003d$(facter hostname)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    if bundle_can_be_restarted ${BUNDLE_NAME}; then"},{"line_number":79,"context_line":"    # if the resource is running locally, restart it"},{"line_number":80,"context_line":"    if crm_resource -r $BUNDLE_NAME --locate 2\u003e\u00261 | grep -w -q \"${HOST}\"; then"},{"line_number":81,"context_line":"        log \"Restarting ${BUNDLE_NAME} locally on \u0027${HOST}\u0027\""},{"line_number":82,"context_line":"        /sbin/pcs resource restart $BUNDLE_NAME \"${HOST}\""},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    else"},{"line_number":85,"context_line":"        # At this point, if no resource is running locally, it\u0027s"},{"line_number":86,"context_line":"        # either because a) it has failed previously, or b) because"},{"line_number":87,"context_line":"        # it\u0027s an A/P resource running elsewhere."},{"line_number":88,"context_line":"        # By cleaning up resource, we ensure that a) it will try to"},{"line_number":89,"context_line":"        # restart, or b) it won\u0027t do anything if the resource is"},{"line_number":90,"context_line":"        # already running elsewhere."},{"line_number":91,"context_line":"        log \"${BUNDLE_NAME} is currently not running on \u0027${HOST}\u0027,\" \\"},{"line_number":92,"context_line":"             \"cleaning up its state to restart it if necessary\""},{"line_number":93,"context_line":"        /sbin/pcs resource cleanup $BUNDLE_NAME node\u003d\"${HOST}\""},{"line_number":94,"context_line":"    fi"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # Wait until the resource is in the expected target state"},{"line_number":97,"context_line":"    $RESTART_SCRIPTS_DIR/pacemaker_wait_bundle.sh \\"},{"line_number":98,"context_line":"            $RESOURCE_NAME $BUNDLE_NAME \\"},{"line_number":99,"context_line":"            \"$WAIT_TARGET_LOCAL\" \"$WAIT_TARGET_ANYWHERE\" \\"},{"line_number":100,"context_line":"            \"${HOST}\" __PCMKTIMEOUT__"},{"line_number":101,"context_line":"    else"},{"line_number":102,"context_line":"        log \"No restart needed for ${BUNDLE_NAME}.\""},{"line_number":103,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"24e6c619_1ced6f2a","line":100,"range":{"start_line":79,"start_character":0,"end_line":100,"end_character":37},"in_reply_to":"541e127a_9435d6d3","updated":"2022-02-28 15:15:48.000000000","message":"Done","commit_id":"cc61a9235a62f59dc50c5f31287cce918f805910"},{"author":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"change_message_id":"76d5ec921303b2ccc81d5d23ba280105a8c69f24","unresolved":true,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        # Wait until the resource is in the expected target state"},{"line_number":97,"context_line":"        $RESTART_SCRIPTS_DIR/pacemaker_wait_bundle.sh \\"},{"line_number":98,"context_line":"                $RESOURCE_NAME $BUNDLE_NAME \\"},{"line_number":99,"context_line":"                \"$WAIT_TARGET_LOCAL\" \"$WAIT_TARGET_ANYWHERE\" \\"},{"line_number":100,"context_line":"                \"${HOST}\" __PCMKTIMEOUT__"},{"line_number":101,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7074ae48_e2d6c2a7","line":98,"updated":"2022-02-28 16:31:23.000000000","message":"This indentation is now 8 characters instead of 4, is that on purpose?","commit_id":"7a61ddea034f20b3f6e0be9a344294744126a975"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8070178cc271f8ad434af89b5bdaba23dbbbdcf4","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        # Wait until the resource is in the expected target state"},{"line_number":97,"context_line":"        $RESTART_SCRIPTS_DIR/pacemaker_wait_bundle.sh \\"},{"line_number":98,"context_line":"                $RESOURCE_NAME $BUNDLE_NAME \\"},{"line_number":99,"context_line":"                \"$WAIT_TARGET_LOCAL\" \"$WAIT_TARGET_ANYWHERE\" \\"},{"line_number":100,"context_line":"                \"${HOST}\" __PCMKTIMEOUT__"},{"line_number":101,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":4,"id":"3ccf60e9_9c908301","line":98,"in_reply_to":"7074ae48_e2d6c2a7","updated":"2022-03-08 23:38:56.000000000","message":"Done. For records I initially intend to distinguish continuous line an logical indent but it might not be required as long as we put blank line properly.","commit_id":"7a61ddea034f20b3f6e0be9a344294744126a975"}]}