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

Keep Alive Failure: out of gas in location: WritePerByte; gasWanted: 128769, gasUsed: 133474: out of gas #1305

Closed
taariq opened this issue Sep 28, 2024 · 5 comments
Assignees
Labels
bug Something isn't working Pigeon issues related with Pigeon service

Comments

@taariq
Copy link
Contributor

taariq commented Sep 28, 2024

What is happening?

Section description Provide as much context as you can. Give as much context as you can to make it easier for the developers to figure what is happening.

Keep Alive Failure: out of gas in location: WritePerByte; gasWanted: 128769, gasUsed: 133474: out of gas
See https://paloma.explorers.guru/transaction/B9FB761043481C3178CC5131630E94FB996555E529C2736C21E09ACE48C914F9?height=24462541

Paloma and pigeon versions and logs

Section description Write down paloma version. Write down pigeon version. Copy and paste pigeon config file as well as relevant ENV variables.

Paloma v2.2.2 and Pigeon 2.2.2

How to reproduce?

Section description Please write detailed steps of what you were doing for this bug to appear.

Paloma Pigeon sending Keep Alive messages

What is the expected behaviour?

Section description If you know, please write down what is the expected behaviour. If you don't know, that's ok. We can have a discussion in comments.

Keep alives do not run out of gas.

@taariq taariq added bug Something isn't working Pigeon issues related with Pigeon service labels Sep 28, 2024
@taariq taariq added this to Paloma Sep 28, 2024
@taariq
Copy link
Contributor Author

taariq commented Oct 2, 2024

Fixed with palomachain/pigeon#445

@taariq taariq closed this as completed Oct 2, 2024
@github-project-automation github-project-automation bot moved this to Done in Paloma Oct 2, 2024
@maharifu
Copy link
Contributor

maharifu commented Oct 3, 2024

@taariq This was not fixed in the PR. I will revert these changes in the next PR.

The gas-adjustment value is already configurable on pigeon's config.yaml. The default value is 3.0, but we should set it to be higher now. I'll adjust it in our validator.

@taariq taariq reopened this Oct 3, 2024
@taariq
Copy link
Contributor Author

taariq commented Oct 3, 2024

@maharifu do all validators need to adjust?

@maharifu
Copy link
Contributor

maharifu commented Oct 3, 2024

@maharifu do all validators need to adjust?

@taariq
Where is this error coming from? I assumed it was our own validator. If not, they need to check the paloma.gas-adjustment setting in pigeon config and set it to be a bit higher. Other validators might have a different setting altogether.

@taariq
Copy link
Contributor Author

taariq commented Oct 3, 2024

Good. I'll let the pigeons know. Thank you!

@taariq taariq closed this as completed Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Pigeon issues related with Pigeon service
Projects
Status: Done
Development

No branches or pull requests

3 participants