summit: add new network

[nprt]

Adds placeholder values for the summit network.

[sigurpol]

Some early comments :

• https://github.com/paritytech/chainspecs/blob/nprt/summit-network/summit/relaychain/chainspec.json#L190-L194

              "5HKRZm64ppY99gqyPTZPFvy46DhVEGpjD6PnweFXSqTL8oxV",
              {
                "authority_discovery": "5GECqEAXq3oeCMnU1t1KKJJ1gJcNvZEeLDeot6AFXWuGvD2M",
                "babe": "5GECqEAXq3oeCMnU1t1KKJJ1gJcNvZEeLDeot6AFXWuGvD2M",

same session key for authority_discovery and babe

• parachainInfo.parachainId inconsistent with the top-level para_id: AH top level 1000 - genesis 1500, people top level 1004 - genesis: missing. @seadanda was mentioning we need 1000, 1004 and 1010 (bulletin).
• No DAP buffer account funding on AH (Paseo's prefunds it with ED - see asset-hub-paseo/src/genesis_config_presets.rs:56:

balances.push((dap_buffer_account(), ASSET_HUB_POLKADOT_ED)))

• AH collators (5DZQtq…, 5Dt3Xq…) have no balance on AH - is it ok? Should be at least put ED?
• not sure what's the plan for num_cores + parachain registration (still WIP?)

[sigurpol]

@seadanda , @Ank4n , @kianenigma - PTAL as well!

[sigurpol]

Some early comments :

• https://github.com/paritytech/chainspecs/blob/nprt/summit-network/summit/relaychain/chainspec.json#L190-L194

              "5HKRZm64ppY99gqyPTZPFvy46DhVEGpjD6PnweFXSqTL8oxV",
              {
                "authority_discovery": "5GECqEAXq3oeCMnU1t1KKJJ1gJcNvZEeLDeot6AFXWuGvD2M",
                "babe": "5GECqEAXq3oeCMnU1t1KKJJ1gJcNvZEeLDeot6AFXWuGvD2M",

same session key for authority_discovery and babe

• parachainInfo.parachainId inconsistent with the top-level para_id: AH top level 1000 - genesis 1500, people top level 1004 - genesis: missing. @seadanda was mentioning we need 1000, 1004 and 1010 (bulletin).
• No DAP buffer account funding on AH (Paseo's prefunds it with ED - see asset-hub-paseo/src/genesis_config_presets.rs:56:

balances.push((dap_buffer_account(), ASSET_HUB_POLKADOT_ED)))

• AH collators (5DZQtq…, 5Dt3Xq…) have no balance on AH - is it ok? Should be at least put ED?
• not sure what's the plan for num_cores + parachain registration (still WIP?)

Thanks @nprt 🙇 All my comments above are now addressed, except

• funding the DAP account (Actually two: buffer and staging). To be honest, it's not crucial to address it , mostly means that slashes under ED are lost. But for a short-living testnet is really not an issue so it's mostly hygiene item, definitely not a blocker. If you feel generous, this would be the difference in summit/parachain/asset-hub/chainspec.json:64 (sorry, but GH refuses to let me comment directly inline 😢 )

   "balances": {
     "balances": [
       ["5FyQB9HjYDB32jVcXnRa3aQpN8nY68cJyYt8t4FH5eZ5DgCj", 10000000000000000],
       ... (existing entries) ...
  -    ["5Ge8pVre9DuMnxUNQzwQsazFtAMZVcdjiBSHhp9zsDP5ZFuD", 10000000000000000]
  +    ["5Ge8pVre9DuMnxUNQzwQsazFtAMZVcdjiBSHhp9zsDP5ZFuD", 10000000000000000],
  +    ["5EYCAe5gKBMwLZyq5doL8n26pkA5pV6HSXdnmN4YawhPc2HH", 10000000000000000],
  +    ["5EYCAe5gKBMwLZyq5dw41Hw3V68nsCVTZQ2d4AYfbMb4kF29", 10000000000000000]
     ]
   },

(ED would suffice)

• Still configuration.config.scheduler_params.num_cores = 0 + missing register parachains + hrmp . Not sure if the idea is to do all the sudo registration post genesis. Definitely I am not authoritative enough to comment meaningfully, so I am just leaving it as a dumb question 😄

[nprt]

Some early comments :

• https://github.com/paritytech/chainspecs/blob/nprt/summit-network/summit/relaychain/chainspec.json#L190-L194

              "5HKRZm64ppY99gqyPTZPFvy46DhVEGpjD6PnweFXSqTL8oxV",
              {
                "authority_discovery": "5GECqEAXq3oeCMnU1t1KKJJ1gJcNvZEeLDeot6AFXWuGvD2M",
                "babe": "5GECqEAXq3oeCMnU1t1KKJJ1gJcNvZEeLDeot6AFXWuGvD2M",

same session key for authority_discovery and babe

• parachainInfo.parachainId inconsistent with the top-level para_id: AH top level 1000 - genesis 1500, people top level 1004 - genesis: missing. @seadanda was mentioning we need 1000, 1004 and 1010 (bulletin).
• No DAP buffer account funding on AH (Paseo's prefunds it with ED - see asset-hub-paseo/src/genesis_config_presets.rs:56:

balances.push((dap_buffer_account(), ASSET_HUB_POLKADOT_ED)))

• AH collators (5DZQtq…, 5Dt3Xq…) have no balance on AH - is it ok? Should be at least put ED?
• not sure what's the plan for num_cores + parachain registration (still WIP?)

Thanks @nprt 🙇 All my comments above are now addressed, except

* funding the DAP account (Actually two: buffer and staging). To be honest, it's not crucial to address it , mostly means that slashes under ED are lost. But for a short-living testnet is really not an issue so it's mostly hygiene item, definitely not a blocker. If you feel generous, this would be the difference in summit/parachain/asset-hub/chainspec.json:64 (sorry, but GH refuses to let me comment directly inline 😢 )

   "balances": {
     "balances": [
       ["5FyQB9HjYDB32jVcXnRa3aQpN8nY68cJyYt8t4FH5eZ5DgCj", 10000000000000000],
       ... (existing entries) ...
  -    ["5Ge8pVre9DuMnxUNQzwQsazFtAMZVcdjiBSHhp9zsDP5ZFuD", 10000000000000000]
  +    ["5Ge8pVre9DuMnxUNQzwQsazFtAMZVcdjiBSHhp9zsDP5ZFuD", 10000000000000000],
  +    ["5EYCAe5gKBMwLZyq5doL8n26pkA5pV6HSXdnmN4YawhPc2HH", 10000000000000000],
  +    ["5EYCAe5gKBMwLZyq5dw41Hw3V68nsCVTZQ2d4AYfbMb4kF29", 10000000000000000]
     ]
   },

(ED would suffice)

* Still `configuration.config.scheduler_params.num_cores = 0`  + missing register parachains  + hrmp . Not sure if the idea is to do all the sudo registration post genesis. Definitely I am not authoritative enough to comment meaningfully, so I am just leaving it as a dumb question 😄

Awesome, thanks for the suggestions! I'd definitely take a declarative approach here, and bundle all the relevant genesis configuration within the chainspec. I'll check the registering/onboarding process for paras tomorrow.

I'll fund the DAP account as well.