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.

Sign up for GitHub

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

Jump to bottom

Estimate gas failed during using POW #1553

Closed
EvgeniyZZ opened this issue Aug 14, 2023 · 3 comments · Fixed by #1557, #1562 or #1569
Closed

Estimate gas failed during using POW #1553

EvgeniyZZ opened this issue Aug 14, 2023 · 3 comments · Fixed by #1557, #1562 or #1569
Assignees
@sergiy-skalelabs
Labels
bug Something isn't working release:2.2

Comments

@EvgeniyZZ
Copy link

EvgeniyZZ commented Aug 14, 2023 •
edited
Loading

Description
In some cases(most common - node rotation) node can have 0 balance of SFuel.
In this case should be used POW algorithm.

Environment
IMA Agent: 2.0.0-beta.7
Network: Regression
Schain name: portly-passionate-sirius

Step to reproduce

  • Run node rotation tests OR make node SFuel balance to 0.
  • Run any IMA transfer(S2M, S2S)
  • Observe logs

Actual result
IMA message verified and failed on tx processing.

Expected behavior
Estimate gas works properly during POW and tx processing

imaInsufficientSF.txt
@EvgeniyZZ EvgeniyZZ added bug Something isn't working release:2.2 labels Aug 14, 2023
@sergiy-skalelabs sergiy-skalelabs linked a pull request Aug 15, 2023 that will close this issue
Ticket 1553/estimate gas failed during pow #1557
Merged
@sergiy-skalelabs sergiy-skalelabs linked a pull request Aug 18, 2023 that will close this issue
develop-2-beta #1562
Merged
@EvgeniyZZ
Copy link
Author

Verified: Not working. Now its not working when balance is more 0.
IMA Agent: 2.0.0-beta.9

imaPowStillNotWorking.txt
imaPowStillNotWorkingPOW.txt

@sergiy-skalelabs sergiy-skalelabs linked a pull request Aug 29, 2023 that will close this issue
ticket-1553 Fixed frong non-BN argument of PoW mined transaction sent to skaled #1569
Merged
@DmytroNazarenko
Copy link
Contributor

DmytroNazarenko commented Sep 1, 2023 •
edited
Loading

IMA: 2.0.0-beta.10

@EvgeniyZZ
Copy link
Author

Verified: estimateGas wokrs during POW.
IMA Agent:
Node address balance: 0
PoW-mining is needed for sender 0x5ad0ffdb09bd8cc90ff73e489e7caa47e2483601 with balance 0x00 using required balance 0x01162d1d711400
powWorkNow.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sergiy-skalelabs sergiy-skalelabs

Labels
bug Something isn't working release:2.2
Projects
SKALE Engineering 🚀
Archived in project
Milestone
No milestone
4 participants
@DmytroNazarenko @sergiy-skalelabs @EvgeniyZZ @PolinaKiporenko