Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement bridge stats #82

Merged
merged 11 commits into from
Aug 7, 2023
Merged

Implement bridge stats #82

merged 11 commits into from
Aug 7, 2023

Conversation

DNK90
Copy link
Contributor

@DNK90 DNK90 commented Jul 23, 2023

Implement logics to send processedBlock, last errors, pending/failed tasks to ronin stats

@DNK90 DNK90 requested a review from minh-bq July 23, 2023 08:11
listener/types.go Show resolved Hide resolved
listener/types.go Show resolved Hide resolved
stats/bridge_stats.go Outdated Show resolved Hide resolved
stats/bridge_stats.go Show resolved Hide resolved
stats/bridge_stats.go Outdated Show resolved Hide resolved
stats/bridge_stats.go Outdated Show resolved Hide resolved
@DNK90 DNK90 requested a review from minh-bq July 24, 2023 07:17
stats/bridge_stats.go Outdated Show resolved Hide resolved
stats/bridge_stats.go Outdated Show resolved Hide resolved
* fix compile error

* Remove unused code

* Rename from pass to secret

* Add log for tracking


* Add for tracking report process

* Add read loop

* Rework ronin stats logic

* Remove lock

* Rework readLoop

* Add ping handler from server

* Reduce ticker from 10 to 5s

* Remove unused locking, add handle signal quit, and move functionality to a separate function

* Fix typo and create a constant

* Remove lock in Service struct
Copy link
Contributor

@huyngopt1994 huyngopt1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Already tested.

@DNK90 DNK90 merged commit 4f21bd6 into master Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants