Differential D1003

use tty6 for installedtest post_fail_hook
ClosedPublic
Actions

Authored by adamwill on Sep 15 2016, 10:04 PM.

Details

Reviewers
garretraziel
jskladan
Commits
rOPENQATESTSaf00b71790a4: use tty6 for installedtest post_fail_hook
Summary

tty2 is where wayland desktop sessions run. I think it makes
sense to use a high tty for the post_fail_hook, so we know the
lower ones can be used by the tests...

Test Plan

Run a Workstation post-install test that fails, check
the hook works.

Diff Detail

Repository
rOPENQATESTS os-autoinst-distri-fedora
Branch
postinstall-tty6
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 843
Build 843: arc lint + arc unit
adamwill retitled this revision from to use tty6 for installedtest post_fail_hook.Sep 15 2016, 10:04 PM
adamwill updated this object.
adamwill edited the test plan for this revision. (Show Details)
adamwill added reviewers: garretraziel, jskladan.
garretraziel accepted this revision.Sep 19 2016, 8:52 AM
This revision is now accepted and ready to land.Sep 19 2016, 8:52 AM
adamwill closed this revision.Sep 20 2016, 5:52 PM

this landed onto develop somehow, but I don't remember doing it? did someone else? oh well.

Revision Contents

PathPackages
Mlib/installedtest.pm (2 lines)

Diff 2561

View Options

lib/installedtest.pm

Show All 17 Lines12sub root_console {
18 18​
19 send_key "ctrl-alt-f$args{tty}"; 19​ send_key "ctrl-alt-f$args{tty}";
20 $self->console_login(check=>$args{check}); 20​ $self->console_login(check=>$args{check});
21} 21​}
22 22​
23sub post_fail_hook { 23​sub post_fail_hook {
24 my $self = shift; 24​ my $self = shift;
25 25​
26 $self->root_console(tty=>2); 26​ $self->root_console(tty=>6);
27 27​
28 # If /var/tmp/abrt directory isn't empty (ls doesn't return empty string) 28​ # If /var/tmp/abrt directory isn't empty (ls doesn't return empty string)
29 my $vartmp = script_output "ls /var/tmp/abrt"; 29​ my $vartmp = script_output "ls /var/tmp/abrt";
30 if ($vartmp ne '') { 30​ if ($vartmp ne '') {
31 # Upload /var/tmp ABRT logs 31​ # Upload /var/tmp ABRT logs
32 script_run "cd /var/tmp/abrt && tar czvf tmpabrt.tar.gz *"; 32​ script_run "cd /var/tmp/abrt && tar czvf tmpabrt.tar.gz *";
33 upload_logs "/var/tmp/abrt/tmpabrt.tar.gz"; 33​ upload_logs "/var/tmp/abrt/tmpabrt.tar.gz";
34 } 34​ }
▲ Show 20 Lines • Show All 72 Lines • Show Last 20 Lines