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

Fix false positive in validate_raid #8438

Merged
merged 1 commit into from Sep 13, 2019

Conversation

jknphy
Copy link
Contributor

@jknphy jknphy commented Sep 12, 2019

@ggardet
Copy link
Collaborator

ggardet commented Sep 12, 2019

Could you provide VRs on aarch64 as well, please?

@jknphy jknphy force-pushed the fix_validate_raid branch 5 times, most recently from 9de664f to 5a29985 Compare September 13, 2019 06:48
$btrfs, $swap,
);
if (is_sle('<=15-SP1') || get_var('STAGING') ||
is_leap() || (is_tumbleweed() && check_var('FLAVOR', 'NET'))) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am wondering why NET flavor should be handled separately?

Copy link
Contributor Author

@jknphy jknphy Sep 13, 2019

Choose a reason for hiding this comment

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

DVD TW have both arrays in the same raid level while NET TW has this 'special' RAID0 for swap.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ok, I see it in openQA. But it looks like a bug to me. DVD and NET should have the same behavior, no?

Copy link
Contributor Author

@jknphy jknphy Sep 13, 2019

Choose a reason for hiding this comment

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

Now I have checked better, I can see that this is an artificial difference that we have created, we use different module to do the raid, an OOP module installation/partitioning/raid_gpt and a legacy one installation/partitioning_raid. I would leave that out-of-scope of this PR and I will share with the team to fix in the future. Ticket created: https://progress.opensuse.org/issues/56870

@jknphy jknphy changed the title [WIP] Fix false positive in validate_raid Fix false positive in validate_raid Sep 13, 2019
@rwx788 rwx788 merged commit 6774c33 into os-autoinst:master Sep 13, 2019
@nilxam
Copy link
Member

nilxam commented Sep 16, 2019

I'm afraid this failure https://openqa.opensuse.org/tests/1032687#step/validate_raid/9 is related to this change?

@jknphy
Copy link
Contributor Author

jknphy commented Sep 16, 2019

I'm afraid this failure https://openqa.opensuse.org/tests/1032687#step/validate_raid/9 is related to this change?

Thanks. I missed that one: #8445

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