Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run list-timers before shutdown #15294

Merged

Conversation

michaelgrifalconi
Copy link
Contributor

Run list-timers before shutdown and print the result to the console

@@ -13,6 +13,9 @@ use power_action_utils 'power_action';
use utils;

sub run {
my $self = shift;
select_console('root-console');
script_run('systemctl list-timers --all');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would recommend to replace this with $self->save_and_upload_log('systemctl list-timers --all', '/tmp/all_timers.log', {screenshot => 1}); . that will store a file in the Logs and Assets wdyt?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, upload won't work because shutdown is run after cleanup_before_shutdown that takes down the network. See

if ($network_status =~ /wicked/) {
systemctl 'stop network.service';
systemctl 'stop wickedd.service';
assert_script_run('ls /var/lib/wicked/');
save_screenshot;
script_run('rm -f /var/lib/wicked/*.xml');

and https://openqa.suse.de/tests/9230015#step/shutdown/6

I guess it should be enough to print it since it's a rather small output and easy to read, but open to ideas

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@michaelgrifalconi michaelgrifalconi changed the title [WIP] Run list-timers before shutdown Run list-timers before shutdown Aug 1, 2022
@michaelgrifalconi michaelgrifalconi merged commit a2a4b1f into os-autoinst:master Aug 1, 2022
jlausuch pushed a commit to jlausuch/os-autoinst-distri-opensuse that referenced this pull request Aug 1, 2022
jlausuch pushed a commit to jlausuch/os-autoinst-distri-opensuse that referenced this pull request Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants