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
[OPENJDK-1335] microdnf update weak deps #332
[OPENJDK-1335] microdnf update weak deps #332
Conversation
Ensure images do not have a set of RPMs installed which are not hard dependencies of OpenJDK or Maven but might get installed as transitive weak dependencies if microdnf is not configured with install_weak_deps=0 Signed-off-by: Jonathan Dowland <jdowland@redhat.com>
The new test passed for ubi8/openjdk-8 but shouldn't have. I think it's a race condition, due to using "available container log". I'm going to try just "container log". This unfortunately means a 30s wait per step in the scenario, but "correct" trumps "fast". |
This is potentially racy. Use 'container log' instead. Signed-off-by: Jonathan Dowland <jdowland@redhat.com>
Signed-off-by: Jonathan Dowland <jdowland@redhat.com>
modules/util/pkg-update/execute.sh
Outdated
@@ -4,7 +4,7 @@ set -euo pipefail | |||
for candidate in yum dnf microdnf; do | |||
if command -v "$candidate"; then | |||
mgr="$(command -v "$candidate")" | |||
"$mgr" update -y | |||
"$mgr" update --setopt=install_weak_deps=0 -y |
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 also set --setopt=tsflags=nodocs
that's what regular install commands have as well.
Signed-off-by: Jonathan Dowland <jdowland@redhat.com>
3b81c0d
to
6b7030a
Compare
Signed-off-by: Jonathan Dowland <jdowland@redhat.com>
d8e2234
to
760d7cf
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. Thanks!
Part 1 of a 2-commit PR: I want to see CI red lights for this commit before pushing the fix.
This commit adds a test for some RPMs that are installed as transitive weak dependencies via microdnf when it is not configured with install_weak_deps=0.
https://issues.redhat.com/browse/OPENJDK-1335?filter=12398815