log: [NPM] log any error and make sure node ID is set before first log

[huntergregory]

metrics.SendErrorLog() the error string when running NPM fails

Also, NPM node ID isn't set yet when the first metrics.SendLog() is send (message with: "starting NPM")

[github-actions]

This issue is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days

[github-actions]

Issue closed due to inactivity.

[huntergregory]

node ID can be unset at first because it is populated in the background here:

azure-container-networking/aitelemetry/telemetrywrapper.go

Line 195 in bbe5cb9

go getMetadata(th)

Based on NPM's ai config:

azure-container-networking/npm/metrics/ai-utils.go

Line 24 in bbe5cb9

aiConfig := aitelemetry.AIConfig{

[huntergregory]

metrics.SendErrorLog() the error string when running NPM fails

We seem to always call this function before exiting with error, except metrics.SendErrorLog() doesn't work as intended in a few places during bootup since telemetry is not initialized until later. See #3316 for more details