broadcast node's height info into p2p network

[envestcc]

Description

Set up a mechanism to pull other delegate nodes info if a delegate node want , in order to have a better understanding of nodes status on the chain.

• node will broadcast a RequestNodeInfoMessage to notify other nodes if it's configured a valid broadcast interval
• node will unicast a ResponseNodeInfoMessage with signature to the sender when recieve the request
• node will verify the ResponseNodeInfoMessage and then update metric

Design: p2p network monitor message design

Fixes #3724

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• make test
• TestNewDelegateManager
• TestDelegateManager_HandleNodeInfo
• TestDelegateManager_RequestNodeInfo
• TestDelegateManager_TellNodeInfo

Test Configuration:

• Firmware version:
• Hardware:
• Toolchain:
• SDK:

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[codecov]

Codecov Report

Merging #3744 (37fdaaa) into master (263ff82) will increase coverage by 0.99%.
The diff coverage is 80.52%.

@@            Coverage Diff             @@
##           master    #3744      +/-   ##
==========================================
+ Coverage   74.47%   75.47%   +0.99%     
==========================================
  Files         269      303      +34     
  Lines       23925    25817    +1892     
==========================================
+ Hits        17818    19485    +1667     
- Misses       5174     5315     +141     
- Partials      933     1017      +84     

Impacted Files	Coverage Δ
action/actctx.go	90.62% <ø> (+6.25%)	⬆️
action/action.go	82.35% <ø> (ø)
action/action_deserializer.go	57.14% <ø> (ø)
action/candidate_update.go	88.88% <0.00%> (-4.14%)	⬇️
action/consignment_transfer.go	90.41% <ø> (ø)
action/const.go	100.00% <ø> (ø)
action/execution.go	83.78% <ø> (ø)
action/grantreward.go	68.57% <ø> (ø)
action/protocol/account/protocol.go	86.90% <ø> (ø)
action/protocol/dock.go	100.00% <ø> (ø)
... and 282 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[Liuhaai]

broadcastTask could be nil when EnableBroadcastNodeInfo is false

[dustinxie]

right, i left same comment earlier

[envestcc]

remain the task has an advantage that a new delegate can enable broadcast automatically without restart

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
6 Code Smells

No Coverage information
0.0% Duplication