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

Notify harvest timeout #49

Closed
cattivik66 opened this issue Apr 27, 2021 · 3 comments
Closed

Notify harvest timeout #49

cattivik66 opened this issue Apr 27, 2021 · 3 comments

Comments

@cattivik66
Copy link

Based on https://chiaforum.com/t/troubleshooting-failure-to-farm-with-lots-of-plots-due-to-harvester-30s-timeout/413/3 and Chia-Network/chia-blockchain#2651 , it seems there could be some troubles if, for any reason, the harvester takes too much time to reply.

2021-04-24T16:03:29.433 harvester chia.harvester.harvester: INFO 3 plots were eligible for farming 1c75d8d21c... Found 0 proofs. Time: 7.92976 s. Total 522 plots

The Time should be, usually, under 5 seconds. So in this example it would be nice to get a notification.
It this time is going around 15 seconds there is the risk to miss the rewards.
If it is 30 seconds than matematically any possible reward is lost, even if there were some eligible.

@martomi
Copy link
Owner

martomi commented Apr 27, 2021

There is already a check for that. You can also see it in the example notifications in the README.

The daily stats are also calculating the average and max time to respond over the past 24 hours.

Anyone is welcome to open a PR that also calculated the number of responses > 5 seconds. Alert notifications are currently sent for anything over 20 seconds because that's already risky late.

@martomi martomi closed this as completed Apr 27, 2021
martomi added a commit that referenced this issue Apr 27, 2021
@martomi
Copy link
Owner

martomi commented Apr 27, 2021

Thanks @cattivik66 for the suggestions, I've added a statistic for number of occasions > 5 seconds search times happen during the day.

martomi added a commit that referenced this issue Apr 27, 2021
martomi added a commit that referenced this issue Apr 27, 2021
martomi added a commit that referenced this issue Apr 27, 2021
martomi added a commit that referenced this issue Apr 27, 2021
@martomi
Copy link
Owner

martomi commented Apr 27, 2021

Example of the new daily summary about search times: (this is now in the newest release)

paste

jinnatar pushed a commit to jinnatar/chiadog that referenced this issue Feb 5, 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

No branches or pull requests

2 participants