Project

General

Profile

Actions

action #26928

closed

coordination #23650: [sle][functional][ipmi][epic][u] Fix test suite gnome to work on ipmi 12-SP3 and 15 (WAS: test fails in boot_from_pxe - connection refused trying to ipmi host over ssh?)

[sle][functional]gnome@64bit-ipmi using VNC installation can not reach VNC server anymore -> regression on a29497af?

Added by okurz over 6 years ago. Updated almost 5 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Bugs in existing tests
Target version:
-
Start date:
2017-10-20
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

observation

gnome@64bit-ipmi fails to connect to the VNC server for VNC based installation. The test incorrectly looks for the SSH server to be ready and then try to connect to the VNC server. But this worked in before.

reproducible

Every time

expected result

last good: sle-15-Leanos-DVD-x86_64-Build248.1-gnome@64bit-ipmi seems to find the ssh server just fine on openqaw1:1 and then connects to VNC.

problem

  • H1 regression in product -> E1-1 crosscheck with older build, e.g. 248.1
  • H2 test is incorrect and fails now because product changed slightly -> E2-1 change test to make more sense, e.g. don't look for ssh server when you want to connect to a VNC server

further details

see parent ticket for details

Seems that the following commit introduced a dubious change although tests were still fine after that:

commit a29497af
Author: alice <xlai@suse.com>
Date:   Tue Jul 25 13:54:15 2017 +0800

    Virtualization: fix autoyast installation failure on 3rd generation ipmi backend for sle11sp4

diff --git a/tests/boot/boot_from_pxe.pm b/tests/boot/boot_from_pxe.pm
index 4880be37..279d1622 100644
--- a/tests/boot/boot_from_pxe.pm
+++ b/tests/boot/boot_from_pxe.pm
@@ -55,7 +55,7 @@ sub run {
     type_string "vga=791 ",   $type_speed;
     type_string "Y2DEBUG=1 ", $type_speed;

-    if (check_var('BACKEND', 'ipmi')) {
+    if (check_var('BACKEND', 'ipmi') && !check_var('AUTOYAST', '1')) {
         my $cmdline = '';
         if (check_var('VIDEOMODE', 'text')) {
             $cmdline .= 'ssh=1 ';    # trigger ssh-text installation
@@ -79,7 +79,9 @@ sub run {
     }

     type_string "console=$serialdev,115200 ", $type_speed;    # to get crash dumps as text
-    type_string "console=tty ",               $type_speed;
+    if (!(check_var('BACKEND', 'ipmi') && get_var('AUTOYAST'))) {
+        type_string "console=tty ", $type_speed;
+    }

See #23650#note-17 for details: "Removing the additional "console=tty" and just let "console=ttyS1,115200" in there results in the expected output on the serial console."

Actions #1

Updated by okurz over 6 years ago

  • Description updated (diff)
  • Due date set to 2017-10-25
  • Status changed from New to In Progress
Actions #2

Updated by okurz over 6 years ago

  • Description updated (diff)
  • Assignee set to nicksinger
Actions #3

Updated by okurz over 6 years ago

conducting E1-1 with

openqa_clone_job_osd 1167409 _GROUP=0

-> https://openqa.suse.de/t1228402

Actions #4

Updated by okurz over 6 years ago

  • Assignee changed from nicksinger to okurz
Actions #5

Updated by nicksinger over 6 years ago

Well, we relied on a race condition here so IMHO it does not matter why it worked in before and we have to fix it. https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/3770/commits/e629652fa4e32dae304894722d248bbd8ad32e8b fixes this.

Actions #6

Updated by okurz over 6 years ago

After needle adaptions and so on retriggered build 310.1 as 318 is busted -> https://openqa.suse.de/t1232080

Actions #7

Updated by okurz over 6 years ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by SLindoMansilla almost 5 years ago

  • Due date deleted (2017-10-25)
Actions

Also available in: Atom PDF