DeprecationWarning: worker.suicide is deprecated

[svipas]

Bug

2017-03-19 17:45:49: (node:4680) DeprecationWarning: worker.suicide is deprecated. Please use worker.exitedAfterDisconnect

Software versions used

OS         : Linux Ubuntu 16.04
node.js    : v7.7.3
PM2        : v2.4.2

PM2 Log output

2017-03-19 17:45:49: Starting execution sequence in -cluster mode- for app name:server id:0
2017-03-19 17:45:49: App name:server id:0 online
2017-03-19 17:45:49: Starting execution sequence in -cluster mode- for app name:server id:1
2017-03-19 17:45:49: (node:4680) DeprecationWarning: worker.suicide is deprecated. Please use worker.exitedAfterDisconnect.
2017-03-19 17:45:49: App name:server id:1 online
2017-03-19 17:45:49: Starting execution sequence in -cluster mode- for app name:server id:2
2017-03-19 17:45:49: (node:4680) DeprecationWarning: worker.suicide is deprecated. Please use worker.exitedAfterDisconnect.
2017-03-19 17:45:49: App name:server id:2 online
2017-03-19 17:45:49: Starting execution sequence in -cluster mode- for app name:server id:3
2017-03-19 17:45:49: (node:4680) DeprecationWarning: worker.suicide is deprecated. Please use worker.exitedAfterDisconnect.
2017-03-19 17:45:49: App name:server id:3 online
2017-03-19 17:45:49: (node:4680) DeprecationWarning: worker.suicide is deprecated. Please use worker.exitedAfterDisconnect.

[vmarchaud]

Are you sure that your daemon is up-to-date ? Try making a pm2 update and see if the warning is still there.

[svipas]

@vmarchaud PM version 2.4.2, did and still the same warning.

[sanpoChew]

can confirm, getting the same warning with pm2 2.4.2 and node 7.7.2

[jyotman]

Same here on Ubuntu 16.04!
pm2 2.4.4
node 7.9.0

[damianobarbati]

news on this?

[vmarchaud]

It's a warning, you can ignore it.

[Unitech]

we never use .suicide in pm2, I don't understand how this warning end up in logs

[isaacs]

Is it possible to run your app with the --trace-deprecation argument? Like node --trace-deprecation my-app.js This will show where the deprecated API is being used.

[vmarchaud]

@isaacs I read the PR over nodejs/node#13684, currently i am maintaining PM2.
But as Unitech said, we dropped the use of worker.suicide already few months back so we don't really understand from where it can be coming.
I actually run on the issue localy multiple times but ignored it, i'll try with --trace-deprecation tomorrow (don't hesitate to ping me i might forget ^^)

[isaacs]

@vmarchaud Any update on this? This is me pinging you because maybe you forgot :)

[vmarchaud]

Able to reproduce, it's coming from a dependency fclone, could you check it out @soyuka ?

2017-06-20 21:20:22: (node:30836) [DEP0007] DeprecationWarning: worker.suicide is deprecated. Please use worker.exitedAfterDisconnect.
    at fclone (/home/vmarchaud/keymetrics/pm2/node_modules/fclone/dist/fclone.js:74:32)
    at Object.clone (/home/vmarchaud/keymetrics/pm2/lib/Utility.js:60:12)
    at /home/vmarchaud/keymetrics/pm2/lib/God/ActionMethods.js:204:32
    at ready (/home/vmarchaud/keymetrics/pm2/lib/God.js:150:15)
    at Worker.<anonymous> (/home/vmarchaud/keymetrics/pm2/lib/God.js:197:18)
    at Object.onceWrapper (events.js:312:19)
    at emitNone (events.js:105:13)
    at Worker.emit (events.js:207:7)
    at online (internal/cluster/master.js:249:10)
    at Worker.onmessage (internal/cluster/master.js:236:5)

Corresponding to this line in the source

EDIT: Looks like its a little problem with the cloning of enumerable property.

[vmarchaud]

@isaacs I don't think we can actually avoid accessing worker.suicide because we can't just hardcode a key inside fclone to ignore it and we need to access it anyway to see if it's a function.
But anyway it should have no impact if removed since we don't use it.