Project

General

Profile

Actions

action #70153

closed

[y] system_state should upload collected data

Added by ybonatakis over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Enhancement to existing tests
Target version:
Start date:
2020-08-18
Due date:
2020-10-06
% Done:

0%

Estimated time:
3.00 h
Difficulty:

Description

The system_state.pm[0] is just collecting data from the process list and system loads. I think it would be meaningful to upload the collected data to the OpenQA logs and assets, wherever this is used. Doing such we can review those data afterwards as it is tricky to add automatic validation on the output.

[0] https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/tests/console/system_state.pm

So basically we can:

  • Fix the text uploading the file.
  • Additionally, perform a bit of research about where the post fail is trigger with ps and from which class you need to inherit to have it.
  • Consider if we need this module.
Actions #1

Updated by riafarov over 4 years ago

  • Due date changed from 2020-08-25 to 2020-09-08
  • Category set to Enhancement to existing tests
  • Target version set to future
Actions #2

Updated by riafarov over 4 years ago

I'm not sure why would we need this, except some special conditions, and I believe we measure systemload in the post-fail hooks. But definitely makes sense to make data more visible, at least as record info, so it's easy to copy-paste, for instance.

Actions #3

Updated by riafarov over 4 years ago

  • Due date changed from 2020-09-08 to 2020-09-22
Actions #4

Updated by JERiveraMoya over 4 years ago

  • Description updated (diff)
  • Status changed from New to Workable
  • Estimated time set to 3.00 h
Actions #5

Updated by ybonatakis over 4 years ago

  • Status changed from Workable to In Progress
  • Assignee set to ybonatakis
Actions #6

Updated by ybonatakis over 4 years ago

https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/11044

there is save_system_logs subroutine which collects the running processes and the cpu loads
during the post_fail_hook of each installation module. For that reason i remove the system_state
from yast modules. Although i enhanced the module and it can be used in case of a successful test
to provide those info in the logs of the job.

Actions #7

Updated by riafarov over 4 years ago

  • Due date changed from 2020-09-22 to 2020-10-06
Actions #8

Updated by ybonatakis over 4 years ago

  • Status changed from In Progress to Feedback
Actions #9

Updated by ybonatakis about 4 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF