Skip to content

zetacored : tss migration migrators check using consensus type #2555

@kingpinXD

Description

@kingpinXD

Currently we check for the length of migrators list against the number of supported chains .
This is correct , as every chain should have its own migratory before we update the TSS address .

However presently, we have Solana added as a chain but the Solana cctx lifecycle is still WIP.

It would make sense to refactor the original check and make it based on consensus types ETH and Bitcoin

	supportedChains := append(k.zetaObserverKeeper.GetSupportedChainsByConsensus(ctx, chains.Consensus_ethereum),
		k.zetaObserverKeeper.GetSupportedChainsByConsensus(ctx, chains.Consensus_bitcoin)...)
	// Each connected chain should have its own tss migrator
	if len(supportedChains) != len(tssMigrators) {
		return nil, errorsmod.Wrap(
			types.ErrUnableToUpdateTss,
			"cannot update tss address incorrect number of migrations have been created and completed",
		)
	}

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions