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

fix(cmd/paratime): Fix proposer stats #149

Merged
merged 1 commit into from Oct 28, 2023

Conversation

peternose
Copy link
Contributor

@peternose peternose commented Oct 27, 2023

Ensure that a node is not counted twice as a proposer when it serves the roles of both a worker and a backup worker.

Testnet:

./oasis paratime statistics 17993756 17993757

Before:

|                  ENTITY ADDR                   | PRIMARY | BACKUP | PROPOSER |
|------------------------------------------------|---------|--------|----------|
| oasis1qphtg2256r8q3ljs6j5yqg0em8t62ycv9q7yysls |       1 |      1 |        2 |

After:

|                  ENTITY ADDR                   | PRIMARY | BACKUP | PROPOSER |
|------------------------------------------------|---------|--------|----------|
| oasis1qphtg2256r8q3ljs6j5yqg0em8t62ycv9q7yysls |       1 |      1 |        1 |

@peternose peternose changed the title Fix proposer stats fix(cmd/paratime): Fix proposer stats Oct 27, 2023
@peternose peternose force-pushed the peternose/bugfix/fix-proposer-stats branch from 22eaf61 to c0eaabb Compare October 27, 2023 20:08
@peternose peternose added bug Something isn't working paratime ParaTime (runtime) related CLI issues go Pull requests that update Go code labels Oct 27, 2023
@peternose peternose requested a review from matevz October 27, 2023 20:16
@peternose peternose merged commit ae3e43d into master Oct 28, 2023
3 checks passed
@peternose peternose deleted the peternose/bugfix/fix-proposer-stats branch October 28, 2023 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working go Pull requests that update Go code paratime ParaTime (runtime) related CLI issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants