action #54770
closed[functional][y] Rename test variable WORKAROUND_MODULES as is used in staging as normal behavior
0%
Description
Motivation¶
Many test suites on osd define WORKAROUND_MODULES mentioning bsc#1054375 which is VERIFIED FIXED by now – since two years already ;) . So I guess we need to either remove the WORKAROUND_MODULES part or update the test suite comments
As we can see name is confusing. It was introduced as temporary thing, but is used in SLE 15 staging
Acceptance criteria¶
- AC1: WORKAROUND_MODULES is renamed to reflect it's purpose
- AC2: Variable description is updated (https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/variables.md)
- AC3: All staging test suites are adjusted accordingly. NOTE: some of the settings might be set for mediums!
Updated by riafarov over 5 years ago
This is used in all staging tests, the main problem is that we thought it's temporary, so used wrong naming =). I will highjack this ticket to rename the variable.
Updated by riafarov over 5 years ago
- Subject changed from [functional][y] Evaluate if test variable WORKAROUND_MODULES is still necessary, bsc#1054375 is VERIFIED FIXED to [functional][y] Rename test variable WORKAROUND_MODULES as is used in staging as normal behavior
- Description updated (diff)
- Due date set to 2019-09-10
- Target version set to future
Updated by okurz over 5 years ago
thank you. I don't even consider this hijacking. I recommend to keep the original AC still in place though with "suite settings do not mention the above bug anymore but a more recent one" but you can extend that to " … or no bug" of course :)
Updated by riafarov over 5 years ago
- Status changed from New to Workable
- Estimated time set to 3.00 h
Updated by ybonatakis over 5 years ago
- Status changed from Workable to In Progress
- Assignee set to ybonatakis
Updated by ybonatakis over 5 years ago
I run two test(x86_64) against sle12 and sle15 without the WORKAROUND_MODULES and both work properly. As the AC1 requires to rename it, I want to ask if i can go on and remove it completely.
Updated by ybonatakis over 5 years ago
- Status changed from In Progress to Feedback
Updated by riafarov over 5 years ago
- Due date changed from 2019-09-10 to 2019-09-24
Yanis has pointed out that we can get rid of the variable, but need to set proper settings in the test suite.
Updated by riafarov over 5 years ago
- Status changed from Feedback to Resolved
I guess we can resolve this one, hooray ;).
Updated by ybonatakis over 5 years ago
The PR is still in WIP and with various changes due to the experiments in order to make it work for xen.