add health check sc for validators #2128

[Hitenjain14]

Changes

Added common method for both validator and blobber to send health check transaction.
Added healthcheck frequency to config/0chain_validator
Created new method type for VALIDATOR_HEALTH_CHECK in blobbercore/transaction/entity

Tests

Tasks to complete before merging PR:

• Ensure system tests are passing. If not Run them manually to check for any regressions 📋
• Do any new system tests need added to test this change? do any existing system tests need updated? If so create a PR at 0chain/system_test
• Merge your system tests PR to master AFTER merging this PR

Associated PRs (Link as appropriate):

• 0chain: add health check sc for validator
• gosdk:
• system_test:
• zboxcli:
• zwalletcli:
• Other: ...

[codecov-commenter]

Codecov Report

Merging #951 (207874e) into staging (ccd9e2f) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##           staging     #951      +/-   ##
===========================================
- Coverage    26.53%   26.44%   -0.10%     
===========================================
  Files           76       76              
  Lines         7638     7665      +27     
===========================================
  Hits          2027     2027              
- Misses        5345     5372      +27     
  Partials       266      266              

Flag	Coverage Δ
Unit-Tests	26.44% <0.00%> (-0.10%)	⬇️

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

Impacted Files	Coverage Δ
code/go/0chain.net/blobbercore/handler/health.go	0.00% <0.00%> (ø)
code/go/0chain.net/blobbercore/handler/protocol.go	0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[peterlimg]

Let's remove dead code

[peterlimg]

LGTM, but we will need to wait for the validator_health_check SC implemented on 0chain, before merging this one.

[Hitenjain14]

LGTM, but we will need to wait for the validator_health_check SC implemented on 0chain, before merging this one.

I have been working on it , just wanted to know if we can emit validator health check in DbHealthCheck or new table has to be created.

[peterlimg]

LGTM

[service-0chain]

Manual system tests [success] with the following config

config	value
tests skipped	TRUE
system-tests	master
gosdk	NOT OVERRIDDEN
zboxcli	staging
zwalletcli	staging
0chain	staging
blobber	validator-health
authorizer	staging
0box	staging
0block	staging
0dns	staging
explorer	master
0proxy	staging
0search	staging