apache / apache/incubator-pegasus

Test workflow for NodeJS client failed

Open
#2,277 0 comments 0 reactions 0 assignees View on GitHub
github
Dominant language
C++
Stars
2.1k
Forks
328
PR merge metrics
No merged PRs in 30d

Description

[Test workflow for NodeJS client](https://github.com/apache/incubator-pegasus/actions/workflows/test_nodejs-client.yml) failed due to following error:

```
Run ./recompile_thrift.sh
./recompile_thrift.sh
npm install
npm test
shell: bash --noprofile --norc -e -o pipefail {0}
env:
ARTIFACT_NAME: release_for_nodejs_client
Scanning /__w/incubator-pegasus/incubator-pegasus/idl/dsn.layer2.thrift for includes
Scanning /__w/incubator-pegasus/incubator-pegasus/idl/dsn.thrift for includes
Parsing /__w/incubator-pegasus/incubator-pegasus/idl/dsn.thrift for types
Parsing /__w/incubator-pegasus/incubator-pegasus/idl/dsn.layer2.thrift for types
Program: /__w/incubator-pegasus/incubator-pegasus/idl/dsn.layer2.thrift
Generating "js:node,with_ns"
Scanning /__w/incubator-pegasus/incubator-pegasus/idl/rrdb.thrift for includes
Scanning /__w/incubator-pegasus/incubator-pegasus/idl/dsn.layer2.thrift for includes
Scanning /__w/incubator-pegasus/incubator-pegasus/idl/dsn.thrift for includes
Parsing /__w/incubator-pegasus/incubator-pegasus/idl/dsn.thrift for types
Parsing /__w/incubator-pegasus/incubator-pegasus/idl/dsn.layer2.thrift for types
Scanning /__w/incubator-pegasus/incubator-pegasus/idl/dsn.thrift for includes
Parsing /__w/incubator-pegasus/incubator-pegasus/idl/dsn.thrift for types
Parsing /__w/incubator-pegasus/incubator-pegasus/idl/rrdb.thrift for types
Program: /__w/incubator-pegasus/incubator-pegasus/idl/rrdb.thrift
Generating "js:node,with_ns"
done
npm WARN deprecated cryptiles@2.0.5: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated boom@2.10.1: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated sntp@1.0.9: This module moved to @hapi/sntp. Please make sure to switch over as this distribution is no longer supported and may contain bugs and critical security issues.
npm WARN deprecated samsam@1.3.0: This package has been deprecated in favour of @sinonjs/samsam
npm WARN deprecated node-uuid@1.4.8: Use uuid module instead
npm WARN deprecated lodash.get@4.4.2: This package is deprecated. Use the optional chaining (?.) operator instead.
npm WARN deprecated q@1.0.1: You or someone you depend on is using Q, the JavaScript Promise library that gave JavaScript developers strong feelings about promises. They can almost certainly migrate to the native JavaScript promise now. Thank you literally everyone for joining me in this bet against the odds. Be excellent to each other.
npm WARN deprecated
npm WARN deprecated (For a CapTP with native promises, see @endo/eventual-send and @endo/captp)
npm WARN deprecated uuid@3.4.0: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.
npm WARN deprecated har-validator@5.1.5: this library is no longer supported
npm WARN deprecated hoek@2.16.3: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated socks@1.1.9: If using 2.x branch, please upgrade to at least 2.1.6 to avoid a serious bug with socket data flow and an import issue introduced in 2.1.0
npm WARN deprecated mailcomposer@4.0.1: This project is unmaintained
npm WARN deprecated buildmail@4.0.1: This project is unmaintained
npm WARN deprecated nodemailer@2.7.2: All versions below 4.0.1 of Nodemailer are deprecated. See https://nodemailer.com/status/
npm WARN deprecated mailgun-js@0.18.1: Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
npm WARN deprecated date-format@1.2.0: 1.x is no longer supported. Please upgrade to 4.x or higher.
npm WARN deprecated rimraf@2.6.3: Rimraf versions prior to v4 are no longer supported
npm WARN deprecated circular-json@0.3.3: CircularJSON is in maintenance only, flatted is its successor.
npm WARN deprecated streamroller@0.7.0: 0.x is no longer supported. Please upgrade to 3.x or higher.
npm WARN deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm WARN deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported
npm WARN deprecated request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated circular-json@0.5.9: CircularJSON is in maintenance only, flatted is its successor.
at ReplicaSession.onRpcReply (src/session.js:399:10)
at ReplicaSession. (src/session.js:312:14)
at Request. (src/connection.js:258:15)
at Request.emit (node:events:513:28)
at Request.callComplete (src/connection.js:394:10)
at Request.setException (src/connection.js:370:10)
at Connection._cleanupRequests (src/connection.js:151:17)
at Connection._handleClose (src/connection.js:112:10)
at Connection.call (src/connection.js:262:21)
at ReplicaSession.operate (src/session.js:314:21)
at Timeout._onTimeout (src/session.js:410:18)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)

2) test/client.test.js
get
simple get:
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/__w/incubator-pegasus/incubator-pegasus/nodejs-client/test/client.test.js)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)

3) test/client.test.js
multi get
simple multi get:
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/__w/incubator-pegasus/incubator-pegasus/nodejs-client/test/client.test.js)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)

4) test/client.test.js
multi get
multi get all sortKey-value:
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/__w/incubator-pegasus/incubator-pegasus/nodejs-client/test/client.test.js)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)

Error: Process completed with exit code 4.
```

Contributor guide

Open the contributing guide

Research direction

Start with the test_nodejs-client.yml workflow and its recompile_thrift.sh, npm install, and npm test steps. Reproduce the failure around nodejs-client/test/client.test.js, focusing on the ReplicaSession connection error and 10-second get and multi-get timeouts; done means the workflow completes successfully with these tests passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
ci-cd, testing-qa
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.