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

addon build failures on Windows machines #1622

Closed
joyeecheung opened this issue Dec 5, 2018 · 2 comments
Closed

addon build failures on Windows machines #1622

joyeecheung opened this issue Dec 5, 2018 · 2 comments

Comments

@joyeecheung
Copy link
Member

Refs: nodejs/reliability#19

Reason Error: C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe failed with exit code: 1
Type BUILD_FAILURE
Failed PR 7 (nodejs/node#24796, nodejs/node#24803, nodejs/node#24737, nodejs/node#24727, nodejs/node#24814, nodejs/node#20640, nodejs/node#24733)
Appeared test-azure_msft-win10-x64-5, test-rackspace-win2012r2-x64-12
First CI https://ci.nodejs.org/job/node-test-pull-request/19147/
Last CI https://ci.nodejs.org/job/node-test-pull-request/19190/
Example
Error: `C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (c:\workspace\node-test-binary-windows\deps\npm\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:219:12)
gyp ERR! System Windows_NT 10.0.10240

Reason Error: C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe failed with exit code: 3762504530
Type BUILD_FAILURE
Failed PR 7 (nodejs/node#24775, nodejs/node#24796, nodejs/node#24803, nodejs/node#20640, nodejs/node#23423, nodejs/node#24733, nodejs/ecmascript-modules#18)
Appeared test-azure_msft-win2016-x64-6
First CI https://ci.nodejs.org/job/node-test-pull-request/19124/
Last CI https://ci.nodejs.org/job/node-test-pull-request/19211/
Example
Error: `C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe` failed with exit code: 3762504530
gyp ERR! stack     at ChildProcess.onExit (c:\workspace\node-test-binary-windows\deps\npm\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:219:12)
gyp ERR! System Windows_NT 10.0.14393

The real cause seem to be either

(https://ci.nodejs.org/job/node-test-binary-windows/COMPILED_BY=vcbt2015,RUNNER=win10,RUN_SUBSET=0/22176/console)

MSBUILD : error MSB4132: The tools version "2.0" is unrecognized. Available tools versions are "4.0".
18:29:20 gyp ERR! build error 

Or (https://ci.nodejs.org/job/node-test-binary-windows/COMPILED_BY=vs2015,RUNNER=win2016,RUN_SUBSET=1/22176/console)

Microsoft.Build.BuildEngine.InvalidToolsetDefinitionException: MSBuildToolsPath is not specified for the ToolsVersion "14.0" defined at "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\MSBuild\ToolsVersions\14.0", or the value specified evaluates to the empty string.
@joaocgreis
Copy link
Member

joaocgreis commented Dec 11, 2018

This happens only when testing v6, because the tests run on machines with VS2017 installed and it is not supported by Node v6.

This was a recent regression with agent labels and VS versions. I've fixed it for now by leaving the affected machines out of node-test-binary-windows, but we'll have to find a better (and less confusing) way of labeling the machines and distributing the jobs.

@joyeecheung the only PRs I see afected by this are nodejs/node#24796 and nodejs/node#24803 (the only two testing v6). Unless I missed something, all others have PR related failures or unmarked flaky tests. I think these reports are automatically generated, so there might be something wrong with the tool. And thanks for reporting this!

@joyeecheung
Copy link
Member Author

Unless I missed something, all others have PR related failures or unmarked flaky tests. I think these reports are automatically generated, so there might be something wrong with the tool.

Yes, looks like the tool is still trying to query the skipped builds for failures (e.g. for https://ci.nodejs.org/job/node-test-binary-windows/22162/ the tool does not understand that win10-vcbt2015-1 is a skipped build and the subbuild does not actually belong to that job). I'll tweak the tool later and see if the Jenkins API has anything useful for the tool to understand that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants