Properly bubble up failures in executeDql to the end-user

[christian-kreuzberger-dtx]

If there is a failure in here should we throw a customer error message instead of undefined?

Originally posted by @krisdyna in #30 (comment)

[MrManny]

Please make sure to check the notifications in the result payload. Here's an example when it kaboomed into my face:

2025-07-29 13:47:32.836 [warning] Failed to parse message: "executeDql response {\n"
2025-07-29 13:47:32.836 [warning] Failed to parse message: "  state: 'RUNNING',\n"
2025-07-29 13:47:32.836 [warning] Failed to parse message: "  requestToken: 'QkZ1ynnnSayvaGtLUKU7rg==',\n"
2025-07-29 13:47:32.836 [warning] Failed to parse message: "  ttlSeconds: 399\n"
2025-07-29 13:47:32.837 [warning] Failed to parse message: "}\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "executeDql pollResponse {\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "  state: 'SUCCEEDED',\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "  progress: 100,\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "  result: {\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "    records: [ { 'count()': '0' } ],\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "    types: [\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "      {\n"
2025-07-29 13:47:34.988 [warning] Failed to parse message: "        indexRange: [ 0, 0 ],\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "        mappings: { 'count()': { type: 'long' } }\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "      }\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "    ],\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "    metadata: {\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "      grail: {\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "        canonicalQuery: 'fetch logs, from:now() - 24h, to:now(), scanLimitGBytes:-1\\n' +\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "          '| filter dt.app.id == \"dynatrace.site.reliability.guardian\" AND dt.app.function == \"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/api/validate-guardian-action\" AND loglevel == \"ERROR\"\\n' +\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "          '| summarize count()',\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "        timezone: 'Z',\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "        query: 'fetch logs, scanLimitGBytes: -1, from: now()-24h, to:now()\\n' +\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "          '| filter dt.app.id == \"dynatrace.site.reliability.guardian\" AND dt.app.function == \"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/api/validate-guardian-action\" AND loglevel == \"ERROR\"\\n' +\n"
2025-07-29 13:47:34.989 [warning] Failed to parse message: "          '| summarize count()',\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        scannedRecords: 0,\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        dqlVersion: 'V1_0',\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        scannedBytes: 0,\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        scannedDataPoints: 0,\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        analysisTimeframe: {\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "          start: 2025-07-28T11:47:37.619Z,\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "          end: 2025-07-29T11:47:37.619Z\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        },\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        locale: 'und',\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        executionTimeMilliseconds: 26,\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "        notifications: [\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "          {\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "            severity: 'WARNING',\n"
2025-07-29 13:47:34.990 [warning] Failed to parse message: "            messageFormat: 'No bucket permissions for table %1$s.',\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "            messageFormatSpecifierTypes: [ 'TABLE_NAME' ],\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "            arguments: [ 'logs' ],\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "            notificationType: 'MISSING_BUCKET_PERMISSIONS',\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "            message: 'No bucket permissions for table logs.'\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "          }\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "        ],\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "        queryId: '424675ca-79e7-49ac-af68-6b4b50a53bae',\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "        sampled: false\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "      }\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "    }\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "  }\n"
2025-07-29 13:47:34.991 [warning] Failed to parse message: "}\n"

[MrManny]

Scopes: storage:events:read, storage:bizevents:read, storage:logs:read, davis-copilot:nl2dql:execute, davis-copilot:dql2nl:execute, davis-copilot:conversations:execute, davis:analyzers:read, app-engine:apps:run, app-engine:functions:run, automation:workflows:read, storage:metrics:read, storage:spans:read, storage:entities:read, storage:security.events:read, storage:system:read, storage:user.events:read, storage:user.sessions:read, settings:objects:read

Suspecting that storage:buckets:read might be missing.

Edit: yep, that was the one. Misleading error message from Grail imho.