Actions
action #71896
closed[y] Expert partitioning framework refactoring
Status:
Rejected
Priority:
Normal
Assignee:
Category:
-
Target version:
QA (public, currently private due to #173521) - future
Start date:
2020-09-25
Due date:
% Done:
0%
Estimated time:
Difficulty:
Description
As we are expanding and adding new subroutines in the framework we need to consider some refactoring to avoid duplication and make it more usable.
Things to refactor:
- _set_partitioning subroutine
- this does not work for all the partitioning options
- _encrypt_with_lvm subroutine
- The encryption should work for a higher level for all the Pages that implement encryption
- select_item_in_system_view_table subroutine
- replace if-else with state design or find another solution.
- Consider expansion?
- naming of the Pages
- The pages use the name of the page that they access but it is hard to tell if which proposal(current or suggested) comes from
Let start with small things because more can come up but design is something that need to be always a task in progress but we cant cover everything.
Files
Updated by ybonatakis about 4 years ago
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/11037 does not resolve the duplication issue with the pages.
Similar Pages with different names can be subclassed by another higher class to avoid duplicattion
an example
Updated by riafarov about 4 years ago
- Status changed from New to Rejected
- Assignee set to riafarov
Rejecting as we are going to use REST API with libyui here instead of further development of the partitioning functionality.
Actions