Skip to content

Conversation

@cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Jan 18, 2020

This test provides missing coverage for __wasi_path_symlink() and __wasi_path_readlink().

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Jan 18, 2020
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@cjihrig
Copy link
Contributor Author

cjihrig commented Jan 19, 2020

CI is yellow with the exception of the things related to the failing git-nodesource-update-reference job.

@cjihrig cjihrig added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 19, 2020
@Trott
Copy link
Member

Trott commented Jan 20, 2020

CI is yellow with the exception of the things related to the failing git-nodesource-update-reference job.

I marked the failing node as offline, notified rvagg, and resumed your CI job.... 🤞

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@cjihrig
Copy link
Contributor Author

cjihrig commented Jan 20, 2020

Now it looks like everything is passing, but the Java post processing is failing.

20:06:27 FATAL: Remote call on JNLP4-connect connection from 203.217.18.114/203.217.18.114:41002 failed 20:06:27 Also: hudson.remoting.Channel$CallSiteStackTrace: Remote call to JNLP4-connect connection from 203.217.18.114/203.217.18.114:41002 20:06:27 at hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1741) 20:06:27 at hudson.remoting.UserRequest$ExceptionResponse.retrieve(UserRequest.java:356) 20:06:27 at hudson.remoting.Channel.call(Channel.java:955) 20:06:27 at hudson.Launcher$RemoteLauncher.kill(Launcher.java:1086) 20:06:27 at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:510) 20:06:27 at hudson.model.Run.execute(Run.java:1853) 20:06:27 at hudson.matrix.MatrixRun.run(MatrixRun.java:153) 20:06:27 at hudson.model.ResourceController.execute(ResourceController.java:97) 20:06:27 at hudson.model.Executor.run(Executor.java:427) 20:06:27 java.lang.NoClassDefFoundError: Could not initialize class hudson.slaves.SlaveComputer 20:06:27 at hudson.util.ProcessTree.get(ProcessTree.java:432) 20:06:27 at hudson.Launcher$RemoteLauncher$KillTask.call(Launcher.java:1103) 20:06:27 at hudson.Launcher$RemoteLauncher$KillTask.call(Launcher.java:1094) 20:06:27 at hudson.remoting.UserRequest.perform(UserRequest.java:212) 20:06:27 at hudson.remoting.UserRequest.perform(UserRequest.java:54) 20:06:27 at hudson.remoting.Request$2.run(Request.java:369) 20:06:27 at hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:72) 20:06:27 at java.util.concurrent.FutureTask.run(FutureTask.java:266) 20:06:27 at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) 20:06:27 at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) 20:06:27 at hudson.remoting.Engine$1.lambda$newThread$0(Engine.java:93) 20:06:27 at java.lang.Thread.run(Thread.java:748) 20:06:27 Caused: java.io.IOException: Remote call on JNLP4-connect connection from 203.217.18.114/203.217.18.114:41002 failed 20:06:27 at hudson.remoting.Channel.call(Channel.java:961) 20:06:27 at hudson.Launcher$RemoteLauncher.kill(Launcher.java:1086) 20:06:27 at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:510) 20:06:27 at hudson.model.Run.execute(Run.java:1853) 20:06:27 at hudson.matrix.MatrixRun.run(MatrixRun.java:153) 20:06:27 at hudson.model.ResourceController.execute(ResourceController.java:97) 20:06:27 at hudson.model.Executor.run(Executor.java:427) 20:06:27 [Text Finder] Looking for pattern '^not ok' in the files at '*.tap'

@Trott
Copy link
Member

Trott commented Jan 20, 2020

Now it looks like everything is passing, but the Java post processing is failing.

Marked more hosts offline... 😞

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jan 20, 2020

This test provides missing coverage for __wasi_path_symlink()
and __wasi_path_readlink().

PR-URL: nodejs#31403
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: David Carlier <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@cjihrig cjihrig merged commit 4f11fb6 into nodejs:master Jan 20, 2020
@cjihrig cjihrig deleted the wasi-symlinks branch January 20, 2020 16:04
codebytere pushed a commit that referenced this pull request Feb 17, 2020
This test provides missing coverage for __wasi_path_symlink()
and __wasi_path_readlink().

PR-URL: #31403
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: David Carlier <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@codebytere codebytere mentioned this pull request Feb 17, 2020
codebytere pushed a commit that referenced this pull request Mar 14, 2020
This test provides missing coverage for __wasi_path_symlink()
and __wasi_path_readlink().

PR-URL: #31403
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: David Carlier <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
codebytere pushed a commit that referenced this pull request Mar 17, 2020
This test provides missing coverage for __wasi_path_symlink()
and __wasi_path_readlink().

PR-URL: #31403
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: David Carlier <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@codebytere codebytere mentioned this pull request Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants