Project

General

Profile

Actions

action #19540

closed

[migration] register_system types too fast for ppc64le

Added by RBrownSUSE almost 7 years ago. Updated over 6 years ago.

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

100%

Estimated time:
Difficulty:

Description

Now ppc64le tests are running more reliably we have found an issue in an existing test that was lost in the noise of earlier problems

Example: https://openqa.suse.de/tests/976761#step/register_system/18

This test has a type_string which is far too long to type quickly for the ppc64le architecture:

repos_check.pm:20 triggers a type_string "zypper lr --uri | awk -F \'|\' -v OFS=\' \' \'{ print $2,$3,$4,$NF }\' | tr -s \' \' | grep "SLE-SDK12-SP2:alnum:][:punct:-*Updates SLE-SDK12-SP2:alnum:][:punct:][:space:-Updates Yes http[s]://.*suse"', max_interval=250, wait_screen_changes=0"

Please fix the test to either

1) use wait_screen_changes=1 so the system only types after confirmation of each keypress
2) use a script instead of such a long type_string

Similar length type_strings probably need to be treated in a similar way

Thank you

Actions #1

Updated by RBrownSUSE almost 7 years ago

  • Description updated (diff)
Actions #2

Updated by mitiao almost 7 years ago

Richard, appreciated your fixes on Power worker. Thanks.
From the results of recent builds, I could see the migration jobs on ppc64le works much better than before.
Now I am working on poo#17204 to implement migration on aarch64, so I will fix it after that. :)

Actions #3

Updated by mitiao almost 7 years ago

  • Status changed from New to In Progress
Actions #4

Updated by mitiao almost 7 years ago

Based on previous work https://progress.opensuse.org/issues/17204

simply add wait_screen_change=1 for type_string works fine for this issue, but it would spent very long time in case of running many long commands, for example repos_check.pm test, if we register base product and plus multiple addons, the validate command:

 "zypper lr --uri | awk -F \'|\' -v OFS=\' \' \'{ print $2,$3,$4,$NF }\' | tr -s \' \' | grep "SLE-SDK12-SP2[[:alnum:][:punct:]]*-*Updates SLE-SDK12-SP2[[:alnum:][:punct:][:space:]]*-*Updates Yes http[s]*://.*suse"',

has to be run dozen times which may need hours to complete this single test.

An example of time spent on only check base product urls (5):
07:29:19.0316 15371 ||| finished repos_check sle12_online_migration at 2017-06-26 07:29:19 (1617 s)

So trying to script very long command to shorten typing length and provide 3 typing speed from type_string parameters to try to avoid the issue with execute time consideration.

Actions #6

Updated by mitiao over 6 years ago

  • Status changed from In Progress to Feedback
Actions #7

Updated by mitiao over 6 years ago

Feedback looks better than before, but not 100% solved, like this:

https://openqa.suse.de/tests/1056660

Need slower 'slow type'.

Actions #8

Updated by mitiao over 6 years ago

mitiao wrote:

Feedback looks better than before, but not 100% solved, like this:

https://openqa.suse.de/tests/1056660

Need slower 'slow type'.

PR submitted:
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/3269

Actions #9

Updated by mitiao over 6 years ago

  • % Done changed from 0 to 100

PR merged and will see the result of re-trigger

Actions #10

Updated by mitiao over 6 years ago

  • Status changed from Feedback to Resolved

mitiao wrote:

PR merged and will see the result of re-trigger

Looks fine with the re-trigger's result:
https://openqa.suse.de/tests/1064225#step/zypper_lr/48
Set to resolved.

Actions #11

Updated by okurz over 6 years ago

This is an autogenerated message for openQA integration by the openqa_review script:

This bug is still referenced in a failing openQA test: om_smt_sles12sp2_ha+sdk+all_module_minimal_patch_by_yast
http://openqa.suse.de/tests/1062798

Actions #12

Updated by okurz over 6 years ago

This is an autogenerated message for openQA integration by the openqa_review script:

This bug is still referenced in a failing openQA test: om_smt_sles12sp2_ha+sdk+all_module_minimal_patch_by_yast
https://openqa.suse.de/tests/1062798

Actions

Also available in: Atom PDF