Project

General

Profile

Actions

action #70873

closed

Test fails because auto_review:"Encoder not accepting data":retry, video is missing

Added by mkittler over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Regressions/Crashes
Target version:
Start date:
2020-09-02
Due date:
2020-11-14
% Done:

0%

Estimated time:

Description

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:
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 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.

Related issues 3 (1 open2 closed)

Related to openQA Project - action #67342: Support using a "generic" video encoder like ffmpeg instead of only relying on the one provided by os-autoinst itselfResolvedmkittler2020-05-27

Actions
Related to openQA Infrastructure - coordination #68923: [epic] Use external videoencoder in production auto_review:"External encoder not accepting data"Resolvedokurz2020-11-13

Actions
Copied to openQA Project - action #81002: Test incompletes with auto_review:"(?s)openqa.suse.de.*Encoder not accepting data.*Uploading video":retry but video is thereNew

Actions
Actions

Also available in: Atom PDF