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

Moneta Mainnet Release Checklist #87

Closed
5 of 6 tasks
the-frey opened this issue Nov 18, 2021 · 13 comments
Closed
5 of 6 tasks

Moneta Mainnet Release Checklist #87

the-frey opened this issue Nov 18, 2021 · 13 comments
Labels
Priority This needs to be done ASAP

Comments

@the-frey
Copy link
Contributor

the-frey commented Nov 18, 2021

In order to call the Moneta upgrade complete, we need to do the following:

- Gov prop to set block max gas #82
- Moneta alpha upgrade #71 #72
- Moneta beta upgrade #17 #85

~Once moneta is up, we will want to pin some contracts, to reduce their cost. This is via governance. ~

- Contract pinning gov prop

Some example contracts that might be pinned:

- DAODAO
- JunoMint
- Junoswap
- whoami
etc...

This will be followed by a final release:

- Replace wasmd fork with wasmd 0.21 when SDK 44 <-> wasmd is complete and audited


New working plan:

Once moneta is up, we will want to pin some contracts. This is done via governance.

  • Contract pinning gov prop

Some example contracts that might be pinned:

  • DAODAO
  • JunoMint
  • Junoswap
  • whoami
  • CW20
  • CW721
    etc...

This will be followed by a final release:

  • Replace wasmd fork with wasmd 0.21 when SDK 44 <-> wasmd is complete and audited
@the-frey the-frey added this to the v2.0.0 moneta: juno<->cw milestone Nov 18, 2021
@faddat
Copy link
Contributor

faddat commented Nov 19, 2021

Little note on wasmd.

This has proven a bit trick and I'm still on it over the weekend. I think that these updates are high priority because of some of the recently discovered issues.

@the-frey the-frey added the Priority This needs to be done ASAP label Nov 22, 2021
@faddat
Copy link
Contributor

faddat commented Nov 23, 2021

Hi, given what I've seen recently (Sentinel, Regen) I'd like to recommend that we include the following in Moneta, and recommend it rather strongly.

  • ibc-go -> 2.0 (this means writing protocol negotiation for cosmwasm in wasmd)
  • tendermint -> 0.35 (this means a refactored and more secure p2p)

I am taking this in phases. Right now I am doing the module migration process for wasmd, so that it'll have no changes other than the sdk upgrade. I think our mileage may vary. But when that is done I'll make a PR with that to the main CW repo.

So my phase one gets us to:

  • TM 0.34.14
  • SDK 0.44.3
  • IBC-go v1.2.3

If it takes too long, and it might, this is probably okay for moneta, but I do prefer the above route.

@the-frey
Copy link
Contributor Author

I think naively that phase one would be adequate, as the 44.3 <-> wasmd / wasmvm security patch is really the biggest thing concerning me atm. Want to be in the position where when the Confio official ver comes out we can switch over pretty seamlessly.

@faddat
Copy link
Contributor

faddat commented Nov 23, 2021

I'm trying to write the confio official version by staying extremely, extremely close to what they've currently got.

@faddat
Copy link
Contributor

faddat commented Nov 23, 2021

@giansalex
Copy link
Member

should we revert this pull #51?
keplr supports sdk 0.44 with a new feature no-legacy-stdTx

@the-frey
Copy link
Contributor Author

just so I understand - keplr supports 0.44 only if legacy is disabled?

@giansalex
Copy link
Member

no-legacy-stdTx is for cosmos-sdk 0.44x chains, /txs is not necessary.

@the-frey
Copy link
Contributor Author

hmm, yeah well if we can remove unneeded code then that's code we don't need to maintain in future - so I would vote for that

@surferryan
Copy link

Gas should be set @ 7.5/2100
Adjust for 5% slippage
Adjustment for 10% taxation of net withdrawal.

@giansalex
Copy link
Member

we can pin codes, after DAODAO launch?

@the-frey
Copy link
Contributor Author

I think we can yeah

@blockcreators
Copy link
Contributor

I think we can yeah

Can we close this issue now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority This needs to be done ASAP
Projects
None yet
Development

No branches or pull requests

6 participants