search

bitforge-me / beryllium

1 stars 9 forks source link

-add crown/bank balance to monitor info endpoint #97

Closed djpnewton closed 2 years ago

djpnewton commented 2 years ago

@eoliveros can you test this change with grafana/kapacitor

djpnewton commented 2 years ago

it works but there is a lot of:

[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance

is that not fixed by https://github.com/zap-me/beryllium/pull/96?

eoliveros commented 2 years ago

it works but there is a lot of:

[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO] :: calling balance..
[crown_financial INFO]    GET - https://api.mycrown.services/balance

is that not fixed by #96?

the quietening of logs applied on #96 appears to affect dasset.py. as far as i can tell, #96 doesn't affect crown_finance.py

djpnewton commented 2 years ago

ok try now

eoliveros commented 2 years ago

still getting the following every 15seconds:

[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO]    GET - https://api.mycrown.services/balance
[crown_financial INFO]    GET - https://api.mycrown.services/balance
djpnewton commented 2 years ago

ah ok try now

eoliveros commented 2 years ago

no more extra logs from crown.

got the following in grafana: image

had to adjust the dasset balances because of the change on monitor_endpoint.py and added the crown balance as well.

adjusted the tick script for dasset since renaming to exchange_{asset}_available and added a tick script for crown_balance with a threshold of 2k.

in my local server this is what grafana looks like: image