Add valid bugref to power_action_utils

[DrMullings]

Now the default value of the soft timeout is bsc#1055462
"openQA test fails in shutdown - machine fails to shutdown within
one minute"

• Related ticket: https://progress.opensuse.org/issues/41183
• Related Bug: https://bugzilla.suse.com/show_bug.cgi?id=1055462

[okurz]

@OleksandrOrlov ?

[OleksandrOrlov]

I guess we shouldn't make the default value to be contained the bugref.

The soft-fail raised for that case because the test suite contains LIVECD variable. And the appropriate issue is https://bugzilla.suse.com/show_bug.cgi?id=1096241.

So, my suggestion is to leave default value as is ('No bugref specified') and to just add 'bugref' to soft_fail_data for LIVECD case:

    if (get_var('LIVECD')) {
        $soft_fail_data = {bugref=>'bsc#1096241', soft_timeout => 60, timeout => $shutdown_timeout *= 4};
    }

Actually it should already be specified. It seems like I missed this 'bugref' while implemented the subroutine.

[OleksandrOrlov]

I guess we shouldn't make the default value to be contained the bugref.

The soft-fail raised for that case, because the test suite contains LIVECD variable. And the appropriate issue is https://bugzilla.suse.com/show_bug.cgi?id=1096241.

So, my suggestion is to leave default value as is ('No bugref specified') and to just add 'bugref' to the soft_fail_data for LIVECD case:

    if (get_var('LIVECD')) {
        $soft_fail_data = {bugref=>'bsc#1096241', soft_timeout => 60, timeout => $shutdown_timeout *= 4};
    }

Actually it should already be specified. It seems like I missed this 'bugref' while implemented the subroutine.

[OleksandrOrlov]

I guess we shouldn't make the default value to be contained the bugref.

The soft-fail raised for that case, because the test suite contains LIVECD variable. And the appropriate issue is https://bugzilla.suse.com/show_bug.cgi?id=1096241.

So, my suggestion is to leave default value as is ('No bugref specified') and to just add 'bugref' to the soft_fail_data for LIVECD case:

    if (get_var('LIVECD')) {
        $soft_fail_data = {bugref=>'bsc#1096241', soft_timeout => 60, timeout => $shutdown_timeout *= 4};
    }

Actually it should already be specified. It seems like I missed this 'bugref' while implemented the subroutine.

[DrMullings]

@OleksandrOrlov statement was that we should keep the default as is and just add a bugref for LIVECD

[DrMullings]

@OleksandrOrlov statement was that we should keep the default as is and just add a bugref for LIVECD

[DrMullings]

Updated it accordingly

[jlausuch]

This merge is making Travis fail:
https://travis-ci.org/os-autoinst/os-autoinst-distri-opensuse/builds/444979772?utm_source=github_status&utm_medium=notification

[DrMullings]

Weird, locally I do not get errors and also the spacing and stuff look also fine
Nope there are missing spaces around the arrow, will provide fix after the review meeting

[DrMullings]

#6028
Fixes this