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 deploy-sc command fails on transfer erc-20 it says invalid tx hash #2399

Open
DSHIVAAY-23 opened this issue Aug 10, 2023 · 6 comments · May be fixed by #3110
Open

make deploy-sc command fails on transfer erc-20 it says invalid tx hash #2399

DSHIVAAY-23 opened this issue Aug 10, 2023 · 6 comments · May be fixed by #3110
Assignees

Comments

@DSHIVAAY-23
Copy link

make deploy-sc
go run ./scripts/deploy_sc/main.go .
2023-08-10T14:58:33.699+0530 INFO deploy_sc/main.go:42 connecting to Local L1: http://localhost:8545 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:33.699+0530 INFO deploy_sc/main.go:45 connected {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:33.703+0530 DEBUG deploy_sc/main.go:54 ETH Balance for 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266: 499949983819117399073 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:33.703+0530 DEBUG deploy_sc/main.go:57 Sending TX to deploy Counter SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:35.713+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0x823c90429e1e0c9b7b9f51cee3b7ce6e9ec0fb35f8f82f756c835f972321b6db {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:35.713+0530 DEBUG deploy_sc/main.go:62 Calling Increment method from Counter SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:37.721+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0x0b260ad0960827f909a61970cc2495d77bf24748c7117d70b146d6ad382a592c {"pid": 1430818, "version": "v0.1.0"}

2023-08-10T14:58:37.721+0530 DEBUG deploy_sc/main.go:70 Sending TX to deploy EmitLog SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:39.731+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0x57a9bd72a64ca0e8f79c2b8904483b3344a2cd767be17d615859950416ac5265 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:39.732+0530 DEBUG deploy_sc/main.go:75 Calling EmitLogs method from EmitLog SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:41.744+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0xf73f897f89362dd41651a894a96e77df0bc03ace61795922d5d9033feb00a815 {"pid": 1430818, "version": "v0.1.0"}

