polkadot icon indicating copy to clipboard operation
polkadot copied to clipboard

Introduce OpenGov into Polkadot

Open gavofyork opened this issue 2 years ago • 26 comments

Introduce OpenGov into Polkadot. Remove Gov1 from Kusama.

cumulus companion: https://github.com/paritytech/cumulus/pull/2186

gavofyork avatar Feb 10 '23 17:02 gavofyork

to confirm, we will duplicate fellowship members between Kusama and Polkadot?

xlc avatar Feb 13 '23 02:02 xlc

to confirm, we will duplicate fellowship members between Kusama and Polkadot?

I think it's fine for the Fellowship will be removed from Kusama once the Polkadot/Kusama bridge is functional and can relay the Polkadot Fellowship into Kusama. Until then, I fear we need something there.

gavofyork avatar Feb 13 '23 07:02 gavofyork

Could we put the Fellowship on the Collectives parachain rather than Relay? @muharem can make a companion PR with the Fellowship set up there and the appropriate config to use the origins over XCM.

joepetrowski avatar Feb 13 '23 13:02 joepetrowski

Sounds good. It would also be great to see how far we can get with putting all of OpenGov onto the Collectives parachain. XCMv3 includes instructions for remote locking, so everything should - more or less - be there to implement it.

gavofyork avatar Feb 13 '23 16:02 gavofyork

@muharem I'll let you make a PR onto this one.

gavofyork avatar Feb 13 '23 16:02 gavofyork

@muharem I'll let you make a PR onto this one.

I open PR tomorrow

muharem avatar Feb 14 '23 11:02 muharem

bot rebase

muharem avatar Feb 15 '23 08:02 muharem

Rebased

@gavofyork we probably want to enable proxy for conviction voting, referenda and whitelist pallets' calls?

muharem avatar Feb 16 '23 05:02 muharem

@gavofyork we probably want to enable proxy for conviction voting, referenda and whitelist pallets' calls?

https://github.com/paritytech/polkadot/pull/6729

joepetrowski avatar Feb 16 '23 09:02 joepetrowski

@rphmeier final review?

@rrtti do you want get this under the nose of the auditors asap and track the audit?

gavofyork avatar Feb 16 '23 18:02 gavofyork

@gavofyork PR is queued for audit starting mid-next week: for a week. I will update as auditors work on it.

rrtti avatar Feb 17 '23 13:02 rrtti

@muharem is the collectives chain ready for this to be merged?

gavofyork avatar Feb 24 '23 23:02 gavofyork

@muharem is the collectives chain ready for this to be merged?

The code is ready, it compiles with this PR and looks correct to me. Need more approves there, and now I am working on integrational tests for it.

muharem avatar Feb 25 '23 01:02 muharem

Ok - we should ensure that this doesn't get deployed long prior to the Fellowship collectives chain, so I'd prefer not to merge until that's merged.

gavofyork avatar Feb 25 '23 10:02 gavofyork

@redzsina can you please check the companion of this PR https://github.com/paritytech/cumulus/pull/2186, it also needs an audit

muharem avatar Mar 07 '23 10:03 muharem

bot rebase

muharem avatar Mar 07 '23 10:03 muharem

Rebased

bot bench $ pallet polkadot-dev pallet_conviction_voting bot bench $ pallet polkadot-dev pallet_referenda bot bench $ pallet polkadot-dev pallet_whitelist

muharem avatar Mar 07 '23 11:03 muharem

@muharem https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494173 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet polkadot-dev pallet_conviction_voting. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 28-1623d5a4-d79c-4e37-8fb2-c4114a589fe2 to cancel this command or bot cancel to cancel all commands in this pull request.

command-bot[bot] avatar Mar 07 '23 11:03 command-bot[bot]

@muharem https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494174 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet polkadot-dev pallet_referenda. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 29-bd00bede-bd6a-40f6-bfc9-f68ae76ce11a to cancel this command or bot cancel to cancel all commands in this pull request.

command-bot[bot] avatar Mar 07 '23 11:03 command-bot[bot]

@muharem https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494175 was started for your command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet polkadot-dev pallet_whitelist. Check out https://gitlab.parity.io/parity/mirrors/polkadot/-/pipelines?page=1&scope=all&username=group_605_bot to know what else is being executed currently.

Comment bot cancel 30-4e28c58c-71f7-483f-9f3c-d6513b3bcbf6 to cancel this command or bot cancel to cancel all commands in this pull request.

command-bot[bot] avatar Mar 07 '23 11:03 command-bot[bot]

@muharem Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet polkadot-dev pallet_conviction_voting has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494173 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494173/artifacts/download.

command-bot[bot] avatar Mar 07 '23 11:03 command-bot[bot]

@muharem Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet polkadot-dev pallet_referenda has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494174 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494174/artifacts/download.

command-bot[bot] avatar Mar 07 '23 11:03 command-bot[bot]

@muharem Command "$PIPELINE_SCRIPTS_DIR/commands/bench/bench.sh" pallet polkadot-dev pallet_whitelist has finished. Result: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494175 has finished. If any artifacts were generated, you can download them from https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2494175/artifacts/download.

command-bot[bot] avatar Mar 07 '23 11:03 command-bot[bot]

pushed one commit, updating the weights to drop the usage of deprecated api (from_ref_time)

muharem avatar Mar 07 '23 13:03 muharem

@gavofyork with a few tweaks I've got main interchain cases pass, here the list and tests https://github.com/paritytech/parachains-integration-tests/pull/96 here the tweaks https://github.com/paritytech/polkadot/pull/6855 , https://github.com/paritytech/cumulus/pull/2305

muharem avatar Mar 10 '23 16:03 muharem

@muharem shall we get this merged now?

gavofyork avatar Mar 21 '23 10:03 gavofyork

@muharem shall we get this merged now?

yes, cumulus companion also ready to be merge. I wait for a bit only with the migration and give the last call for the fellowship members.

muharem avatar Mar 21 '23 12:03 muharem

bot merge

muharem avatar Mar 21 '23 13:03 muharem