action #70153
closed
[y] system_state should upload collected data
Added by ybonatakis over 3 years ago.
Updated over 3 years ago.
Category:
Enhancement to existing tests
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.
- Due date changed from 2020-08-25 to 2020-09-08
- Category set to Enhancement to existing tests
- Target version set to future
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.
- Due date changed from 2020-09-08 to 2020-09-22
- Description updated (diff)
- Status changed from New to Workable
- Estimated time set to 3.00 h
- Status changed from Workable to In Progress
- Assignee set to ybonatakis
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.
- Due date changed from 2020-09-22 to 2020-10-06
- Status changed from In Progress to Feedback
- Status changed from Feedback to Resolved
Also available in: Atom
PDF