search

Layr-Labs / avs-sync

Other
2 stars 2 forks source link

Log txhash when tx fails to help debug #13

Open samlaf opened 8 months ago

samlaf commented 8 months ago

Example error message is

2024-02-13T23:05:04.599Z        ERROR   logging/zap_logger.go:57        Error updating stakes of entire operator set for quorum {"err": "execution reverted", "quorum": 0}
github.com/Layr-Labs/eigensdk-go/logging.(*ZapLogger).Error
        /go/pkg/mod/github.com/!layr-!labs/eigensdk-go@v0.1.1-0.20240207213853-8a666597e0f1/logging/zap_logger.go:57
main.(*AvsSync).tryNTimesUpdateStakesOfEntireOperatorSetForQuorum
        /usr/src/app/avssync.go:157
main.(*AvsSync).updateStakes
        /usr/src/app/avssync.go:99
main.(*AvsSync).Start
        /usr/src/app/avssync.go:67
main.avsSyncMain
        /usr/src/app/main.go:115
github.com/urfave/cli.HandleAction
        /go/pkg/mod/github.com/urfave/cli@v1.22.14/app.go:524
github.com/urfave/cli.(*App).Run
        /go/pkg/mod/github.com/urfave/cli@v1.22.14/app.go:286
main.main
        /usr/src/app/main.go:28
runtime.main
        /usr/local/go/src/runtime/proc.go:267

We only see "tx reverted", which makes it hard to debug. We should print the txhash of the failed tx so that we can look it up on etherscan.

samlaf commented 8 months ago

This issue will be fixed once we merge https://github.com/Layr-Labs/eigensdk-go/pull/114/files#r1488814502 and update the eigensdk version of avs-sync. :)