From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56144) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eJ4uE-00069S-WC for guix-patches@gnu.org; Sun, 26 Nov 2017 16:56:11 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eJ4uA-00013Y-4N for guix-patches@gnu.org; Sun, 26 Nov 2017 16:56:07 -0500 Received: from debbugs.gnu.org ([208.118.235.43]:51017) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eJ4uA-00013D-0C for guix-patches@gnu.org; Sun, 26 Nov 2017 16:56:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1eJ4u9-000736-Mh for guix-patches@gnu.org; Sun, 26 Nov 2017 16:56:01 -0500 Subject: [bug#29449] [PATCH] gnu: node: Update to 8.9.1. Resent-Message-ID: References: <877eudg6d7.fsf@gnu.org> <87zi79ueck.fsf@gnu.org> From: Jelle Licht In-reply-to: <87zi79ueck.fsf@gnu.org> Date: Sun, 26 Nov 2017 22:55:26 +0100 Message-ID: <87d144vh69.fsf@fsfe.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Ludovic =?UTF-8?Q?Court=C3=A8s?= Cc: 29449@debbugs.gnu.org Ludovic Courtès writes: > Hello Mike, > > Mike Gerwitz skribis: > >> Let me know if there is a better way to accomplish removing the >> `doc-only' invocation as part of the `check' target. It invokes npm and >> attempts to download `js-yaml', so I figured we may as well just skip it >> altogether since it isn't otherwise used. > > What you did LGTM. > >> From 918303981e0dd3212f1e0c1f1f87fc8ca608cc5e Mon Sep 17 00:00:00 2001 >> From: Mike Gerwitz >> Date: Sat, 25 Nov 2017 20:34:12 -0500 >> Subject: [PATCH] gnu: node: Update to 8.9.1. >> >> * gnu/packages/node.scm (node): Update to 8.9.1. >> [arguments]: Skip 'doc-only' target in 'check', which attempts to use >> npm to retrieve 'js-yaml' package. Remove test/doctool/test-make-doc.js. >> Do not remove now-missing test case. Remove new test case that fails in >> containers due to networking. > > The ‘check’ phase fails for me (on x86_64): > > --8<---------------cut here---------------start------------->8--- > [----------] 10 tests from InspectorSocketServerTest (88 ms total) > > [----------] Global test environment tear-down > [==========] 62 tests from 7 test cases ran. (3863 ms total) > [ PASSED ] 62 tests. > /gnu/store/zlbbayv8rv6z7gnhz435gjq8pzjm06v6-python-2.7.13/bin/python tools/test.py --mode=release -J \ > async-hooks \ > default \ > addons addons-napi \ > doctool \ > known_issues > === release test-http2-server-rst-stream === > Path: parallel/test-http2-server-rst-stream > (node:12213) ExperimentalWarning: The http2 module is an experimental API. > assert.js:42 > throw new errors.AssertionError({ > ^ > > AssertionError [ERR_ASSERTION]: rstWithCancel is not match rstCode > at ClientHttp2Stream.req.on.common.mustCall (/tmp/guix-build-node-8.9.1.drv-0/node-v8.9.1/test/parallel/test-http2-server-rst-stream.js:55:14) > at ClientHttp2Stream. (/tmp/guix-build-node-8.9.1.drv-0/node-v8.9.1/test/common/index.js:533:15) > at emitOne (events.js:121:20) > at ClientHttp2Stream.emit (events.js:211:7) > at emit (internal/http2/core.js:140:8) > at _combinedTickCallback (internal/process/next_tick.js:141:11) > at process._tickCallback (internal/process/next_tick.js:180:9) > Command: out/Release/node /tmp/guix-build-node-8.9.1.drv-0/node-v8.9.1/test/parallel/test-http2-server-rst-stream.js > [10:05|% 100|+ 2010|- 1]: Done > make: *** [Makefile:218: test] Error 1 > phase `check' failed after 667.7 seconds > --8<---------------cut here---------------end--------------->8--- > > > Any ideas? Could it be a non-deterministic issue? > > Thanks, > Ludo’. I get a similar issue to Ludo: --8<---------------cut here---------------start------------->8--- Path: parallel/test-http2-server-rst-stream (node:13036) ExperimentalWarning: The http2 module is an experimental API. assert.js:42 throw new errors.AssertionError({ ^ AssertionError [ERR_ASSERTION]: rstWithProtocolError is not match rstCode at ClientHttp2Stream.req.on.common.mustCall (/tmp/guix-build-node-8.9.1.drv-0/node-v8.9.1/test/parallel/test-http2-server-rst-stream.js:55:14) at ClientHttp2Stream. (/tmp/guix-build-node-8.9.1.drv-0/node-v8.9.1/test/common/index.js:533:15) at emitOne (events.js:121:20) at ClientHttp2Stream.emit (events.js:211:7) at emit (internal/http2/core.js:140:8) at _combinedTickCallback (internal/process/next_tick.js:141:11) at process._tickCallback (internal/process/next_tick.js:180:9) Command: out/Release/node /tmp/guix-build-node-8.9.1.drv-0/node-v8.9.1/test/parallel/test-http2-server-rst-stream.js --8<---------------cut here---------------end--------------->8--- I also found a GitHub issue[0], stating that several other people have similar problems with this specific test. It also seems that commit a41cc020fd in the node git repo addresses this flaky test. We could either backport the fix until it is included in a node release proper, or disable the test for now. - Jelle [0]: https://github.com/nodejs/node/issues/16688