action #54770

[functional][y] Rename test variable WORKAROUND_MODULES as is used in staging as normal behavior

Added by okurz 7 months ago. Updated 5 months ago.

Status:ResolvedStart date:29/07/2019
Priority:LowDue date:24/09/2019
Assignee:ybonatakis% Done:

0%

Category:Enhancement to existing testsEstimated time:3.00 hours
Target version:QA - future
Difficulty:
Duration: 42

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

History

#1 Updated by riafarov 7 months 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.

#2 Updated by riafarov 7 months 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 10/09/2019
  • Target version set to future

#3 Updated by okurz 7 months 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 :)

#4 Updated by riafarov 6 months ago

  • Status changed from New to Workable
  • Estimated time set to 3.00

#5 Updated by ybonatakis 6 months ago

  • Status changed from Workable to In Progress
  • Assignee set to ybonatakis

#6 Updated by ybonatakis 6 months 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.

#7 Updated by ybonatakis 6 months ago

  • Status changed from In Progress to Feedback

#8 Updated by riafarov 6 months ago

  • Description updated (diff)

#9 Updated by riafarov 6 months ago

  • Due date changed from 10/09/2019 to 24/09/2019

Yanis has pointed out that we can get rid of the variable, but need to set proper settings in the test suite.

#10 Updated by riafarov 5 months ago

  • Status changed from Feedback to Resolved

I guess we can resolve this one, hooray ;).

#11 Updated by ybonatakis 5 months ago

The PR is still in WIP and with various changes due to the experiments in order to make it work for xen.

Also available in: Atom PDF