-
Notifications
You must be signed in to change notification settings - Fork 87
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
[RHELC-708] Check if shim-x64 package is present in the system #572
Conversation
Codecov Report
@@ Coverage Diff @@
## main #572 +/- ##
==========================================
+ Coverage 91.49% 91.50% +0.01%
==========================================
Files 18 18
Lines 2715 2719 +4
Branches 489 490 +1
==========================================
+ Hits 2484 2488 +4
Misses 172 172
Partials 59 59
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
A sanity check in this case would mean (not) having the This problem occurs because on BIOS, the package might not be installed at all, and this was a leftover mistake from my previous PR that I did not handle that properly... |
That is exactly what I had in mind. |
a7f890c
to
5aa15b3
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.
This looks good. The main change is just adding a single condition to perform the action only when the shim-x64 package is installed.
+1
In PR#553 we removed a condition from the shim-x64 special case workaround where it was only supposed to handle UEFI systems, and now we are handling BIOS as well. The problem is that in the BIOS systems, this package may not be installed at all. This commit introduces a quick check to validate if the package is present on the system before trying to backup it's protection file under `/etc/yum/protected.d`. Signed-off-by: Rodolfo Olivieri <[email protected]>
5aa15b3
to
785ecf3
Compare
@SpyTec, it's a follow up to https://issues.redhat.com/browse/RHELC-708 and its fix #553. After we merged it I spotted the implementation is incomplete and I mentioned it here: #553 (comment). Technically, there's no issue to this follow up bug fix. Even though I think it can be considered as part of the https://issues.redhat.com/browse/RHELC-708. But feel free to create a separate issue for this. |
In PR #553 we removed a condition from the shim-x64 special case workaround
where it was only supposed to handle UEFI systems, and now we are handling BIOS
as well. The problem is that in the BIOS systems, this package may not be
installed at all.
This commit introduces a quick check to validate if the package is present on
the system before trying to backup it's protection file under
/etc/yum/protected.d
.Signed-off-by: Rodolfo Olivieri [email protected]