-
Notifications
You must be signed in to change notification settings - Fork 52
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
Ship docker sysext and rip out torcx [WIP] #982
Conversation
982289a
to
e1fcd5a
Compare
To create the two seperate sysexts we could remove containerd stuff from the docker sysext image: something like |
We could use a temporary |
Build action triggered: https://github.com/flatcar/scripts/actions/runs/6415680058 |
@t-lo @pothos I have already added creating two separate sysext images—one for containerd and the other for Docker—and used the mangle script. Also, I have removed torcx from the coreos base ebuild and removed app-arch/torcx from coreos-overlay. The remaining task for the end of my internship is to rewrite the mantle kola tests to account for the removal of torcx. |
57c7422
to
5815b1b
Compare
@krishjainx can you look into adding license/version information about the packages contained in the built sysexts to our generated license and packages files. I think that needs to be done before merging this. |
Regarding a4b3c24 as Kai suggested to me the license call uses the package list, so it should work as long as the packages get part of the list |
…he unit dependencies from wants to upholds); one sysext for all docker for now
9ec68c4
to
5f9c84d
Compare
Superseded by #1216. |
Ship Docker and Containerd Sysext and deprecate torcx
Solves flatcar/Flatcar#443
Related Issues: flatcar/Flatcar#1092, flatcar/Flatcar#1093
Once we have it building in the CI people can download the image from there and play around already.
How to use
@pothos recommended I file a draft PR for aiding communication
Testing done
When I attempt to use run build_image script and run the launch script the Flatcar provided sysext works.
1.
Notes for reviewers/those interested
Kai's fix seems to work! Solved
Ship docker+containerd sysext images
Remove torcx