Skip to content

First attempt at a CentOS-7 toolbox container.#286

Closed
kronenpj wants to merge 4 commits intocontainers:mainfrom
kronenpj:centos-7
Closed

First attempt at a CentOS-7 toolbox container.#286
kronenpj wants to merge 4 commits intocontainers:mainfrom
kronenpj:centos-7

Conversation

@kronenpj
Copy link
Copy Markdown

@kronenpj kronenpj commented Oct 6, 2019

Signed-off-by: Paul Kronenwetter kronenpj@gmail.com

Usage once it's available:

toolbox create --image centos-toolbox:7 tboxname
toolbox enter tboxname
{Do cool toolbox stuff here.}

It's a bit bloated now because of the yum update but when CentOS releases the 7.7 container it should be better and the yum update could be removed. I needed to perform the update before re-adding the documentation would work with yum reinstall.

Signed-off-by: Paul Kronenwetter <kronenpj@gmail.com>
@kronenpj kronenpj mentioned this pull request Oct 13, 2019
@mvala
Copy link
Copy Markdown

mvala commented Oct 14, 2019

+1

FROM docker.io/library/centos:7

ENV NAME=centos-toolbox VERSION=7
LABEL com.github.debarshiray.toolbox="true" \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe update this based on de5e5df while waiting for further review

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, many of the comments in #298 for the Ubuntu image also apply to this.

@mvala
Copy link
Copy Markdown

mvala commented Oct 23, 2019

Is it possible to test it soon?

@cgwalters
Copy link
Copy Markdown
Collaborator

I think in the end we should try to support the default pre-existing base images and not require a special "toolbox image". Basically, have all of the logic inside our binary and not a derived image. That said, there are some complications around this, like sudo and the documentation.

Copy link
Copy Markdown
Member

@debarshiray debarshiray left a comment

Choose a reason for hiding this comment

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

Thanks for working on this!

I wonder if we should attempt CentOS 8 first. For the simple reason that it's new, and hence a lot more flexible compared to CentOS 7. No strong opinion, though.

FROM docker.io/library/centos:7

ENV NAME=centos-toolbox VERSION=7
LABEL com.github.debarshiray.toolbox="true" \
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, many of the comments in #298 for the Ubuntu image also apply to this.

@debarshiray
Copy link
Copy Markdown
Member

I think in the end we should try to support the default pre-existing base images
and not require a special "toolbox image". Basically, have all of the logic inside
our binary and not a derived image. That said, there are some complications
around this, like sudo and the documentation.

Currently the Toolbox images don't really have any logic. They are just a means of altering the package set of the base images. We could do all the package management dance on the clients running /usr/bin/toolbox but then we lose some of the reproducibility that the images offer.

But yeah, the prospect of just being able to work with the base images is tempting.

@fhoxh
Copy link
Copy Markdown

fhoxh commented Oct 31, 2019

👍

@kronenpj kronenpj requested a review from debarshiray November 8, 2019 22:21
@martymichal martymichal added the 2. Images Images for creating toolbox containers label Dec 5, 2019
@mvala
Copy link
Copy Markdown

mvala commented Dec 13, 2019

Any updates here?

@softwarefactory-project-zuul
Copy link
Copy Markdown

Build succeeded.

@carwyn
Copy link
Copy Markdown

carwyn commented Jul 10, 2020

Arguably there's a strong case for needing both RHEL/CentOS 7 and 8 (any currently supported version). Surely they are two related tasks rather than waiting on 8 blocking 7 if that's closer to being ready?

Base automatically changed from master to main March 25, 2021 22:25
@debarshiray
Copy link
Copy Markdown
Member

Closing.

See #275

@debarshiray debarshiray closed this Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. Images Images for creating toolbox containers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants