metrico / qryn

Polyglot Observability Stack. Lightweight & Drop-in compatible with Loki, Prometheus, Tempo, Pyroscope, Opentelemetry, Datadog & more! WASM powered ⭐️ Star to Support
https://qryn.dev
GNU Affero General Public License v3.0
1.05k stars 63 forks source link

Tempo request raises out of bound error #479

Open deathalt opened 3 months ago

deathalt commented 3 months ago

Tempo request like

{span.service.name = "svc" && span.name =~"api" && span.deployment.environment = "prod"}

grafana return

Query error Error (memory access out of bounds). Please check the server logs for more details.

and message above in qryn logs

│ qryn-read-5c494b6f7d-269fl {"level":50,"time":1710853442666,"pid":19,"hostname":"qryn-read-5c494b6f7d-269fl","name":"qryn","reqId":"req-8a","err":"memory access out of bounds\nRuntimeError: memory access out of bounds\n    at dealloc (wasm://wasm/00d │
│ 862ce:wasm-function[1349]:0x148520)\n    at dealloc.command_export (wasm://wasm/00d862ce:wasm-function[1383]:0x15a550)\n    at Object.dealloc (/app/wasm_parts/main.js:28:49)\n    at Ctx.destroy (/app/wasm_parts/main.js:200:47)\n    at module.exports. │
│ TranspileTraceQL (/app/wasm_parts/main.js:139:18)\n    at search (/app/traceql/index.js:40:15)\n    at searchV2 (/app/lib/handlers/tempo_search.js:74:26)\n    at Object.handler (/app/lib/handlers/tempo_search.js:25:18)\n    at routeOptions.handler (/ │
│ app/node_modules/@fastify/websocket/index.js:150:24)\n    at preHandlerCallback (/app/node_modules/fastify/lib/handleRequest.js:137:37)","msg":"memory access out of bounds"}                                                                              │
│ qryn-read-5c494b6f7d-269fl {"level":30,"time":1710853442668,"pid":19,"hostname":"qryn-read-5c494b6f7d-269fl","name":"qryn","reqId":"req-8a","res":{"statusCode":500},"responseTime":44.54368597269058,"msg":"request completed"}                           │
│ qryn-read-5c494b6f7d-269fl /app/wasm_parts/wasm_exec.js:235                                                                                                                                                                                                │
│ qryn-read-5c494b6f7d-269fl                 return decoder.decode(new DataView(this._inst.exports.memory.buffer, ptr, len));                                                                                                                                │
│ qryn-read-5c494b6f7d-269fl                                                               ^                                                                                                                                                                 │
│ qryn-read-5c494b6f7d-269fl                                                                                                                                                                                                                                 │
│ qryn-read-5c494b6f7d-269fl TypeError: Cannot read properties of undefined (reading 'exports')                                                                                                                                                              │
│ qryn-read-5c494b6f7d-269fl     at loadString (/app/wasm_parts/wasm_exec.js:235:51)                                                                                                                                                                         │
│ qryn-read-5c494b6f7d-269fl     at syscall/js.valueGet (/app/wasm_parts/wasm_exec.js:314:18)                                                                                                                                                                │
│ qryn-read-5c494b6f7d-269fl     at (syscall/js.Value).Get (wasm://wasm/00d862ce:wasm-function[228]:0x1adf3)                                                                                                                                                 │
│ qryn-read-5c494b6f7d-269fl     at runtime.initAll (wasm://wasm/00d862ce:wasm-function[302]:0x3adaa)                                                                                                                                                        │
│ qryn-read-5c494b6f7d-269fl     at runtime.run$1 (wasm://wasm/00d862ce:wasm-function[301]:0x3ac12)                                                                                                                                                          │
│ qryn-read-5c494b6f7d-269fl     at runtime.run$1$gowrapper (wasm://wasm/00d862ce:wasm-function[215]:0x19890)                                                                                                                                                │
│ qryn-read-5c494b6f7d-269fl     at tinygo_launch (wasm://wasm/00d862ce:wasm-function[15]:0xe36)                                                                                                                                                             │
│ qryn-read-5c494b6f7d-269fl     at (*internal/task.Task).Resume (wasm://wasm/00d862ce:wasm-function[188]:0x16f75)                                                                                                                                           │
│ qryn-read-5c494b6f7d-269fl     at runtime.scheduler (wasm://wasm/00d862ce:wasm-function[216]:0x19b76)                                                                                                                                                      │
│ qryn-read-5c494b6f7d-269fl     at _start (wasm://wasm/00d862ce:wasm-function[214]:0x19825)                                                                                                                                                                 │
│ qryn-read-5c494b6f7d-269fl                                                                                                                                                                                                                                 │
│ qryn-read-5c494b6f7d-269fl Node.js v20.11.1 
akvlad commented 3 months ago

@deathalt qryn 3.2.15 passed all initial tests and the suggested request. please try the release.

lmangani commented 3 months ago

@deathalt please revalidate whenever viable and let us know if the patch was successful 🤞