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.

Sign up for GitHub

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

Jump to bottom

images: Added initial definition for ubi8-toolbox image #791

Closed
wants to merge 2 commits into from
Closed

images: Added initial definition for ubi8-toolbox image #791

wants to merge 2 commits into from

Conversation

olivergs
Copy link
Collaborator

Added definition for the ubi8-toolbox image

@olivergs
Copy link
Collaborator Author

I've created a list of the packages that are present in fedora-toolbox image that are not available for installation in ubi8-toolbox image.

https://gist.github.com/olivergs/734ec60d366f62675079b6eb105cf418

Whe should discuss about finding replacements for them or which of them could be ignored

@softwarefactory-project-zuul
Copy link

Build failed.

@olivergs olivergs force-pushed the ubi8-toolbox-image branch from 92332bb to 1d0407c Compare June 18, 2021 16:04
@softwarefactory-project-zuul
Copy link

Build failed.

@HarryMichal HarryMichal added 2. Images Images for creating toolbox containers 3. New Feature New feature labels Jun 21, 2021
debarshiray
debarshiray requested changes Jun 29, 2021
View reviewed changes
Copy link
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 a lot for working on this, @olivergs

images/rhel/rhel8/ubi8-toolbox/Containerfile Outdated Show resolved Hide resolved
images/rhel/rhel8/ubi8-toolbox/Containerfile Outdated Show resolved Hide resolved
images/rhel/rhel8/ubi8-toolbox/Containerfile Outdated Show resolved Hide resolved
images/rhel/rhel8/ubi8-toolbox/extra-packages Outdated Show resolved Hide resolved
images/rhel/rhel8/ubi8-toolbox/extra-packages Outdated Show resolved Hide resolved
@olivergs olivergs force-pushed the ubi8-toolbox-image branch from 1d0407c to 893f23a Compare June 29, 2021 15:52
@olivergs
Copy link
Collaborator Author

After required changes I changed the base image version tag and now, as UBI8.5 is not available yet, you can't build it to check. You would have to change to 8.4 tag until 8.5 is released.

@olivergs olivergs requested a review from debarshiray June 29, 2021 15:54
@softwarefactory-project-zuul
Copy link

Build failed.

@olivergs olivergs force-pushed the ubi8-toolbox-image branch from 893f23a to 4fdd1f8 Compare July 19, 2021 08:37
@softwarefactory-project-zuul
Copy link

Build failed.

@debarshiray debarshiray mentioned this pull request Dec 3, 2021
Open
@debarshiray
Copy link
Member

We need to update this to match the image definition that got released with RHEL 8.5.

@travier travier mentioned this pull request Mar 10, 2022
Closed
@travier
Copy link
Member

travier commented Sep 9, 2022

We've made progress in this front in #1019. See the top comment for links to community supported images.

@debarshiray
Copy link
Member

debarshiray commented Mar 29, 2023 •
edited
Loading

I've created a list of the packages that are present in fedora-toolbox image that are not available for installation in ubi8-toolbox image.

https://gist.github.com/olivergs/734ec60d366f62675079b6eb105cf418

Whe should discuss about finding replacements for them or which of them could be ignored

Thanks for that list. I completely forgot to close the loop here.

Over time, I managed to convince folks to add some of those packages to UBI:

  • flatpak-spawn was added to UBI for 8.5 and is part of the toolbox image since 8.6
  • bash-completion and xorg-x11-xauth got added to 8.5, but I forgot to add it to the image

I will see if I can get a few more packages into UBI.

@debarshiray
Copy link
Member

Anyway, I already copied the UBI-based toolbox images created for RHEL by @olivergs from the internal RHEL Git tree in #1232

My apologies for creating a separate pull request, instead of updating this one. I totally forgot about this one.

Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tpopela tpopela tpopela left review comments

@debarshiray debarshiray Awaiting requested review from debarshiray debarshiray is a code owner

Assignees
No one assigned
Labels
2. Images Images for creating toolbox containers 3. New Feature New feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@olivergs @debarshiray @travier @tpopela @HarryMichal