Skip to content
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

Debug firewalld startup on rawhide TF #19140

Closed
wants to merge 1 commit into from

Conversation

martinpitt
Copy link
Member

Trying to debug the bodhi rawhide and c9s PR failures. These didn't happen upstream, nor do they happen in bodhi/c9s gating for cockpit-machines, which does exactly the same thing.

+ systemctl start firewalld
+ firewall-cmd --add-service=cockpit --permanent
FirewallD is not running

firewalld hasn't changed in c9s in 8 months, systemd did.

Checking in rawhide first. Updating firewalld from 2.0.0-2.fc39 to 2.0.0-3.fc39 does not reproduce it. Updating systemd from 254~rc2-4.fc39 to 254~rc2-5.fc39 also seems to work, at least locally.

There is some weird inconsistency between the above cockpit rawhide run, which installed firewalld package; and e.g. the cockpit-machines run which had it pre-installed apparently.

So I tried to boot a rawhide image, uninstall firewalld, dnf update, reboot, dnf install -y firewalld, firewall-cmd --add-service=cockpit --permanent. This works.

So let's see if I also get this upstream now.

@martinpitt martinpitt added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Jul 26, 2023
@martinpitt
Copy link
Member Author

So no joy -- that run has firewall preinstalled, and it works fine.

@martinpitt martinpitt closed this Jul 26, 2023
@martinpitt martinpitt deleted the firewall-debug branch July 26, 2023 12:58
@martinpitt martinpitt restored the firewall-debug branch July 26, 2023 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-test For doc/workflow changes, or experiments which don't need a full CI run,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant