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

minor improvement on stratum server log for miner status #2301

Merged
merged 3 commits into from
Jan 6, 2019
Merged

minor improvement on stratum server log for miner status #2301

merged 3 commits into from
Jan 6, 2019

Conversation

garyyu
Copy link
Contributor

@garyyu garyyu commented Jan 6, 2019

In case of TUI off on server, it's not easy to read the miner worker status. So, I add this one line of log to show almost same thing as the status showed in TUI:

20190106 23:55:54.007 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 102, Rejected: 0, Stale: 0. Blocks Found: 0/0
20190106 23:56:25.004 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 114, Rejected: 0, Stale: 0. Blocks Found: 0/0
20190106 23:56:56.018 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 141, Rejected: 0, Stale: 0. Blocks Found: 0/1
20190106 23:57:26.971 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 168, Rejected: 0, Stale: 0. Blocks Found: 0/1
20190106 23:57:57.983 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 187, Rejected: 0, Stale: 0. Blocks Found: 0/1
20190106 23:58:28.993 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 208, Rejected: 0, Stale: 0. Blocks Found: 0/1
20190106 23:58:59.973 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 232, Rejected: 0, Stale: 0. Blocks Found: 0/1
20190106 23:59:30.972 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 250, Rejected: 0, Stale: 0. Blocks Found: 0/3
20190107 00:00:02.014 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 273, Rejected: 0, Stale: 0. Blocks Found: 0/3
20190107 00:00:32.984 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 293, Rejected: 0, Stale: 0. Blocks Found: 0/3
20190107 00:01:03.992 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 313, Rejected: 0, Stale: 0. Blocks Found: 0/4
20190107 00:01:34.968 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 335, Rejected: 0, Stale: 0. Blocks Found: 1/6
20190107 00:02:05.986 DEBUG grin_servers::mining::stratumserver - (Server ID: 0) Status of worker: 0 - Share Accepted: 356, Rejected: 0, Stale: 0. Blocks Found: 1/6

@ignopeverell
Copy link
Contributor

@bladedoyle is that going to break your pool?

@garyyu
Copy link
Contributor Author

garyyu commented Jan 6, 2019

@ignopeverell , no, it can't break, because I just log, no change on the message.

@hashmap
Copy link
Contributor

hashmap commented Jan 6, 2019

@garyyu it may, shares accounting is built on top of log parsing

@bladedoyle
Copy link
Contributor

Thanks @garyyu and @ignopeverell I appreciate the heads-up. I have not yet moved away from message parsing, but this isnt one of the messages I use so it will not impact me.

@garyyu
Copy link
Contributor Author

garyyu commented Jan 6, 2019

OK, thanks for your review~ then I go to merging now :)

@garyyu garyyu merged commit 657392b into mimblewimble:master Jan 6, 2019
@garyyu garyyu deleted the work-status-log branch January 6, 2019 23:19
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.

4 participants