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

feat(toolbox-ctr): script using ctr instead of docker #7

Merged
merged 1 commit into from
Oct 27, 2022

Conversation

spnngl
Copy link
Contributor

@spnngl spnngl commented Oct 7, 2022

[Toolbox version using ctr]

Toolbox version using ctr cli instead of docker for system which have only containerd and not docker daemon.

Use debian:unstable-slim image as default.

How to use

Just type toolbox-ctr (as root) on a flatcar running containerd.

Testing done

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

@pothos
Copy link
Member

pothos commented Oct 10, 2022

Thanks for the contribution, I suggest that we do the fallback directly inside the toolbox script. E.g., prefer docker and try again with ctr if that fails or vice versa - with the idea that we could also use podman later.

@spnngl spnngl force-pushed the toolbox-ctr branch 2 times, most recently from 4083f48 to 26c4d31 Compare October 12, 2022 10:31
@spnngl
Copy link
Contributor Author

spnngl commented Oct 12, 2022

Just updated with ctr support with docker fallback directly within toolbox script.

@spnngl spnngl force-pushed the toolbox-ctr branch 2 times, most recently from dd7c52c to 950f8ee Compare October 12, 2022 12:46
toolbox Outdated Show resolved Hide resolved
toolbox Outdated Show resolved Hide resolved
@spnngl spnngl force-pushed the toolbox-ctr branch 2 times, most recently from ead7e82 to ee2289f Compare October 20, 2022 16:17
toolbox Outdated Show resolved Hide resolved
Toolbox version using docker as principal CRI but with ctr fallback for
containerd systems running without docker.

Use debian:unstable-slim image as default.
@pothos
Copy link
Member

pothos commented Oct 27, 2022

Thanks!

@pothos pothos merged commit 9695c9b into flatcar:flatcar-master Oct 27, 2022
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Oct 27, 2022
This pulls in
flatcar/toolbox#7
to download and mount the image with containerd instead of requiring
Docker.
t-lo pushed a commit to flatcar/scripts that referenced this pull request Apr 17, 2023
This pulls in
flatcar/toolbox#7
to download and mount the image with containerd instead of requiring
Docker.
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