add genesis presets for glutton westend

[clangenb]

Extracted from #7473.

Part of: #5704.

I did not use the genesis presets in the parachain-bin, as we rely on passing custom para-ids to the chains specs to launch many glutton chains on one relay chain (is this still true?). This is currently not compatible with the genesis presets, which hard code the para ID, see #7618 and #7384.

However, the parachain-bin genesis config for glutton at least refers to the one from the runtime to prevent duplication.

[clangenb]

/cmd prdoc --audience runtime_dev --bump minor

[bkontur]

@clangenb also please clean cumulus/polkadot-parchain/src/chain_spec/glutton.rs

[clangenb]

@clangenb also please clean cumulus/polkadot-parchain/src/chain_spec/glutton.rs

Hmm, I don't think this is a good idea, polkadot-parachain-bin CLI wants to extract the para-id from the command, see

polkadot-sdk/cumulus/polkadot-parachain/src/chain_spec/mod.rs

Line 155 in e8d17cb

para_id.expect("Must specify parachain id"),

.

But we can't do this with the genesis presets, as the para-id is hardcoded. So I think I should only clean this up, if we decide to remove the option for passing custom para-ids from the CLI. What is your take on this?

[bkontur]

@clangenb also please clean cumulus/polkadot-parchain/src/chain_spec/glutton.rs

Hmm, I don't think this is a good idea, polkadot-parachain-bin CLI wants to extract the para-id from the command, see

polkadot-sdk/cumulus/polkadot-parachain/src/chain_spec/mod.rs

Line 155 in e8d17cb

para_id.expect("Must specify parachain id"),

.

But we can't do this with the genesis presets, as the para-id is hardcoded. So I think I should only clean this up, if we decide to remove the option for passing custom para-ids from the CLI. What is your take on this?

I see, so you don't need to use .with_genesis_config_preset_name(sp_genesis_builder::LOCAL_TESTNET_RUNTIME_PRESET), but you can still use

.with_genesis_config_patch(glutton_westend_runtime::genesis_config_presets::glutton_westend_genesis(
		para_id,
		vec![Sr25519Keyring::Alice.public().into()],
	))

at least we will use one source of this genesis, and it should be ok, while we are referencing the glutton_westend_runtime::WASM_BINARY above.

[clangenb]

Ahh, damn you are right. I could have thought of this too. Thanks. 🚀

[clangenb]

/cmd prdoc --audience runtime_dev --bump minor

[github-actions]

Command "prdoc --audience runtime_dev --bump minor" has failed ❌! See logs here

[clangenb]

/cmd prdoc --audience runtime_dev --bump minor --force

[clangenb]

PR is ready for more reviews. 🚀