Skip to content

toolbox: replace rkt with docker for stable/beta/alpha#1

Merged
dongsupark merged 1 commit intoflatcar-masterfrom
dongsu/replace-rkt-with-docker
Mar 5, 2021
Merged

toolbox: replace rkt with docker for stable/beta/alpha#1
dongsupark merged 1 commit intoflatcar-masterfrom
dongsu/replace-rkt-with-docker

Conversation

@dongsupark
Copy link
Copy Markdown
Member

Run docker instead of rkt, to avoid security issues.
Basically it runs docker save and unpacks the layer tarball.

Run docker instead of rkt, to avoid security issues.
Basically it runs `docker save` and unpacks the layer tarball.
Copy link
Copy Markdown
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

Currently toolbox is broken on edge release and that fixes it, so 👍

Copy link
Copy Markdown
Member

@alban alban left a comment

Choose a reason for hiding this comment

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

LGTM

@dongsupark dongsupark changed the title toolbox: replace rkt with docker toolbox: replace rkt with docker for stable/beta/alpha Oct 31, 2019
@vbatts vbatts requested a review from a team November 13, 2020 15:02
@dongsupark dongsupark merged commit 9a2919e into flatcar-master Mar 5, 2021
@dongsupark dongsupark deleted the dongsu/replace-rkt-with-docker branch March 5, 2021 09:14
dongsupark pushed a commit to flatcar-archive/coreos-overlay that referenced this pull request Mar 5, 2021
Since rkt will be deprecated soon, we should make toolbox run docker
instead of rkt.

Also delete dependency on `app-emulation/rkt`, and update hyperlinks.

It pulls in flatcar/toolbox#1 .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants