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

Add workaround for mdmonitor.service #12104

Merged
merged 1 commit into from
Mar 10, 2021
Merged

Conversation

ge0r
Copy link
Member

@ge0r ge0r commented Mar 10, 2021

Due to bsc#1183229, newer versions of mdadm render the mdmonitor service unable to start and stay running.
A workaround until the issue is fixed is to install the last version where mdadm allowed mdmonitor to run successfully.

Due to bsc#1183229, newer versions of mdadm render the mdmonitor service
unable to start and stay running.
A workaround until the issue is fixed is to install the last version
where mdadm allowed mdmonitor to run successfully.
@ge0r ge0r requested review from dzedro and punkioudi March 10, 2021 09:35
record_soft_failure("qaperf.service fails to start bsc#1182914");
assert_script_run("systemctl reset-failed qaperf.service");
# Reset the failed state of all units so that only new failures are recorded
assert_script_run("systemctl reset-failed");
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't you want to run this after mdadm is fixed ? Don't know if it matters

Copy link
Member Author

Choose a reason for hiding this comment

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

No, this is really because there are already some recorded failures, eg for qaperf.service which fails because root/qaset/config is only available later. So instead of systemd testsuite to think those are failing, we need to reset the failed states and rerun them then.

Copy link
Member Author

@ge0r ge0r Mar 10, 2021

Choose a reason for hiding this comment

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

Also just to mention, mdmonitor.service, so mdadm with "--monitor" and "--scan", never fails, does start, but does not remain running, so it is not a failed state per se

@ge0r ge0r requested a review from dzedro March 10, 2021 10:50
Copy link
Contributor

@punkioudi punkioudi left a comment

Choose a reason for hiding this comment

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

lgtm!

@punkioudi punkioudi merged commit 0098585 into os-autoinst:master Mar 10, 2021
@czerw
Copy link
Contributor

czerw commented Mar 12, 2021

Sorry, but you impacted multipath testing https://openqa.suse.de/tests/5657094#step/kernel_multipath/143.
Please fix it https://progress.opensuse.org/issues/89966

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants