From 99079ac79f1e074d811779c0448e2310c1778510 Mon Sep 17 00:00:00 2001 From: Junhao Liao Date: Wed, 19 Nov 2025 17:50:05 -0500 Subject: [PATCH 1/3] docs(clp-config): Add commented out default values to nullable `mcp_server` and `presto` in `clp-config.yaml`; Clarify configuration steps in the manuals. --- .../package-template/src/etc/clp-config.yaml | 4 ++++ docs/src/user-docs/guides-mcp-server/index.md | 17 +++++------------ docs/src/user-docs/guides-using-presto.md | 9 ++++----- 3 files changed, 13 insertions(+), 17 deletions(-) diff --git a/components/package-template/src/etc/clp-config.yaml b/components/package-template/src/etc/clp-config.yaml index f87a7acb2a..fdb88420f6 100644 --- a/components/package-template/src/etc/clp-config.yaml +++ b/components/package-template/src/etc/clp-config.yaml @@ -69,6 +69,8 @@ # rate_limit: 1000 # #mcp_server: null +## host: "localhost" +## port: 8000 # ## Where archives should be output to #archive_output: @@ -129,6 +131,8 @@ # ## Presto client config #presto: null +## host: "localhost" +## port: 8889 # ## Location where other data (besides archives) are stored. It will be created if ## it doesn't exist. diff --git a/docs/src/user-docs/guides-mcp-server/index.md b/docs/src/user-docs/guides-mcp-server/index.md index 17f7f557bf..8594c22489 100644 --- a/docs/src/user-docs/guides-mcp-server/index.md +++ b/docs/src/user-docs/guides-mcp-server/index.md @@ -21,14 +21,9 @@ This guide assumes: ## Starting the MCP Server -1. Configure `clp-json` to run the MCP server by locating the `mcp_server` section in - `etc/clp-config.yaml` - - ```yaml - #mcp_server: null - ``` - - then uncommenting it and specifying a `host` and `port`: +1. Configure `clp-json` to run the MCP server by uncommenting the `mcp_server` section in + `etc/clp-config.yaml` and specifying a `host` and `port`. Replace the default values if + necessary. ```yaml mcp_server: @@ -37,11 +32,9 @@ This guide assumes: # other settings ``` - Replace the default `host` and `port` shown above if necessary. - :::{note} - If you set `mcp_server: null` or leave the `mcp_server` section commented, the MCP server won't - be started with the rest of `clp-json`. + Setting `mcp_server: null` or leaving the section commented will prevent the MCP server from + starting with `clp-json`. ::: 2. Start `clp-json` and compress the logs you want to query by following the [clp-json diff --git a/docs/src/user-docs/guides-using-presto.md b/docs/src/user-docs/guides-using-presto.md index dc20d073eb..cb40d18ea8 100644 --- a/docs/src/user-docs/guides-using-presto.md +++ b/docs/src/user-docs/guides-using-presto.md @@ -57,14 +57,13 @@ Using Presto with CLP requires: retention_period: null ``` - * Update the `presto` key with the host and port of the Presto cluster. If you follow the - [Setting up Presto](#setting-up-presto) section, the host is `localhost` and the port is - `8889`. + * Update the `presto` key with the `host` and `port` of the Presto cluster. Replace the default + values if necessary. ```yaml presto: - host: "" - port: + host: "localhost" + port: 8889 ``` :::{note} From d9eaff0c7c67477d286e7c874cc33beb79699610 Mon Sep 17 00:00:00 2001 From: Junhao Liao Date: Mon, 16 Feb 2026 21:08:00 -0500 Subject: [PATCH 2/3] docs(config): Update Presto client configuration placeholders for clarity --- .../package-template/src/etc/clp-config.template.json.yaml | 2 ++ .../package-template/src/etc/clp-config.template.text.yaml | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/components/package-template/src/etc/clp-config.template.json.yaml b/components/package-template/src/etc/clp-config.template.json.yaml index ed83f92b50..888a7d726c 100644 --- a/components/package-template/src/etc/clp-config.template.json.yaml +++ b/components/package-template/src/etc/clp-config.template.json.yaml @@ -156,6 +156,8 @@ # ## Presto client config #presto: null +## host: # default: "localhost" +## port: # default: 8889 # ## Location where other data (besides archives) are stored. It will be created if ## it doesn't exist. diff --git a/components/package-template/src/etc/clp-config.template.text.yaml b/components/package-template/src/etc/clp-config.template.text.yaml index 5c55b2f024..59544d8880 100644 --- a/components/package-template/src/etc/clp-config.template.text.yaml +++ b/components/package-template/src/etc/clp-config.template.text.yaml @@ -141,8 +141,8 @@ log_ingestor: null # ## Presto client config #presto: null -## host: "localhost" -## port: 8889 +## host: # default: "localhost" +## port: # default: 8889 # ## Location where other data (besides archives) are stored. It will be created if ## it doesn't exist. From c85afa3b8ef03e6b0640e6a9665ec943c8ef0d39 Mon Sep 17 00:00:00 2001 From: Junhao Liao Date: Tue, 17 Feb 2026 14:06:24 +0800 Subject: [PATCH 3/3] add mcp values to json.yaml as well --- .../package-template/src/etc/clp-config.template.json.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/components/package-template/src/etc/clp-config.template.json.yaml b/components/package-template/src/etc/clp-config.template.json.yaml index 888a7d726c..ee27ce7453 100644 --- a/components/package-template/src/etc/clp-config.template.json.yaml +++ b/components/package-template/src/etc/clp-config.template.json.yaml @@ -105,6 +105,8 @@ # rate_limit: 1000 # #mcp_server: null +## host: "localhost" +## port: 8000 # ## Where archives should be output to #archive_output: