Skip to content

Conversation

@masih
Copy link
Collaborator

@masih masih commented Sep 26, 2025

@masih
Copy link
Collaborator Author

masih commented Sep 26, 2025

Confirmed, upgrade to latest wasmd results in panic in TestClaimSpecificCW721 test.

@masih
Copy link
Collaborator Author

masih commented Sep 26, 2025

The test now consumes 979,625 more gas. This is beyond accepted tolerance of gas increase as a result of upgrade.

@masih
Copy link
Collaborator Author

masih commented Sep 26, 2025

Further to group discussions, proceeding with migration of v0.3.9 as part of mono repo workstream, and opening 2 additional PRs to bubble up the work done in wasmd in sei-chain separately to mitigate risk of unexpected/excessive gas increase.

@masih
Copy link
Collaborator Author

masih commented Sep 26, 2025

Closing as there are no actionables here. The actual fix for the gas increase to follow in the PRs that bubble up the changes

@masih masih closed this Sep 26, 2025
@masih masih deleted the masih/upgrade-wasmd-test branch September 26, 2025 15:33
masih added a commit that referenced this pull request Sep 26, 2025
Merge git commit history from `sei-wasmd` into `sei-chain` and update the replace directive in root of `sei-chain` to use relative path.

>[!NOTE]
> The merged commits from `sei-wasmd` point to [`v0.3.9`](https://github.com/sei-protocol/sei-wasmd/releases/tag/v0.3.9), consistent with the version bubbled up to `sei-chain` prior to this migration. Note that 4 commits from `sei-wasmd` are excluded, where 2 of which carry functional significance. Due to [issues detected during the bubble up related to high gas usage](#2401) the work here explicitly excluded those commits. A seprate set of PRs will re-introduce the excluded commits to isolate risk from mono-repo migration.
> The PRs that need to be seprately bubbled up to `sei-chain` are:
> * sei-protocol/sei-wasmd#80
>* sei-protocol/sei-wasmd#82
jeanettaskafidas added a commit to jeanettaskafidas/glacier-flow that referenced this pull request Sep 29, 2025
Merge git commit history from `sei-wasmd` into `sei-chain` and update the replace directive in root of `sei-chain` to use relative path.

>[!NOTE]
> The merged commits from `sei-wasmd` point to [`v0.3.9`](https://github.com/sei-protocol/sei-wasmd/releases/tag/v0.3.9), consistent with the version bubbled up to `sei-chain` prior to this migration. Note that 4 commits from `sei-wasmd` are excluded, where 2 of which carry functional significance. Due to [issues detected during the bubble up related to high gas usage](sei-protocol/sei-chain#2401) the work here explicitly excluded those commits. A seprate set of PRs will re-introduce the excluded commits to isolate risk from mono-repo migration.
> The PRs that need to be seprately bubbled up to `sei-chain` are:
> * sei-protocol/sei-wasmd#80
>* sei-protocol/sei-wasmd#82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants