Use glogger as remote logging infra

[Jongy]

Description

Closes: https://github.com/Granulate/gprofiler/issues/525

I didn't support this feature in A/B mode as suggested in the ticket, it's much changes than I expected and it'll be hard to maintain both infras in the code.

Some changes I did as part of it:

• state run_id and cycle_id are no longer added in logging level, then omitted in local logger - I add them in glogger level.

TODOs:

• [x] Call the handler.close() method of glogger - is it needed to ensure all logs are sent back? Or does glogger have some kind of finalization?

How Has This Been Tested?

• [x] NO_SERVER_LOG_KEY doesn't send the message to the server.
• [x] NO_SERVER_EXTRA_KEY doesn't send extras to the server.
• [x] NO_SERVER_LOG_KEY and NO_SERVER_EXTRA_KEY are not sent to server / printed locally.
• [x] plain messages are logged locally
• [x] plain messages marked with NO_SERVER_LOG_KEY are logged locally
• [x] messages with extras are logged locally with the extras.
• [x] messages with extras marked with NO_SERVER_EXTRA_KEY are logged locally with the extras.
• [x] messages with traceback logged locally with traceback
• [x] messages with traceback logged remotely with traceback
• [x] cycle_id passes to the backend properly
• [x] changing --glogger-server works

[d3dave]

reviewed all. just that one comment.

[Jongy]

Waiting on https://github.com/Granulate/gprofiler/pull/621 so CI is fixed.

[Jongy]

I need to fix the tests (some tests referenced the extras via the old way)

[Jongy]

@d3dave - about this note:

Call the handler.close() method of glogger - is it needed to ensure all logs are sent back? Or does glogger have some kind of finalization?

How does glogger handle it? It seems that messages are sent by some mechanism, before the process goes down.

[d3dave]

How does glogger handle it? It seems that messages are sent by some mechanism, before the process goes down.

logging registers an atexit handler that closes all stray handlers. You only need to do it if you need to flush before process exit.

[Jongy]

Made one final change - ready to merge now