Project

General

Profile

Actions

action #19226

closed

[regression] script run dies

Added by dimstar almost 7 years ago. Updated almost 7 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Regressions/Crashes
Target version:
-
Start date:
2017-05-18
Due date:
% Done:

0%

Estimated time:

Description

Test run used for analysis: https://openqa.opensuse.org/tests/405291#step/network_configuration/7

the script run correctly output "1Q7_W-0-" in serial0.txt, indicating that the command was successful.

From autoinst.txt though:

Argument "1Q7_W-0-\r\n" isn't numeric in numeric eq (==) at /usr/lib/os-autoinst/testapi.pm line 734.

This issue was introduced with https://github.com/os-autoinst/os-autoinst/commit/3bba01e77adf5dea58c1cbd538e74f8990c25742, where we now have:

     my $ret = wait_serial("$str-\\d+-", $wait);
     _handle_script_run_ret($ret, $cmd);

$ret will contain the entire string as returned on the serial console (e.g. 1Q7_W-0), which is then compared in _handle_script_run_ret at:

    croak $die_msg unless ($ret == 0);

$ret being the entire string from the serial, not stripped to the actual return code, causes this error


Related issues 1 (0 open1 closed)

Related to openQA Project - action #19224: Argument isn't numeric in numeric eq (==) at /usr/lib/os-autoinst/testapi.pm line 734.Resolvedokurz2017-05-18

Actions
Actions #1

Updated by okurz almost 7 years ago

  • Related to action #19224: Argument isn't numeric in numeric eq (==) at /usr/lib/os-autoinst/testapi.pm line 734. added
Actions #2

Updated by okurz almost 7 years ago

  • Status changed from New to In Progress
  • Assignee set to okurz
Actions #3

Updated by okurz almost 7 years ago

  • Project changed from openQA Tests to openQA Project
  • Category changed from Bugs in existing tests to Regressions/Crashes
  • Status changed from In Progress to Feedback

https://github.com/os-autoinst/os-autoinst/pull/779 from mnowak should fix it. Waiting for deployment.

Actions #4

Updated by okurz almost 7 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF