Skip to content

Replace bind9.18-utils with bind-utils #278

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
Merged
phracek merged 4 commits into from
Jan 12, 2026
Merged

Replace bind9.18-utils with bind-utils #278

phracek merged 4 commits into from
Jan 12, 2026

Conversation

@phracek
Copy link
Member

@phracek phracek commented Jan 5, 2026 •
edited by github-actions bot
Loading

Replace bind9.18-utils with bind-utils

The version package is not needed at all.
See Jira Ticket See Jira Ticket: https://issues.redhat.com/browse/RHEL-80345

The PR that brings that issue was #243

@phracek
Replace bind9.18-utils with bind-utils.
cfd2083 
The version package is not needed at all.
See Jira Ticket See Jira Ticket: https://issues.redhat.com/browse/RHEL-80345

Signed-off-by: Petr "Stone" Hracek <[email protected]>
@phracek phracek requested a review from pkhartsk January 5, 2026 11:26
@github-actions
Copy link

github-actions bot commented Jan 5, 2026 •
edited
Loading

Pull Request validation

Failed

🔴 Review - Missing review from a member (1 required)

Success

🟢 CI - All checks have passed

@pkhartsk
Copy link
Contributor

pkhartsk commented Jan 5, 2026

Isn't it better to remove it altogether? bind-utils shouldn't be required for the container to work correctly as seen in all the other Dockerfiles.

@phracek
Copy link
Member Author

phracek commented Jan 6, 2026

Isn't it better to remove it altogether? bind-utils shouldn't be required for the container to work correctly as seen in all the other Dockerfiles.

bind-utils is not needed? Then yes sure.

@phracek
Remove bind-utils completely
86e42f9 
Signed-off-by: Petr "Stone" Hracek <[email protected]>
@phracek
Copy link
Member Author

phracek commented Jan 6, 2026

[test-all]

@github-actions
Copy link

github-actions bot commented Jan 6, 2026 •
edited
Loading

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
Fedora - PyTest - 2.4Fedora-latestx86_64✅ passed12.01.2026 09:59:597min 39stest pipeline
CentOS Stream 10 - 2.4CentOS-Stream-10x86_64✅ passed12.01.2026 10:08:209min 25stest pipeline
CentOS Stream 10 - 2.4-microCentOS-Stream-10x86_64✅ passed12.01.2026 10:12:0715min 51stest pipeline
CentOS Stream 10 - PyTest - 2.4-microCentOS-Stream-10x86_64✅ passed12.01.2026 10:08:049min 35stest pipeline
CentOS Stream 10 - PyTest - 2.4CentOS-Stream-10x86_64✅ passed12.01.2026 10:14:0114min 27stest pipeline
Fedora - 2.4Fedora-latestx86_64✅ passed12.01.2026 10:10:208min 20stest pipeline
RHEL10 - Unsubscribed host - PyTest - 2.4RHEL-10-Nightlyx86_64✅ passed12.01.2026 10:08:5911min 51stest pipeline
RHEL9 - Unsubscribed host - 2.4RHEL-9.6.0-Nightlyx86_64✅ passed12.01.2026 09:59:5815min 34stest pipeline
RHEL8 - PyTest - 2.4RHEL-8.10.0-Nightlyx86_64✅ passed12.01.2026 10:00:0612min 31stest pipeline
Fedora - 2.4-microFedora-latestx86_64✅ passed12.01.2026 10:00:088min 22stest pipeline
RHEL9 - Unsubscribed host - PyTest - 2.4RHEL-9.6.0-Nightlyx86_64✅ passed12.01.2026 10:00:2515min 52stest pipeline
CentOS Stream 9 - PyTest - 2.4CentOS-Stream-9x86_64✅ passed12.01.2026 10:10:219min 42stest pipeline
CentOS Stream 9 - PyTest - 2.4-microCentOS-Stream-9x86_64✅ passed12.01.2026 10:08:0110min 2stest pipeline
RHEL8 - 2.4RHEL-8.10.0-Nightlyx86_64✅ passed12.01.2026 10:00:1012min 52stest pipeline
Fedora - PyTest - 2.4-microFedora-latestx86_64✅ passed12.01.2026 10:00:417min 51stest pipeline
RHEL10 - Unsubscribed host - 2.4RHEL-10-Nightlyx86_64✅ passed12.01.2026 10:00:0111min 27stest pipeline
CentOS Stream 9 - 2.4-microCentOS-Stream-9x86_64✅ passed12.01.2026 10:12:0817min 7stest pipeline
RHEL9 - PyTest - 2.4RHEL-9.6.0-Nightlyx86_64✅ passed12.01.2026 10:12:0716min 57stest pipeline
CentOS Stream 9 - 2.4CentOS-Stream-9x86_64✅ passed12.01.2026 10:12:2615min 51stest pipeline
RHEL10 - PyTest - 2.4RHEL-10-Nightlyx86_64✅ passed12.01.2026 10:08:2411min 45stest pipeline
RHEL10 - 2.4RHEL-10-Nightlyx86_64✅ passed12.01.2026 10:10:2712min 34stest pipeline
RHEL9 - 2.4RHEL-9.6.0-Nightlyx86_64✅ passed12.01.2026 10:14:0617min 33stest pipeline
RHEL8 - OpenShift 4 - 2.4RHEL-8.10.0-Nightlyx86_64✅ passed12.01.2026 10:14:2814min 18stest pipeline
RHEL8 - PyTest - OpenShift 4 - 2.4RHEL-8.10.0-Nightlyx86_64✅ passed12.01.2026 10:18:1017min 50stest pipeline
RHEL10 - OpenShift 4 - 2.4RHEL-10-Nightlyx86_64✅ passed12.01.2026 10:16:4913min 38stest pipeline
RHEL10 - PyTest - OpenShift 4 - 2.4RHEL-10-Nightlyx86_64✅ passed12.01.2026 10:16:1615min 57stest pipeline
RHEL9 - OpenShift 4 - 2.4RHEL-9.6.0-Nightlyx86_64✅ passed12.01.2026 10:18:0719min 56stest pipeline
RHEL9 - PyTest - OpenShift 4 - 2.4RHEL-9.6.0-Nightlyx86_64✅ passed12.01.2026 10:14:3520min 52stest pipeline

@phracek
Copy link
Member Author

phracek commented Jan 12, 2026

Fixed PyTest issue in $HTTPD_TLS_PATH.
Also pre-commit fixes are added.

[test-all]

pkhartsk
pkhartsk approved these changes Jan 12, 2026
View reviewed changes
Copy link
Contributor

@pkhartsk pkhartsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Took a while to discern what in the tests is cosmetic and what are actual changes. LGTM

@phracek phracek merged commit 7dc06b9 into sclorg:master Jan 12, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pkhartsk pkhartsk pkhartsk approved these changes

Assignees

No one assigned

Labels

pr/missing-review ready for review READY-to-MERGE

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@phracek @pkhartsk