Project

General

Profile

action #70873

Updated by okurz over 3 years ago

## Observation ### observation 
 It looks like the videoencoder could not be started or got stuck before it could create the video file: 

 ``` 
 [2020-09-02T04:31:14.339 CEST] [debug] Backend process died, backend errors are reported below in the following lines: 
 Encoder not accepting data at /usr/lib/os-autoinst/backend/baseclass.pm line 157. 

 [2020-09-02T04:31:14.341 CEST] [info] ::: OpenQA::Qemu::Proc::save_state: Saving QEMU state to qemu_state.json 
 [2020-09-02T04:31:16.395 CEST] [debug] Passing remaining frames to the video encoder 
 [2020-09-02T04:31:16.395 CEST] [debug] Unable to pass remaining frames to video encoder: Encoder not accepting data at /usr/lib/os-autoinst/backend/baseclass.pm line 157. 

 [2020-09-02T04:31:16.395 CEST] [debug] Waiting for video encoder to finalize the video 
 [2020-09-02T04:31:16.395 CEST] [debug] The built-in video encoder (pid 7856) terminated 
 ``` 

 job reference: https://openqa.suse.de/tests/4628195 

 ## Steps to reproduce 

 * Look for references of this ticket detected on o3 and osd: 

 ``` ### suggestions 
 for i in o3 osd; do ssh $i "sudo -u geekotest psql openqa -c \"select jobs.id,t_finished,state,result,test,reason,host from jobs, comments, workers where t_finished >= '2020-01-01' and jobs.assigned_worker_id = workers.id and jobs.id = comments.job_id and comments.text ~ 'poo#70873' order by t_finished desc limit 20;\""; done 
 ``` 


 ## Suggestions 

 * DONE: ~~Let's Let's see whether it happens more often. Maybe the error handling when starting the video encoder could be improved.~~ yes, happened multiple times on both o3 and osd 
 * Check our test code coverage for the code in question, e.g. backend/baseclass.pm line 157 
 * Try to simulate failures in local tests, e.g. kill video encoder while a test is running 
 * Improve the error handling, e.g. extend the die message to inform about if the corresponding process is still there, etc. improved.

Back