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

GHO CCIP 1.5.1 Upgrade Eth<>Arb #19

Closed
wants to merge 49 commits into from
Closed

GHO CCIP 1.5.1 Upgrade Eth<>Arb #19

wants to merge 49 commits into from

Conversation

DhairyaSethi
Copy link
Member

@DhairyaSethi DhairyaSethi commented Dec 11, 2024

Goal

Upgrade Existing ETH & ARB token pools to CCIP 1.5.1 by deploying new token pools, migrating liquidity to them & completely deprecating existing token pools.
This AIP aims to achieve this by:

  1. Accepting ownerships of:
    • new token pool
    • admin role for GHO token of CCIP TokenAdminRegistry
    • existing proxy pool (even though they'll be deprecated)
  2. Migrating Liquidity between old & new token pools:
    • ETH (LockRelease type): move tokens to new pool & update currentBridged
    • ARB (BurnMint type): Mint tokens on new pool, transfer to old pool, and burn tokens from old pool - this is done to offboard old pool as a facilitator and enable the new pool to offRamp existing bridged supply. (note that a facilitator can only burn (offRamp) amount which it has itself minted)
  3. Setup new token pool with rate limits of 300k capacity & 60 rate.

Notes:

  • sanity check no leftover tokens in existing eth token pool.
  • we can upgrade the existing ETH pool to reset bridgedAmount as well, but it's not needed since we reset bridgeLimit.
  • 1.5.1 TokenPools: CCIP 1.5.1 ccip#20
  • For 2.b: Since the existing burn mint type token pool does not have an explicit method to burn supply, a new permissioned method directBurn is introduced in DirectBurn on RemotePool ccip#21

Copy link

height bot commented Dec 11, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@miguelmtzinf miguelmtzinf self-requested a review December 31, 2024 10:17
dependabot bot and others added 6 commits January 2, 2025 08:55
…abs#556)

Bumps [lib/aave-helpers](https://github.com/bgd-labs/aave-helpers) from `1821d62` to `7cdcda6`.
- [Release notes](https://github.com/bgd-labs/aave-helpers/releases)
- [Commits](bgd-labs/aave-helpers@1821d62...7cdcda6)

---
updated-dependencies:
- dependency-name: lib/aave-helpers
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@miguelmtzinf miguelmtzinf self-requested a review January 3, 2025 10:17
@miguelmtzinf miguelmtzinf self-requested a review January 3, 2025 10:50
Rozengarden and others added 6 commits January 3, 2025 21:51
* feat: Gnosis update

* fix: osGNO pricefeed

* implement Part 1 without OsGNO

---------

Co-authored-by: Marc Zeller <[email protected]>
Co-authored-by: marczeller <[email protected]>
* feat: initial commit

* feat: finish tests

* chore: update readme
@miguelmtzinf miguelmtzinf self-requested a review January 6, 2025 13:51
@DhairyaSethi
Copy link
Member Author

closing in favour of bgd-labs#557

@DhairyaSethi DhairyaSethi reopened this Jan 8, 2025
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.

6 participants