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
udev: fix importing program result #18885
udev: fix importing program result #18885
Conversation
This fixes a bug introduced by 28a5065. Fixes RHBZ#1935062 (https://bugzilla.redhat.com/show_bug.cgi?id=1935062).
66a2e2b
to
86c783d
Compare
test/units/testsuite-59.sh
Outdated
|
||
echo OK > /testok | ||
|
||
exit 0 |
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 do see that this doesn't fit too well anywhere else, but maybe this could be combined with one of the other tests? Spinning up a VM for this feels wasteful.
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.
Test-17 maybe?
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.
Ok
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.
Tests are merged. PTAL.
002f205
to
3509c78
Compare
For now just ignore the TEST-58 and TEST-56 failures, I'll fix it in CentOS CI once this PR is merged. |
coredump issue in TEST-17-UDEV seems not related. See https://github.com/systemd/systemd-centos-ci/blob/e5b63491d8d7447770fb39e37396c492ad496935/agent/testsuite.sh#L168-L172. These lines also needs to be updated. |
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
Fixes a bug introduced by 28a5065.
Fixes RHBZ#1935062 (https://bugzilla.redhat.com/show_bug.cgi?id=1935062).