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

Update wasmvm dependencies and update Dockerfile for localnet #818

Merged
merged 7 commits into from
May 11, 2022

Conversation

Taztingo
Copy link
Contributor

@Taztingo Taztingo commented May 10, 2022

Description

The go.mod file was updated to use the latest version of wasmvm. The main reason for this update is to get the prebuilt aarch64 wasmvm libraries, which are needed by our localnet Dockerfile. Additionally, the Dockerfile was updated to incorporate these new libraries.

A M1 kafka issue was found, and a fix was added to install the needed dependencies on M1 machines.

closes: #817


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer
  • Review Codecov Report in the comment section below once CI passes

…rfile to use the correct libs from wasmvm. Also fixed a kafka issue for M1s in the Dockerfile.
@Taztingo Taztingo marked this pull request as draft May 10, 2022 13:56
@Taztingo Taztingo marked this pull request as ready for review May 10, 2022 14:22
@Taztingo Taztingo marked this pull request as draft May 10, 2022 18:21
@Taztingo Taztingo marked this pull request as ready for review May 10, 2022 19:04
Copy link
Contributor

@egaxhaj egaxhaj left a comment

Choose a reason for hiding this comment

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

LGTM

@Taztingo Taztingo merged commit c998708 into main May 11, 2022
@Taztingo Taztingo deleted the taztingo/817-wasmvm-to-rc-1.0.0 branch May 11, 2022 20:22
@iramiller iramiller added this to the v1.10.0 milestone Jun 2, 2023
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.

Update wasmvm to rc-1.0.0
4 participants