Expose node component management

[janezpodhostnik]

I wanted to use the framework for composing a node that we have in the gateway without duplicating a lot of the code onflow/flow-evm-gateway#682

The changes I needed were:

• FlowNodeImp has *NodeConfig which I did not want to use so I pulled out a sub component NodeImp that does not have that. This forced me to change some log lines. All the info is still there, but it is in the fields instead of the message.
• moving the ctx, cancel := context.WithCancel(context.Background()) from Run to run is not strictly necessary, but it was a bit confusing so I refactored it (I can potentially put this in a separate PR)
• Next thing I needed was the handleComponent from FlowNodeBuilder so I changed it into a function that accepts components and a component builder + the input AddWorkersFromComponents. On Flow nodes the input is *NodeConfig, but with making ReadyDoneFactory generic the input can be anything.
• Context was added to Node.Run for ease of testing. Closing the context will trigger the shutdown of the node, just like a SIGQUIT
• Last change is strictly not related, and I can put it in a separate PR: I changed the metrics server into a component so that the context can be injected at the start of the metrics server.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 83.54430% with 13 lines in your changes missing coverage. Please review.

Project coverage is 41.01%. Comparing base (aedb8dc) to head (e6fb7ad).

Files with missing lines	Patch %	Lines
cmd/scaffold.go	95.34%	2 Missing ⚠️
cmd/access/main.go	0.00%	1 Missing ⚠️
cmd/collection/main.go	0.00%	1 Missing ⚠️
cmd/consensus/main.go	0.00%	1 Missing ⚠️
cmd/execution/main.go	0.00%	1 Missing ⚠️
cmd/ghost/main.go	0.00%	1 Missing ⚠️
cmd/node_builder.go	50.00%	1 Missing ⚠️
cmd/observer/main.go	0.00%	1 Missing ⚠️
cmd/verification/main.go	0.00%	1 Missing ⚠️
insecure/cmd/access/main.go	0.00%	1 Missing ⚠️
... and 2 more

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6769   +/-   ##
=======================================
  Coverage   41.00%   41.01%           
=======================================
  Files        2103     2103           
  Lines      184990   184995    +5     
=======================================
+ Hits        75862    75873   +11     
+ Misses     102764   102762    -2     
+ Partials     6364     6360    -4     

Flag	Coverage Δ
unittests	41.01% <83.54%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[peterargue]

looks good. one small naming suggestion.

Thank you for upgrading the metrics server. I think that should go in a separate PR though

[peterargue]

How about calling this initialize or construct?

[janezpodhostnik]

renamed. Let me know if it is better.

[zhangchiqing]

This change would cause every single line of log to include this two fields, is it necessary?
To me, the node role and spork id does not change after startup, so they only need to be logged once, as long as they can be found easily, we don't have to include them in other logs.

I would suggest to move the role and spork id info back to node startup complete log.

[janezpodhostnik]

this logger is used only in this file, which is like 4 log lines. I don't thing the impact is that big.