toolbox/distrobox rhel

that part i think needs some more thinking, are we talking about using the toolbox command instead or running the toolbox image inside distrobox like we do now?
53 Replies
bsherman
bshermanā€¢12mo ago
here
bsherman
bshermanā€¢12mo ago
https://github.com/ublue-os/config/issues/186 specifically came out of a PR to add rhel toolboxes
GitHub
provide option to use toolbox in our distrobox just recipes Ā· Issue...
I suggest maybe add an argument to all our distrobox recipes which allows toggling toolbox, and perhaps allows setting an env var which defaults to toolbox. The idea people that some users definite...
HikariKnight
HikariKnightOPā€¢12mo ago
because if it is the latter, we can just supply a -toolbox.ini
bsherman
bshermanā€¢12mo ago
the original ticket was not for adding rhel distroboxes but actually to add them with toolbox because ostensibly, "real RHEL people" are using toolbox, not distrobox
j0rge
j0rgeā€¢12mo ago
and toolbox can't do ini files
bsherman
bshermanā€¢12mo ago
so, i approved a PR which did add rhel images to distrobox ... but with the caveat, and creation of ticket 186
HikariKnight
HikariKnightOPā€¢12mo ago
hence i wanted to ask if we were talking about toolbox image or toolbox the binary
bsherman
bshermanā€¢12mo ago
I had intended to implement a just arg for the distrobox images which could make them toobox instead... but then you @HikariKnight started your efforts so i wanted to get out of theway
j0rge
j0rgeā€¢12mo ago
yeah the red hatters use toolbox the binary, they don't use distrobox at all. So the idea was "if you're using rhel then let's give it to them the way redhat wants it"
bsherman
bshermanā€¢12mo ago
</storytelling>
HikariKnight
HikariKnightOPā€¢12mo ago
i will see if i can come up with something for toolbox, if i can make it use the ini files that would be awesome too, less maintenance work šŸ˜›
j0rge
j0rgeā€¢12mo ago
but there's also like zero reason to use toolbox for everyone else
HikariKnight
HikariKnightOPā€¢12mo ago
if its just a check for the rhel/fedora image then thats easy well should be easy
bsherman
bshermanā€¢12mo ago
i imagined that either an environment variable or an arg to all the distrobox just recipes would flip between using distrobox or toolbox... but there's different args for each, too but still, it would make us more friendly to people who may wish to use toolbox not distrobox
HikariKnight
HikariKnightOPā€¢12mo ago
ujust toolbox- here we come
bsherman
bshermanā€¢12mo ago
but once you started working, i wasn't really sure how it would look. just wanting to make sure the context is clear here
j0rge
j0rgeā€¢12mo ago
yeah I'm not sure it's worth the added complexity to have people switch to a tool we specifically moved away from, but don't have strong opinions as long as it's well scoped and distrobox is the default like, if people ask which should I use we tell them distrobox but having a choice implies that using toolbox is fine
bsherman
bshermanā€¢12mo ago
toolbox IS fine šŸ™‚
HikariKnight
HikariKnightOPā€¢12mo ago
if args can be relatively translated, it can be added to the distrobox tooling (at least the Distrobox function) if it cant then its a separate just recipe
bsherman
bshermanā€¢12mo ago
i do think your changes got rid of all the distrobox-FOO recipes and replaced with a single distrobox which takes an arg/choice, right? if that's the case, then having toolbox in addition to distrobox may be fine but i do see what Jorge is saying... having both implies they can both be used and they can... but it's confusing
HikariKnight
HikariKnightOPā€¢12mo ago
yeah i got rid of all the distrobox-foo commands
j0rge
j0rgeā€¢12mo ago
right and then we'll have people saying "I can't use this properly" because they're following fedora's recommendations
bsherman
bshermanā€¢12mo ago
i wish we'd started with "toolbox-foo" but used distrobox as the implementation... since even distrobox uses the "toolbox" container tag
j0rge
j0rgeā€¢12mo ago
"how come I use ublue and I can't create seperate home directories for my toolboxes?" (I'm playing devil's advocate)
bsherman
bshermanā€¢12mo ago
but our just recipe doesn't create separate home dirs
j0rge
j0rgeā€¢12mo ago
no what I'm saying is we use distrobox instead of toolbox
bsherman
bshermanā€¢12mo ago
and i don't think it should... should be a simple set of defaults but we don't remove toolbox
HikariKnight
HikariKnightOPā€¢12mo ago
new one has an arg for it but only if you pass it, it never asks if you do the "guided tour"
bsherman
bshermanā€¢12mo ago
@HikariKnight : you've seen our respective thoughts šŸ˜„ please crank out something awesome and we'll go from there šŸ˜„ we DO need another approver on the first PR though...
HikariKnight
HikariKnightOPā€¢12mo ago
i think the easiest would be a toolbox- recipe with its own tooling but you cant use toolbox and distrobox at the same time for containers? if so i can write in a check for that as i have 0 experience with toolbox, didnt exist last time i touched a centos box
bsherman
bshermanā€¢12mo ago
well, it existed before distrobox šŸ˜‰
HikariKnight
HikariKnightOPā€¢12mo ago
last time i touched a centos box i believe dnf didnt exist
bsherman
bshermanā€¢12mo ago
i don't think it would be horrible... the "docstring" for distrobox- could say "Create a toolbox container (using uBlue recommend distrobox)" and toolbox docstring could say "Alternative way to create toolbox containers" or something
HikariKnight
HikariKnightOPā€¢12mo ago
"the RedHat way to create toolbox containers"
bsherman
bshermanā€¢12mo ago
yeah, something like that
HikariKnight
HikariKnightOPā€¢12mo ago
toolbox can use normal images or is it limited to just toolbox tailored images?
bsherman
bshermanā€¢12mo ago
they both use the same images these days, I think that was the original inspiration for distrobox, i think, ability to use "non-standard" images, but then toolbox added the same thing
HikariKnight
HikariKnightOPā€¢12mo ago
a super dumb way to do the ini file to toolbox chain then could be to just look for each section/name and grab the value of the image key, if we can pass a full image path to toolbox šŸ¤” splitting the release to its own string shouldnt be hard either as long as toolbox supports latest as a release but i can make the toolbox recipe just do rhel images first and see how it goes from there i know i was joking about being stuck in config i didnt mean it literally šŸ˜‚
bsherman
bshermanā€¢12mo ago
i mean, yeah, the core requirement here is for us to provide the set of rhel/fedora toolbox images via the toolbox tool, since that's what is expected by RedHatters and hard core RHEL/Fedora users... but none of the other images NEED to be exposed that way
HikariKnight
HikariKnightOPā€¢12mo ago
oh then this will be easier than i thought
bsherman
bshermanā€¢12mo ago
and when i say fedora, i don't mean custom built fedora images from bluefin or anything like that, just bog standard upstream fedora-toolbox:39 or 38 etc
HikariKnight
HikariKnightOPā€¢12mo ago
:latest šŸ˜›
bsherman
bshermanā€¢12mo ago
eh, whatever the existing pattern is for the rhel stuff, i would have to look
HikariKnight
HikariKnightOPā€¢12mo ago
but i can just give them a list then they can write version gives me a reason to add input to ugum i guess šŸ˜‚ i will work on toolbox before i clean up bazzite just files and bluefin just files then, how that sound? after the distrobox PR is merged so bluefin and maybe bazzite can get to enjoy an "auto generated" image list for distroboxes
bsherman
bshermanā€¢12mo ago
if you think toolbox- for rhel/fedora is a distinct recipe, then i wouldn't consider it blocking to your existing changeset so i'd suggest making it another PR
HikariKnight
HikariKnightOPā€¢12mo ago
yeah if the image scope is way smaller than distrobox. It can be altered if the need ever arises. but from what i gathered here a separate recipe is what makes the most sense in my head right now plus it would not give the false impression that they can yeet the homedir somewhere else yeah glancing over the docs i would say having it as its own thing would be for the best, creates a clear separation too
j0rge
j0rgeā€¢12mo ago
I could ack the PR but I'm going to let an engineer do it. (sorry got pulled into a meeting)
bsherman
bshermanā€¢12mo ago
sounds reasonable, and it's a distinct PR from the distrobox improvements you already have in flight
HikariKnight
HikariKnightOPā€¢12mo ago
just need to find one who is free šŸ¤£ i dont know everyone in here well enough yet to know what everyone does. but hi, if you have not figured yet. i do scripting and utilities and as my github says "i make things work" šŸ¤£
j0rge
j0rgeā€¢12mo ago
@Kyle Gospo or @EyeCantCU will be the hero we need.
EyeCantCU
EyeCantCUā€¢12mo ago
If you need help with something, feel free to ping me at any point during the day
HikariKnight
HikariKnightOPā€¢12mo ago
@bsherman @EyeCantCU should be ready for checking now https://github.com/ublue-os/config/pull/199 the distrobox part of the tooling
Want results from more Discord servers?
Add your server