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

refactor: re-implement the slight shim with libcontainer instance #125

Merged
merged 2 commits into from
Aug 22, 2023

Conversation

Mossaka
Copy link
Member

@Mossaka Mossaka commented Aug 18, 2023

This commit re-implements the slight shim with LibcontainerInstance trait. This follows #121

@Mossaka Mossaka force-pushed the default-executor-for-slight branch from eb8fb47 to 7f2c09b Compare August 18, 2023 02:07
Signed-off-by: jiaxiao zhou <jiazho@microsoft.com>
@Mossaka Mossaka force-pushed the default-executor-for-slight branch from 551b637 to 1aae8ed Compare August 18, 2023 23:08
@Mossaka
Copy link
Member Author

Mossaka commented Aug 20, 2023

I've updated this PR to include a few other things

  1. split python setup script to bash scripts for better configurations.
  2. the CI uses built artifacts for integration tests, instead of building its own binaries.

@Mossaka Mossaka force-pushed the default-executor-for-slight branch from f4a6fe5 to c8195d4 Compare August 20, 2023 23:10
@Mossaka
Copy link
Member Author

Mossaka commented Aug 21, 2023

Woohoo!! The integration test is passed!

@Mossaka
Copy link
Member Author

Mossaka commented Aug 21, 2023

Could you please take a look? @devigned @0xE282B0 @jprendes

@Mossaka Mossaka requested a review from devigned August 21, 2023 23:58
@Mossaka Mossaka force-pushed the default-executor-for-slight branch from 4c0f8e7 to 21752f6 Compare August 21, 2023 23:59
this allows the CI and makefile to have more configurations

Signed-off-by: jiaxiao zhou <jiazho@microsoft.com>
@Mossaka Mossaka force-pushed the default-executor-for-slight branch from 21752f6 to 6105fb5 Compare August 22, 2023 00:00
@0xE282B0
Copy link
Contributor

Yay, looks good!
Is there a reason that all shims are built together?
When more shims are added we may run into the same problem.

If it is ok to build the shims in parallel, it could look like this:
Mossaka#1
https://github.com/0xE282B0/containerd-wasm-shims/actions/runs/5935591013

@Mossaka
Copy link
Member Author

Mossaka commented Aug 22, 2023

If it is ok to build the shims in parallel, it could look like this:

I created an issue #126 to discuss ways to make the CI faster. Building the shims in parallel is a great idea! Would you like to create a PR for this repo?

@Mossaka
Copy link
Member Author

Mossaka commented Aug 22, 2023

Merging it in now

@Mossaka Mossaka merged commit f9c70f2 into deislabs:main Aug 22, 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.

2 participants