[Bug]: asm.js type error in latest 1.61.0 version

[GrzegorzDrozd]

What happened?

Entering trace compare page produces error in javascript console.
I am unable to pan, zoom or do anything in the graph.

Steps to reproduce

1. enter trace comparison page.
2. see error in the javascript console

Expected behavior

Do not see errors in js console.

Relevant log output

asm.js type error: expecting argument type declaration for 'A' of the form 'arg = arg|0' or 'arg = +arg' or 'arg = fround(arg)' at line 1, character: 1546428

Screenshot

Firefox, after pretty printing, highlights this line:

Additional context

No response

Jaeger backend version

1.61.0

SDK

Docker compose:

 inst-s-jaeger:
    image: jaegertracing/all-in-one:1.61.0
    environment:
      COLLECTOR_ZIPKIN_HOST_PORT: 9412
    ports:
      - "9412:9412"
      - "14317:4317"
      - "14318:4318"
      - "14268:14268"
      - "14250:14250"
      - "16686:16686"`

Pipeline

No response

Stogage backend

No response

Operating system

No response

Deployment model

docker compose

Deployment configs

No response

[yurishkuro]

did it work in the previous version?

[GrzegorzDrozd]

Hi @yurishkuro

I was unable to make 1.60 work. I could not make it accept traces from open telemetry collector :( it works like a charm with 1.61.0 (ingestion).

[madmecodes]

Hi @GrzegorzDrozd is this issue resolved, now?

[hostaway-greg]

Hi,

I finally got back to this. I see issue on 2.1.0 in both Firefox and on Chrome:


Attached is ams.js file from chrome. Please note that it looks like Chrome added new lines, and they might not match the error message…
asm.txt

[yurishkuro]

Confirmed the issue w/ v2.1.0

@anshgoyalevil any ideas what could be causing this? The error seems all over Google, but I don't see a common root cause.

[anshgoyalevil]

This is caused by viz.js as per development console.

The viz.js version used by Plexus might be deprecated and using Node 12, which according to other repositories is harmless.

This error doesn't seem to be related to the zoom/pan not working on the TraceDiff page.

[yurishkuro]

@anshgoyalevil thanks for the tips. We have an active PR to upgrade viz.js, so perhaps this error will go away. And for panning I will bring it up as part of the LFX project.

[GrzegorzDrozd]

I assume that Pan and zoom on trace compare fix did not make it to 2.2.0 ?
Edit: I just checked with 2.2.0 and it did not :(

[yurishkuro]

No , just missed it