search

martomi / chiadog

A watch dog providing a peace in mind that your Chia farm is running smoothly 24/7.
MIT License
457 stars 121 forks source link

Support reward notifications for more scenarios #167

Closed dvic closed 3 years ago

dvic commented 3 years ago

I believe that if you follow the best practice of using a separate (offline) wallet (describe here https://github.com/Chia-Network/chia-blockchain/wiki/Good-Security-Practices-on-Many-Machines#keep-your-wallet-separate) you won't get notified (because the wallet logs won't contain the lines for the correct wallet)? Maybe it's a good idea to have a separate notification for when you have won and another one that watches the wallet logs?

martomi commented 3 years ago

I'll hijack that issue and generalise it a little so that we have the full context in the same place.

The XCH notifications currently work only in the following scenario:

Running full node + harvester + wallet on the same machine and receiving rewards on that same wallet.

These notifications are just a bonus on top of critical alerts in monitoring your harvesters, so resolving that won't be a priority from my side but contributions are welcome.

martomi commented 3 years ago

The low hanging fruit here is to bring back the "Found proof" notifications. I'll do that even though a found proof doesn't guarantee an actual reward. It's the best we can do that works for all possible setups.

dvic commented 3 years ago

Having the "found proof" notifications are enough for me for now. Thanks!

Ganbin commented 3 years ago

Hi,

Since we have the official pool protocol, finding proofs doesn't give much more info if we have found a block with a reward destination inna separated wallet.

The only usefull information in this case I have found in the debug log was this line : 

2021-07-24T16:01:42.245 full_node chia.full_node.full_node: INFO     Starting to make the unfinished block
2021-07-24T16:01:42.255 full_node chia.full_node.full_node: INFO     Made the unfinished block
2021-07-24T16:01:42.328 full_node chia.full_node.full_node: INFO     🍀 ️Farmed unfinished_block 409126291baa2debe79099cff716ac0aedc451354105ed2667dc1ddfed3d8be0, SP: 42, validation time: 0.04041004180908203, cost: 202394374

Could we consider to add such parser?

Ganbin commented 3 years ago

Actually the answer in https://github.com/martomi/chiadog/issues/252#issuecomment-878092188 would solve this issue.

Hope this will come soon :)