search

golemfactory / yagna-triage

Repository for issues which we don't yet know about enough to assign to proper repo
1 stars 0 forks source link

yagna did a DoS on testnet geth #217

Closed etam closed 1 year ago

etam commented 2 years ago

Nginx requests received by our testnet geth: nginx_requests

It was caused by a checker running in the testnet subnet. Here's a copy of whole datadir (with databases and logs): https://drive.google.com/file/d/1lciB-pffjX_Q6PQSf7H6cAIhf3Gz9ERg/view?usp=sharing

It has tons of errors

ERROR ya_erc20_driver::erc20::wallet] Error sending transaction: GenericError { inner: "RPC error: Error { code: ServerError(-32000), message: \"insufficient funds for gas * price + value\", data: None }" }

and

INFO  ya_erc20_driver::driver::cron] Transaction not found on chain
etam commented 2 years ago

When trying to restart checker, I found that during payment fund it could send a tx that was rejected by geth. Later attempts to run payment fund were causing a line in logs "Already pending a mint transactin.".

johny-b commented 2 years ago

I've seen this once or twice last week (i.e. I've seen the error messages, I don't know about geth requests) so I'm not super-sure about impact: low.

I'll tell you if I see this again.

nieznanysprawiciel commented 1 year ago

@scx1332 is working on new payment driver implementation. It is not worth investigating it, if the problem won't occur after introducing changes