fix config errors

Signed-off-by: Joshua Li <[email protected]>
joshuali925 · Dec 21, 2023 · 80fbe1a · 80fbe1a
1 parent 36a82fc
commit 80fbe1a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/opensearch_dashboards.json b/opensearch_dashboards.json
@@ -1,7 +1,7 @@
 {
   "id": "observabilityDashboards",
-  "version": "2.11.0.0",
-  "opensearchDashboardsVersion": "2.11.0",
+  "version": "3.0.0.0",
+  "opensearchDashboardsVersion": "3.0.0",
   "server": true,
   "ui": true,
   "requiredPlugins": [

diff --git a/server/routes/query_assist/routes.ts b/server/routes/query_assist/routes.ts
@@ -37,7 +37,7 @@ export function registerQueryAssistRoutes(router: IRouter, config: Observability
     ppl_agent_id: pplAgentId,
     response_summary_agent_id: responseSummaryAgentId,
     error_summary_agent_id: ErrorSummaryAgentId,
-  } = config.observability.query_assist;
+  } = config.query_assist;
 
   router.post(
     {
@@ -54,7 +54,12 @@ export function registerQueryAssistRoutes(router: IRouter, config: Observability
       request,
       response
     ): Promise<IOpenSearchDashboardsResponse<any | ResponseError>> => {
-      if (!pplAgentId) return response.custom({ statusCode: 400, body: 'PPL agent not found.' });
+      if (!pplAgentId)
+        return response.custom({
+          statusCode: 400,
+          body:
+            'PPL agent not found in opensearch_dashboards.yml. Expected observability.query_assist.ppl_agent_id',
+        });
 
       const client = context.core.opensearch.client.asCurrentUser;
       try {
@@ -112,7 +117,11 @@ export function registerQueryAssistRoutes(router: IRouter, config: Observability
       response
     ): Promise<IOpenSearchDashboardsResponse<any | ResponseError>> => {
       if (!responseSummaryAgentId || !ErrorSummaryAgentId)
-        return response.custom({ statusCode: 400, body: 'Summary agent not found.' });
+        return response.custom({
+          statusCode: 400,
+          body:
+            'Summary agent not found in opensearch_dashboards.yml. Expected observability.query_assist.response_summary_agent_id and observability.query_assist.error_summary_agent_id',
+        });
 
       const client = context.core.opensearch.client.asCurrentUser;
       const { index, question, query, response: _response, isError } = request.body;