Alex Gartner

Results 26 issues of Alex Gartner

When restarting localnet, you must currently delete and recreate all containers. This is because we always attempt initialization every container start. We should allow users to restart their localnets when...

feature

We build our release binaries for ubuntu/glibc. This means that we cannot use our release binaries in the alpine docker containers. We should just use ubuntu everywhere for consistency.

ci

It seems like it's been very easy to break the upgrade tests. We need to run the upgrade tests in PR/mergequeue CI.

feature
ci

I expected bufio reader pools to not allocate any memory. But it actually [allocates a bunch of small buffers](https://github.com/golang/go/blob/master/src/bufio/bufio.go#L499-L501). Let's just allocate one big buffer since we know the exact...

Upgrade github.com/btcsuite/btcd. TODO: - [ ] restore HSM code - [ ] simplify key conversion logic - [ ] improve BIP 341/342 compliance by removing `txscript.NewCannedPrevOutputFetcher([]byte{}, 0)` - [ ]...

breaking:cli
no-changelog
UPGRADE_LIGHT_TESTS
PERFORMANCE_TESTS

Blocked by: https://github.com/zeta-chain/infrastructure/issues/1523

no-changelog
UPGRADE_LIGHT_TESTS

# Description https://github.com/zeta-chain/node/pull/2932 broke the tss migration tests because the V2 ERC20 custody contract did not have `UpdateTSSAddress` on the current protocol-contracts version (related to https://github.com/zeta-chain/protocol-contracts/pull/363). Use the latest version...

no-changelog
TSS_MIGRATION_TESTS

The V2 ERC20 Custody contract does not allow the TSS address to pause: https://github.com/zeta-chain/protocol-contracts/blob/475acfac26eff67472136fce2eb84f4dbfb5ea70/v2/contracts/evm/ERC20Custody.sol#L38-L49 The V1 ERC20 Custody contract only allows the TSS address to pause: https://github.com/zeta-chain/protocol-contracts/blob/main/v1/contracts/evm/ERC20Custody.sol#L118-L127 This means that...

bug

Convert `pkg/rpc.Clients` to an interface. Create a `pkg/rpc.MultiClients` which will automatically switch to a different client if it encounters a network error.

# Description All my changes were merged upstream. Continue using our fork to avoid docker hub rate limiting. Main impact is a continued reduction in G115 false-negatives and false-positives. We...

ci
no-changelog