Skip to content

Remove the Convex AMO strategy from the OETH Vault#2907

Open
clement-ux wants to merge 2 commits into
masterfrom
clement/remove-oeth-convex-amo
Open

Remove the Convex AMO strategy from the OETH Vault#2907
clement-ux wants to merge 2 commits into
masterfrom
clement/remove-oeth-convex-amo

Conversation

@clement-ux
Copy link
Copy Markdown
Collaborator

@clement-ux clement-ux commented May 29, 2026

Summary

Removes the Convex OETH/ETH AMO strategy (0x1827F9eA98E0bf96550b2FC20F7233277FcD7E63ConvexEthMetaStrategyProxy / addresses.mainnet.ConvexOETHAMOStrategy) from the OETH Vault through a governance proposal. The old Curve OETH/ETH pool used by this AMO is no longer used.

The proposal performs a single action on the OETH Vault:

  • removeStrategy(0x1827F9eA98E0bf96550b2FC20F7233277FcD7E63)

removeStrategy withdraws all assets from the strategy back to the Vault (and reverts if any non-dust balance remains) before marking it as unsupported, so no separate withdrawal action is needed.

No new contracts are deployed — this is a governance action only.

Verified by running the full deploy chain on a mainnet fork (FORK=true pnpm run node:fork): the proposal is submitted, voted, queued and executed on the OETH Vault without revert.

Governance

Deploy checklist

Two reviewers complete the following checklist:

- [ ] All deployed contracts are listed in the deploy PR's description
- [ ] Deployed contract's verified code (and all dependencies) match the code in master
- [ ] Contract constructors have correct arguments
- [ ] The transactions that interacted with the newly deployed contract match the deploy script.
- [ ] Governance proposal matches the deploy script
- [ ] Smoke tests pass after fork test execution of the governance proposal

@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.83%. Comparing base (9c90a5d) to head (397f7e8).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2907   +/-   ##
=======================================
  Coverage   50.83%   50.83%           
=======================================
  Files         110      110           
  Lines        4873     4873           
  Branches     1353     1353           
=======================================
  Hits         2477     2477           
  Misses       2392     2392           
  Partials        4        4           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants