-
Notifications
You must be signed in to change notification settings - Fork 269
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 test scenario to verify AY installation while using disk as MD member #7913
Add test scenario to verify AY installation while using disk as MD member #7913
Conversation
The commit extracts the part of the test_setup function related to autoyast initialization to the separate function and makes the function utilitarian by placing it in lib/autoyast.pm
The commit adds the function to have ability to find all the nodes in xml file by xpath and returns them as an array. That allows to manipulate with the found nodes in the manner that is required for the certain test (e.g. get values from the nodes, count them etc.)
</partition> | ||
<partition> | ||
<create config:type="boolean">true</create> | ||
<mount>/boot</mount> |
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.
Shall we implement feature to inject values from test_data
structure, similarly to what we've done with vars.json?
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 never thought about that, but it might be great to have the feature I guess, as it will allow to change some data just in one place, rather then check all the places where it is used.
But, I guess it should be done in scope of another task?
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.
Sure. Just wanted to hear your opinion. I will create a ticket for that.
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
The commit adds all the required perl modules and the yaml schedule to test autoyast installation while using disk as MD member.
fdc4b25
to
b9c63ef
Compare
Please, enable the test suite for the relevant distributions and versions. |
Enabled for YaST Job group, Sle15, 64bit. |
The PR adds all the required perl modules and the yaml schedule to test autoyast installation while using disk as MD member.
After being merged, the PR requires the following configuration to be updated on openQA:
Test Suites:
autoyast_disk_as_md_member
Job Groups->SLE15->YaST->new test:
autoyast_disk_as_md_member