Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WasmTestOnNodeJs-System.Threading.Channels.Tests failing with 'RuntimeError: invalid index into function table' #65016

Closed
runfoapp bot opened this issue Feb 8, 2022 · 10 comments
Labels
arch-wasm WebAssembly architecture

Comments

@runfoapp
Copy link

runfoapp bot commented Feb 8, 2022

Example console.log: https://helixre8s23ayyeko0k025g8.blob.core.windows.net/dotnet-runtime-refs-heads-main-c8efc49db304496b92/WasmTestOnNodeJs-System.Threading.Channels.Tests/1/console.6ce28db0.log?%3Fsv%3D2019-07-07%26se%3D2022-02-28T10%253A01%253A21Z%26sr%3Dc%26sp%3Drl%26sig%3D6kA2ZDGw6U4aObyqAunyjYSJ%252FvYVemwNYywEToUzifQ%253D

[10:23:15] fail: C:\helix\work\workitem\e\dotnet.js:12
...
[10:23:15] fail: RuntimeError: invalid index into function table
[10:23:15] fail:     at <anonymous>:wasm-function[1884]:0x833dd
[10:23:15] fail:     at <anonymous>:wasm-function[1883]:0x8338e

Runfo Tracking Issue: wasmtestonnodejs-system.threading.channels.tests work item

Build Definition Kind Run Name Console Core Dump Test Results Run Client
1597924 runtime-extra-platforms PR 64330 net7.0-Browser-Release-wasm-Mono_Release-WasmTestOnNodeJs-Ubuntu.1804.Amd64.Open console.log runclient.py
1596968 runtime-extra-platforms PR 64330 net7.0-Browser-Release-wasm-Mono_Release-WasmTestOnNodeJs-Ubuntu.1804.Amd64.Open console.log runclient.py

Build Result Summary

Day Hit Count Week Hit Count Month Hit Count
0 2 2
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Feb 8, 2022
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

@elinor-fung elinor-fung changed the title wasmtestonnodejs-system.threading.channels.tests work item WasmTestOnNodeJs-System.Threading.Channels.Tests failing with 'RuntimeError: invalid index into function table' Feb 8, 2022
@elinor-fung elinor-fung added the arch-wasm WebAssembly architecture label Feb 8, 2022
@ghost
Copy link

ghost commented Feb 8, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Example console.log: https://helixre8s23ayyeko0k025g8.blob.core.windows.net/dotnet-runtime-refs-heads-main-c8efc49db304496b92/WasmTestOnNodeJs-System.Threading.Channels.Tests/1/console.6ce28db0.log?%3Fsv%3D2019-07-07%26se%3D2022-02-28T10%253A01%253A21Z%26sr%3Dc%26sp%3Drl%26sig%3D6kA2ZDGw6U4aObyqAunyjYSJ%252FvYVemwNYywEToUzifQ%253D

[10:23:15] fail: C:\helix\work\workitem\e\dotnet.js:12
...
[10:23:15] fail: RuntimeError: invalid index into function table
[10:23:15] fail:     at <anonymous>:wasm-function[1884]:0x833dd
[10:23:15] fail:     at <anonymous>:wasm-function[1883]:0x8338e

Runfo Tracking Issue: wasmtestonnodejs-system.threading.channels.tests work item

Build Definition Kind Run Name Console Core Dump Test Results Run Client
1597924 runtime-extra-platforms PR 64330 net7.0-Browser-Release-wasm-Mono_Release-WasmTestOnNodeJs-Ubuntu.1804.Amd64.Open console.log runclient.py
1596968 runtime-extra-platforms PR 64330 net7.0-Browser-Release-wasm-Mono_Release-WasmTestOnNodeJs-Ubuntu.1804.Amd64.Open console.log runclient.py

Build Result Summary

Day Hit Count Week Hit Count Month Hit Count
1 2 2
Author: runfoapp[bot]
Assignees: -
Labels:

arch-wasm, untriaged

Milestone: -

@elinor-fung
Copy link
Member

Potentially related to #65012

@radical
Copy link
Member

radical commented Feb 9, 2022

The issue here is different from #65012

                 
[22:51:24] fail: /datadisks/disk1/work/AD3C0936/w/A69B08B4/e/test-main.js:97
                 
[22:51:24] fail: if (typeof globalThis.crypto === 'undefined') {
                 
[22:51:24] fail: ^
                 
[22:51:24] fail: 
                 
[22:51:24] fail: ReferenceError: globalThis is not defined
                 
[22:51:24] fail:     at Object.<anonymous> (/datadisks/disk1/work/AD3C0936/w/A69B08B4/e/test-main.js:97:1)
                 
[22:51:24] fail:     at Module._compile (internal/modules/cjs/loader.js:778:30)
                 
[22:51:24] fail:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
                 
[22:51:24] fail:     at Module.load (internal/modules/cjs/loader.js:653:32)
                 
[22:51:24] fail:     at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
                 
[22:51:24] fail:     at Function.Module._load (internal/modules/cjs/loader.js:585:3)
                 
[22:51:24] fail:     at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
                 
[22:51:24] fail:     at startup (internal/bootstrap/node.js:283:19)
                 
[22:51:24] fail:     at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
          
[22:51:24] info: Process node exited with 1

cc @pavelsavara @maraf

@radical radical removed the untriaged New issue has not been triaged by the area owner label Feb 9, 2022
@lewing
Copy link
Member

lewing commented Feb 9, 2022

node has defined globalThis since version 12.0.0 so if it is missing it has be set to undefined in that scope

@pavelsavara
Copy link
Member

pavelsavara commented Feb 9, 2022

I think we should not try to run on NodeJs lower than 12 and so I wonder how come we do.

@pavelsavara
Copy link
Member

Log in the main description is pointing to invalid index into function table but the Runfo links are globalThis is not defined

@pavelsavara
Copy link
Member

The globalThis is not defined is issue local to the #64330 and was probably fixed since, right @maraf ?

And runfo is confused here.

@maraf
Copy link
Member

maraf commented Feb 9, 2022

The globalThis is not defined is issue local to the #64330 and was probably fixed since, right @maraf?

Yes. Both mentioned builds are those two where I had a bug in extracting node from correlation payload zip and so all node tests failed.

@maraf
Copy link
Member

maraf commented Feb 9, 2022

Mentioned builds 1597924 and 1596968 are irrelevant. Main console.log is duplicate of #65012

@maraf maraf closed this as completed Feb 9, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Mar 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture
Projects
None yet
Development

No branches or pull requests

5 participants