2023-08-10T14:58:41.744+0530 DEBUG deploy_sc/main.go:84 Sending TX to deploy ERC20 SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:43.753+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0x264a60a9359f0ef7796e3470fca994ab05b33223a8fa335b6c0ea92a26edee4f {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:43.753+0530 DEBUG deploy_sc/main.go:89 Sending TX to do a ERC20 mint {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:45.763+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0xab85653dcd5960e4f490024273a17c1a25298f8097ff5357b6e47872473a0613 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:45.763+0530 DEBUG deploy_sc/main.go:95 Sending TX to do a ERC20 transfer {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:45.771+0530 DEBUG deploy_sc/main.go:99 Sending invalid TX to do a ERC20 transfer {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:45.774+0530 DEBUG deploy_sc/main.go:102 Invalid ERC20 tx hash: 0xc6d9a8c24ce04be2810507e3a7c2c6d971d7b771a3a9be4ad4e403f17d7e74bd {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:47.777+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0x352f68ba38a0869739efe75e24a031941f4c98ad728dd9eea4f24adff677cc17 {"pid": 1430818, "version": "v0.1.0"}

2023-08-10T14:58:47.789+0530 DEBUG deploy_sc/main.go:112 Sending TX to deploy Storage SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:49.797+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0x271f357f6cc491f8ebe22e2a1bdeebafe60d894e7cb33b3ba3c5759f3a1dab34 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:49.797+0530 DEBUG deploy_sc/main.go:117 Calling Store method from Storage SC {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.805+0530 DEBUG operations/wait.go:99 Transaction successfully mined: 0xcbf514b68bdd82018b4c7f22f7271ff57517f90668681c243f6886b7ad519ff5 {"pid": 1430818, "version": "v0.1.0"}

2023-08-10T14:58:51.806+0530 DEBUG deploy_sc/main.go:126 ETH Balance for 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266: 499949983819108066553 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.806+0530 DEBUG deploy_sc/main.go:130 Transfer Amount: 249974991909554033276 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.806+0530 DEBUG deploy_sc/main.go:132 Sending TX to transfer ETH {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.806+0530 INFO deploy_sc/main.go:149 reading nonce for account: 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.807+0530 INFO deploy_sc/main.go:152 nonce: 39 {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.810+0530 INFO deploy_sc/main.go:168 sending transfer tx {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.810+0530 INFO deploy_sc/main.go:171 tx sent: 0xc0adb7614f0298d53cb4eaca52b9df8aae0c34052bd433ca6cbabd097bf65e0d {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.810+0530 INFO deploy_sc/main.go:176 tx rlp: 0xf86a270882520894617b3a3528f9cdd6630fd3301b9c8911f7bf063d890d8d1992b7ac1d127c80820a95a06c18cc207bc3765cc2509957037ef4c742a7962d918df43215c6dda9e8e55f19a0186680e743bd1d84862de54c2d6deca876b7d04b0c6b62157782e5efbb3088d8 {"pid": 1430818, "version": "v0.1.0"}

2023-08-10T14:58:51.810+0530 DEBUG deploy_sc/main.go:138 Sending Invalid TX to transfer ETH {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.811+0530 INFO deploy_sc/main.go:168 sending transfer tx {"pid": 1430818, "version": "v0.1.0"}
2023-08-10T14:58:51.812+0530 FATAL deploy_sc/main.go:183 transaction would cause overdraft
/data/zkevm-node/log/log.go:140 github.com/0xPolygonHermez/zkevm-node/log.appendStackTraceMaybeArgs()
/data/zkevm-node/log/log.go:221 github.com/0xPolygonHermez/zkevm-node/log.Fatal()
/data/zkevm-node/test/scripts/deploy_sc/main.go:183 main.chkErr()
/data/zkevm-node/test/scripts/deploy_sc/main.go:170 main.ethTransfer()
/data/zkevm-node/test/scripts/deploy_sc/main.go:140 main.main()
/home/user/.go/src/runtime/proc.go:250 runtime.main()
{"pid": 1430818, "version": "v0.1.0"}
main.chkErr
/data/zkevm-node/test/scripts/deploy_sc/main.go:183
main.ethTransfer
/data/zkevm-node/test/scripts/deploy_sc/main.go:170
main.main
/data/zkevm-node/test/scripts/deploy_sc/main.go:140
runtime.main
/home/user/.go/src/runtime/proc.go:250
exit status 1
make: *** [Makefile:482: deploy-sc] Error 1

@QEDK
Copy link

QEDK commented Aug 15, 2023

Same issue.

@ToniRamirezM ToniRamirezM self-assigned this Aug 17, 2023
@ToniRamirezM
Copy link
Contributor

ToniRamirezM commented Aug 18, 2023

Hello,

Thank you for reporting the issue.

"transaction would cause overdraft" is a error from a check introduced after the creation of this test script, and it looks like it is causing issues to more people:

ethereum/go-ethereum#27274

As you can see in the logs this is causing the script to fail:

2023-08-10T14:58:51.812+0530 FATAL deploy_sc/main.go:183 transaction would cause overdraft

We will update the deployment script to take into account this check.

@DSHIVAAY-23
Copy link
Author

@ToniRamirezM Thank you for the update looking for upadate deployment script

@abhijeetbhagat
Copy link

got hit with the same problem.

@mc2eqe
Copy link

mc2eqe commented Oct 18, 2023

Same problem here. Any news on this? Thank you.

@rhz1949
Copy link

rhz1949 commented Oct 26, 2023

If any update for this problem? thx

HudsonGraeme added a commit to inference-labs-inc/zkevm-node that referenced this issue Jan 19, 2024
@HudsonGraeme HudsonGraeme linked a pull request Jan 19, 2024 that will close this issue
HudsonGraeme added a commit to inference-labs-inc/zkevm-node that referenced this issue Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants