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

Fix release Dockerfile and add wasmer #316

Merged
merged 2 commits into from
Sep 18, 2023

Conversation

jprendes
Copy link
Collaborator

@jprendes jprendes commented Sep 15, 2023

This PR fixes an issue where the release Dockerfile wouldn't build with BASE_IMAGE=alpine because of the selected features.
It also adds wasmer to the the default list of crates in the release Dockerfile.

Note that adding wasmer is not strictly necessary as the release workflow manually sets the CRATE argument.
Fixing the issue with the features is required for the release.

@jprendes jprendes changed the title Add wasmer to release Add wasmer to release Dockerfile Sep 15, 2023
@jprendes jprendes changed the title Add wasmer to release Dockerfile Fix release Dockerfile and add wasmer Sep 15, 2023
@jprendes jprendes closed this Sep 15, 2023
@jprendes jprendes reopened this Sep 15, 2023
@jprendes jprendes closed this Sep 15, 2023
@jprendes jprendes reopened this Sep 15, 2023
Copy link
Member

@Mossaka Mossaka left a comment

Choose a reason for hiding this comment

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

lgtm!

@Mossaka Mossaka merged commit 0f7124f into containerd:main Sep 18, 2023
96 of 99 checks passed
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.

2 participants