action #44753
closedcoordination #44054: [functional][y][epic] Develop maintainable and scalable test framework
[functional][y] Refactor partitioning_lvm test module using new framework design
0%
Description
See motivation in the parent ticket.
Acceptance criteria¶
- Fully working solution is provided, feedback is collected and follow-up tasks are evaluated
Suggestions¶
- Understand what test really does
- Separate the behavior by versions of the partitioner (Sle12 SP4 has old partitioner, Sle15 Sp0 - partitioner with changed UI, Sle15 SP1, Leap15.1, TW - with more changes in UI);
- Test module should call same behavior methods, but they should be implemented differently, depending on the version of partitioner (e.g. Use enable_encryption() method in the test module without caring of how it is actually implemented).
- PR should aim DoD we have for other tickets, but might get not get merged
- PR should be easy to review (relevant splits per commit)
Updated by okurz over 5 years ago
- Target version set to Milestone 21
ok to work on this if you are motivated but we should keep the effort (story points) and whole sprint scope in mind so we might need you to focus on other tasks first -> sprint planning to decide
Updated by riafarov over 5 years ago
- Description updated (diff)
- Status changed from New to Workable
- Estimated time set to 8.00 h
Updated by riafarov over 5 years ago
- Due date changed from 2018-12-18 to 2019-01-15
- Target version changed from Milestone 21 to Milestone 22
Concept is developed, need to identify differences for the distributions, do actual split and verification runs.
Updated by oorlov over 5 years ago
- Status changed from In Progress to Feedback
Updated by riafarov over 5 years ago
- Due date changed from 2019-01-15 to 2019-01-29
oorlov plans to schedule some meeting to demo and explain idea better and address feedback comments.
Updated by oorlov over 5 years ago
- Due date changed from 2019-01-29 to 2019-02-12
oorlov will address all the comments from the feedback and make it work for all distributions
Updated by okurz about 5 years ago
PR was merged. Please update with next plans in the parent ticket. And I guess we can close this ticket now?
Updated by riafarov about 5 years ago
We need to hot patch https://openqa.suse.de/tests/2453258#step/encrypt_lvm/3
Updated by riafarov about 5 years ago
- Status changed from Feedback to Resolved
The minor regressions will be addressed in #47381, we also plan to document the skeleton and demo how to use this approach to simplify test maintenance.
Updated by riafarov about 5 years ago
- Related to action #47381: [sle][functional][y] test fails in encrypt_lvm_ignore_existing - enable LVM added