Project

General

Profile

Actions

action #115466

closed

Fix synchronization issue in grub_test for migration with AutoYaST from SLE-12-SP5

Added by tinawang123 about 2 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
Start date:
2022-08-18
Due date:
% Done:

0%

Estimated time:

Description

Motivation

Failed case: https://openqa.suse.de/tests/9353685#step/grub_test/2
Successful case: https://openqa.suse.de/tests/8753498#step/grub_test/3

Acceptance criteria

AC1: Fix synchronization issue taking into account that this module is heavily used.
AC2: File a bug if any

Suggestions

Consider what is special in this scenario and if makes more sense to create a additional module for this behavior.
Perhaps we need to touch code in 'lib/grub_utils.pm' file.
Seems that there is a different sequence of needles, this one doesn't follow exactly like in the successful one:
https://openqa.suse.de/tests/9353685#step/grub_test/2

Actions #1

Updated by JERiveraMoya about 2 years ago

  • Description updated (diff)
  • Target version set to Current
Actions #2

Updated by JERiveraMoya about 2 years ago

  • Subject changed from test fails in grub_test - it should not check ' inst-bootmenu-boot-harddisk ' needle, just wait system boot to Fix synchronization issue for migration with AutoYaST from SLE-12-SP5
  • Description updated (diff)
Actions #3

Updated by JERiveraMoya about 2 years ago

  • Subject changed from Fix synchronization issue for migration with AutoYaST from SLE-12-SP5 to Fix synchronization issue in grub_test for migration with AutoYaST from SLE-12-SP5
  • Description updated (diff)
Actions #4

Updated by JERiveraMoya about 2 years ago

  • Tags deleted (qe-yast-refinement)
  • Status changed from New to Workable
Actions #5

Updated by leli about 2 years ago

  • Status changed from Workable to In Progress
  • Assignee set to leli
Actions #6

Updated by leli about 2 years ago

'-' if (check_var("BOOTFROM", "d") && check_var("AUTOUPGRADE") && check_var("PATCH")) {
'+' if (check_var("BOOTFROM", "d") && get_var('AUTOUPGRADE') && get_var('PATCH')) {

I think this is the current reason why this case failed here. wrong using check_var changed to correct get_var while this test flow is wrong originally but never entered here for the wrong using of check_var.

For autoyast, it should be inst-bootmenu -> inst-bootmenu-boot-harddisk -> grub2.

So the code need to be updated to correct flow.

Actions #7

Updated by leli about 2 years ago

Created a branch to fix this issue, wait http://openqa.nue.suse.com/tests/9475375#live to verify

Actions #9

Updated by leli about 2 years ago

  • Status changed from In Progress to Resolved

PR merged.

Actions

Also available in: Atom PDF