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

Sync upstream v0.6.2 #170

Closed
wants to merge 19 commits into from
Closed

Sync upstream v0.6.2 #170

wants to merge 19 commits into from

Conversation

debaghtk
Copy link
Collaborator

@debaghtk debaghtk commented Mar 7, 2024

Why this should be merged

this PR is to merge subnet-evm v0.6.2. There were a bunch of resolve conflicts due to change in gossip mechanism introduced in avalanche-go which impacted coreth as well as subnet-evm

How this works

this PR only resolves the merge conflicts and keeps the original order gossip logic untouched in an attempt to simplify things. The order gossip logic is also prefixed with legacy. The order gossip logic has been updated in this PR

Context

avalanche had a downtime - link
patch released in avalanche go - link
push gossip redesign in avalanche go - link
changes coming into hubblenet from subnet-evm - link

How this was tested

tested locally

Items Pending

  • merge my commits in one single commit to preserve history from avax devs

  • decide on config of validator , non validator, peer for app gossip

aaronbuchwald and others added 7 commits February 28, 2024 12:41
* e2e: Switch warp test to use tmpnet fixture

* e2e: Switch load test to use the tmpnet fixture

* Add readme describing use of existing network

* e2e: Remove network manager and dependency on ANR

* fixup: Switch from VMName to VMID with tmpnet.Chain

* fixup: Respond to review comments

* fixup: Use 5 nodes for load testing

* fixup: Fix shellcheck errors

* cleanup artifact dirs

* fixup: Update to reflect tmpnet configuration changes

* fixup: go mod tidy

* fixup: Use the default evm logging level for load testing

* fixup: Lower the avalanchego logging levels for load testing

* fixup: respond to review comments

* fixup: Respond to review comments #2

* fixup: Disable unnecessary stdout logging

---------

Co-authored-by: Ceyhun Onur <[email protected]>
* remove allowlist events

* add signature to topic count
* sync coreth v0.13.1-rc.5

* fix tests

* fix upgrade bytes test

* bump subnet-evm version

* sort default configs

* sort default configs
* tresor, kitkat, berghain

* resolve debas comments

* chain config

* slight refactor

* resolve lumos comments
@debaghtk debaghtk marked this pull request as ready for review March 12, 2024 10:40
@debaghtk debaghtk force-pushed the sync-upstream-v0.6.2 branch from ec46047 to b0f895d Compare March 14, 2024 08:46
@atvanguard atvanguard force-pushed the aylin branch 2 times, most recently from 94dfbac to 7d58776 Compare March 15, 2024 06:19
@debaghtk debaghtk force-pushed the sync-upstream-v0.6.2 branch from 9b689cb to d6249e5 Compare March 18, 2024 06:30
@debaghtk debaghtk closed this Mar 18, 2024
@debaghtk debaghtk deleted the sync-upstream-v0.6.2 branch April 4, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants