-
Notifications
You must be signed in to change notification settings - Fork 80
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
Signature aggregator sdk #2149
Signature aggregator sdk #2149
Changes from all commits
ce84bda
8255b4f
f910e6e
d34e429
193d908
99ea790
fba4092
604fcb2
61e3b3f
1dbd1aa
00f7ff9
6b7790e
182cc6f
3ebba77
dbb1ae2
2dc7610
c3c1504
f5dede3
69d40c7
d76ae7b
24727ec
89aef71
73e17d5
cbd4bf4
b8c1c62
5d380f5
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,6 +22,8 @@ import ( | |
"github.com/ava-labs/avalanche-cli/pkg/utils" | ||
"github.com/ava-labs/avalanche-cli/pkg/ux" | ||
"github.com/ava-labs/avalanchego/config" | ||
avago_upgrade "github.com/ava-labs/avalanchego/upgrade" | ||
avago_constants "github.com/ava-labs/avalanchego/utils/constants" | ||
"github.com/ava-labs/avalanchego/utils/crypto/bls" | ||
"github.com/ava-labs/avalanchego/utils/formatting" | ||
"github.com/ava-labs/avalanchego/utils/logging" | ||
|
@@ -41,7 +43,7 @@ const ( | |
|
||
func generateCustomCchainGenesis() ([]byte, error) { | ||
cChainGenesisMap := map[string]interface{}{} | ||
cChainGenesisMap["config"] = coreth_params.AvalancheLocalChainConfig | ||
cChainGenesisMap["config"] = coreth_params.GetChainConfig(avago_upgrade.GetConfig(avago_constants.LocalID), coreth_params.AvalancheLocalChainID) | ||
cChainGenesisMap["nonce"] = hexa0Str | ||
cChainGenesisMap["timestamp"] = hexa0Str | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. did you tested this genesis works? this is not the same solution as found for ANR:
but may work. You should check if teleporter msg works between c-chain and other blockchain. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. it's after update to latest coreth There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. could you check that a devnet created with this works with teleporter msg? |
||
cChainGenesisMap["extraData"] = "0x00" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is create devnet being used anywhere in this PR? just curious why this is changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's changed because of the dependency upgrade and changes there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did you tested this genesis works? this is not the same solution as found for ANR:
but may work. You should check if teleporter msg works between c-chain and other blockchain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
its works after we updated coreth to the latest go mod version because awm-relayer uses this version of the coreth