-
Notifications
You must be signed in to change notification settings - Fork 51
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(23.10) add slice for crun and uidmap #224
Conversation
Diff of dependencies: slices/libpam0g.yaml@@ -1,4 +1,3 @@
debconf
-debconf-2.0
libaudit1
libc6 |
182c4a8
to
8fbc2be
Compare
8fbc2be
to
2e9fd75
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. There are some issues regarding the linting (see workflow runs) to be fixed, and it is suggested to add the copyright files explicitly in the slice definitions.
2e9fd75
to
b852b50
Compare
@cjdcordeiro @rebornplusplus can you approve the workflow please? |
e75eea6
to
acac204
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks all good to me! Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see my comments #225 (review) and #225 (comment) in PR #225 and address them here as well.
The file is already there https://github.com/endersonmaia/chisel-releases/blob/crun-to-23.10/slices/libcrypt1.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ty for the proposal.
Apart from the linting, I'd suggest also re-working the debconf
slice definitions. Especially the bins
slice, as it contains much more than just binaries. Those modules deserve a separate slice(s). There are two preferred approaches here: group the modules into functionality, and create slices for each one of those (like we did with the python and openjdk slices); the other approach is to create a generic modules
slices where all of those modules fallback to.
I was experimenting debconf, and although it doesn't explicitly show
I didn't get the point of installing its bins if I can't use it. # head /usr/bin/debconf -n 1
#!/usr/bin/perl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right, so this is a fundamental question for this PR: if debconf
doesn't work without perl
, how did these slices work for your use case?
Please note that in some cases debconf
is a dependency of a pkg because it is used during its postinst
. If your crun
+uidmap
use case works well with the current slices and mutation scripts, then I'd consider leaving the debconf
slices out of this PR -> no need to introduce something we're sure about and that hasn't been thoroughly tested
Regardless of that dependency, if we want to keep the debconf
SDF, it still needs to be reworked according to the previous comment.
let's close this since the release is EOL. #223 is going in soon |
Proposed changes
Forward porting
Testing
Checklist
Additional Context