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

Make sure we use a valid price table when funding an account #906

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

peterjan
Copy link
Member

@peterjan peterjan commented Jan 18, 2024

With this change I hope to get rid of the following error in the log. It does not happen a lot and it happens for the same host, causing me to believe it's simply due to the host being slow and connecting to it taking longer than whatever leeway we put on the price table validity.

{"level":"error","date":"2024-01-13T03:39:48Z","component":"autopilot.accounts","msg":"failed to fund account: couldn't fund account: failed to fund account with ~500.8 mS;FundAccount: ReadResponse: failed to read price table: no price table found\n","accountID":"ed25519:614c0fa737b04dc447d39c489c8911a9bfc614ec86897d90f0053c9be152cd3f","hostKey":"ed25519:fba406818f558676e117323f45665f3affbb10bb025f74548fafe66df0dd5274","balance":"499218792671999618136517","expected":"1 SC"}

Fixes #801

@peterjan peterjan self-assigned this Jan 18, 2024
@peterjan peterjan marked this pull request as ready for review January 18, 2024 12:47
worker/host.go Outdated Show resolved Hide resolved
worker/host.go Outdated Show resolved Hide resolved
@ChrisSchinnerl ChrisSchinnerl merged commit cef20e4 into dev Jan 19, 2024
6 checks passed
@ChrisSchinnerl ChrisSchinnerl deleted the pj/fund-account branch January 19, 2024 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Failed to sync ephemeral account when no price table found
2 participants