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

feat: Add more logs around the driver balance fetching #2806

Closed
m-lord-renkse opened this issue Jul 9, 2024 · 1 comment
Closed

feat: Add more logs around the driver balance fetching #2806

m-lord-renkse opened this issue Jul 9, 2024 · 1 comment

Comments

@m-lord-renkse
Copy link
Contributor

Problem

The driver gets stuck fetching new balances (keeps sending outdated balances) and we do not know why it happens.

Suggested solution

Add more logs (debug/trace logs, we can enabled them on the fly) around the balance fetching in order to figure out where it fails/why it fails.

Alternatives considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context

For context: In order to allow solvers to “internalize” trades (trade with settlement balances instead of an AMM). For this we query the balance of relevant tokens and put them in the auction. These balances don’t get updated anymore. I.e. the balance etherscan reports is no longer identical to the balance we send in the /solve request.

Acceptance criteria

We add more logs which potentially will allow us debug better why the balances are stuck when it happens again.

Copy link

github-actions bot commented Sep 8, 2024

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

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

1 participant