Minor While I was adding some more tests to detect error logs coming from Node in the container, I stumbled into the following one:
info: Creating registration..
error: Failed to register: Unavailable: request to http://backend:3030/registration failed, reason: connect ECONNREFUSED 172.17.0.3:3030
at new Unavailable (/evergreen/client/node_modules/@feathersjs/errors/lib/index.js:191:17)
at toError (/evergreen/client/node_modules/@feathersjs/rest-client/lib/base.js:9:11)
at <anonymous>
at process._tickCallback (internal/process/next_tick.js:188:7)
(node:32) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): Unavailable: request to http://backend:3030/registration failed, reason: connect ECONNREFUSED 172.17.0.3:3030
(node:32) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
So I'm going to add a workaround to the test for now, and maybe this is already fixed in the ongoing upgrade-level PR, but I thought I'd file this to make sure we do not forget.
- links to
[JENKINS-51328] Fix Registration error happening in the container
| Status | Original: Open [ 1 ] | New: In Progress [ 3 ] |
| Remote Link | New: This issue links to "#80 (Web Link)" [ 20677 ] |
| Status | Original: In Progress [ 3 ] | New: In Review [ 10005 ] |
| Resolution | New: Fixed [ 1 ] | |
| Status | Original: In Review [ 10005 ] | New: Resolved [ 5 ] |
| Labels | Original: essentials | New: essentials evergreen |
| Labels | Original: essentials evergreen | New: evergreen |