Skip to content
This repository has been archived by the owner on Jul 13, 2022. It is now read-only.

Add HTTP Health Check Server #511

Merged
merged 26 commits into from
Aug 27, 2020
Merged

Add HTTP Health Check Server #511

merged 26 commits into from
Aug 27, 2020

Conversation

GregTheGreek
Copy link
Member

@GregTheGreek GregTheGreek commented Aug 19, 2020

Changes

  • Implements an HTTP based healthcheck
    • Enabled with --metrics, configure port with --metricsPort
    • Served on /health, provides status of each chain (height, last updated)
    • Returns error if timeout occurs (presently 120 sec)

Closes: #503

GregTheGreek and others added 3 commits August 19, 2020 21:05
* prints json strings for generated configs

* removed unnessessary output files

* go fmt
* isolated bootstrap tests

* go fmt

* make linter happy
metrics/http.go Outdated Show resolved Hide resolved
metrics/http.go Outdated Show resolved Hide resolved
metrics/http.go Outdated Show resolved Hide resolved
metrics/http.go Outdated Show resolved Hide resolved
metrics/metrics.go Outdated Show resolved Hide resolved
Co-authored-by: Miguel Hervas <miguel.hervas.lazaro@gmail.com>
@GregTheGreek GregTheGreek changed the title Greg/backport/metrics Add HTTP Metrics Server Aug 20, 2020
* refactoring files

* changing package names

* adding licenses to files

* edit comment

* adding missing method

* removing unused imports
@GregTheGreek GregTheGreek changed the title Add HTTP Metrics Server Add HTTP Health Check Server Aug 20, 2020
Alex and others added 4 commits August 22, 2020 20:59
* refactoring files

* changing package names

* adding licenses to files

* edit comment

* adding missing method

* removing unused imports

* fixing linter errors for http.go

* fixing up linter

* removing extra import

* handling graceful shutdown
@GregTheGreek GregTheGreek changed the base branch from release/v1.0.3 to master August 24, 2020 17:36
@GregTheGreek
Copy link
Member Author

@ansermino updated

@GregTheGreek GregTheGreek marked this pull request as ready for review August 24, 2020 19:02
blockstore/blockstore.go Outdated Show resolved Hide resolved
blockstore/blockstore.go Outdated Show resolved Hide resolved
chains/ethereum/chain.go Outdated Show resolved Hide resolved
chains/ethereum/listener.go Outdated Show resolved Hide resolved
chains/substrate/chain.go Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
core/core.go Outdated Show resolved Hide resolved
monitor/health/health.go Outdated Show resolved Hide resolved
monitor/health/http.go Outdated Show resolved Hide resolved
monitor/health/http.go Outdated Show resolved Hide resolved
GregTheGreek and others added 5 commits August 25, 2020 13:55
Co-authored-by: David Ansermino <david@chainsafe.io>
Merge branch 'greg/backport/metrics' of github.com:ChainSafe/ChainBridgeV2 into greg/backport/metrics
@ansermino ansermino merged commit 6f3e121 into master Aug 27, 2020
@ansermino ansermino deleted the greg/backport/metrics branch August 27, 2020 14:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define HealthCheck and Monitoring for running bridge
4 participants