-
Notifications
You must be signed in to change notification settings - Fork 73
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
tests: invert the conditions for checking modular vs monolithic daemon distro #449
Conversation
Thanks!
You need to force-push for the pycodestyle error anyway. |
dec2d85
to
a123104
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.
LGTM, bots have the last word.
@@ -57,7 +57,7 @@ systemctl start firewalld | |||
firewall-cmd --add-service=cockpit --permanent | |||
firewall-cmd --add-service=cockpit | |||
|
|||
if grep -Eq 'PLATFORM_ID=.*(f35)' /etc/os-release; then | |||
if grep -Eq 'PLATFORM_ID=.*(f35|f36|el9)' /etc/os-release; then |
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.
I think we should invert that too, and not do it on f34 and el8. But that can also be a follow-up the next time we have to touch this file.
This flake is fixed by #448. Let's rebase this PR after that lands, to not totally ruin our test stats. |
0b7b03c
to
c25c711
Compare
c25c711
to
f2acb18
Compare
…n distro We should have a list that shrinks - not grows.
…cket manual start
f2acb18
to
7b4cedc
Compare
Branch was out of date, so packit srpm build failure. Here it actually matters, so I pushed a rebase. |
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.
Thanks! The packit f35 failure is something entirely different (timeout? VM running out of memory?) but the tests until the hang look good.
tests succeeded also downstream. |
We should have a list that shrinks - not grows