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

Testing the upgrade process #54

Merged
merged 11 commits into from
Jul 30, 2023
Merged

Conversation

alistair-singh
Copy link

Resolves: SNO-580
Snowfork companion: Snowfork/snowbridge#910

doubledup and others added 10 commits July 18, 2023 23:23
* add hashed description

* use existing account

* add call filter
* initial

* update cargo lock

* handle local accounts and pallets
Co-authored-by: claravanstaden <Cats 4 life!>
* Update inbound queue benchmarks.

* Update inbound queue benchmarks.

---------

Co-authored-by: claravanstaden <Cats 4 life!>
* update registry to gateway

* added agent config

* ws

* increase message size
@@ -602,7 +607,7 @@ impl snowbridge_inbound_queue::Config for Runtime {
}

parameter_types! {
pub const MaxMessagePayloadSize: u32 = 512;
pub const MaxMessagePayloadSize: u32 = 2048;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Must accommodate the Max upgrade size of 1024.

type AgentHashedDescription = AgentHashedDescription;
type UniversalLocation = UniversalLocation;
type RelayLocation = RelayLocation;

#[cfg(feature = "runtime-benchmarks")]
type CreateAgentOrigin = EnsureXcmOrigin<RuntimeOrigin, LocalOriginToLocation>;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In case of benchmarks create an xcm multilocation from the signed origin.

type CreateAgentOrigin = EnsureXcmOrigin<RuntimeOrigin, LocalOriginToLocation>;

#[cfg(not(feature = "runtime-benchmarks"))]
type CreateAgentOrigin = EnsureXcm<Everything>;
Copy link
Author

@alistair-singh alistair-singh Jul 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Outside of benchmarks use the underlying multi-location provided by pallet-xcm.

@alistair-singh alistair-singh marked this pull request as ready for review July 30, 2023 06:18
@alistair-singh alistair-singh changed the base branch from proxy-contracts to snowbridge July 30, 2023 18:31
@alistair-singh alistair-singh merged commit 5f9f6cd into snowbridge Jul 30, 2023
0 of 2 checks passed
@alistair-singh alistair-singh deleted the alistair/upgrade-tests branch July 30, 2023 18:31
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.

4 participants