Actions
action #178114
opencoordination #127031: [saga][epic] openQA for SUSE customers
coordination #165393: [epic] Improved code coverage in openQA
[beginner][openQA][CI] Unhandled output in t/deploy.t
Start date:
2025-03-01
Due date:
% Done:
0%
Estimated time:
Tags:
Description
Motivation¶
https://app.circleci.com/pipelines/gh/os-autoinst/openQA/16015/workflows/93b25d9c-129a-40d0-8dec-82b2390fb56b/jobs/154230 shows a problem of unhandled output. We want to have a clean TAP output with all output from tested application code to be captured:
[00:01:33] t/deploy.t ................................................ 6/? [WARN] Job IDs will be converted to bigint. That may take multiple hours on big databases. It is safe to stop the service (and start from scratch on the next startup).
[WARN] Worker database IDs will be converted to bigint. It is safe to stop the service (and start from scratch on the next startup).
[WARN] Further database IDs will be converted to bigint. That may take multiple hours on big databases. It is safe to stop the service (and start from scratch on the next startup).
[00:01:33] t/deploy.t ................................................ 14/? SQL::Translator::Diff::schema_diff(): producer_args is deprecated -- it does not go straight to the producer, it goes to the internal sqlt object. Please use sqlt_args, which reflects how it's used at /usr/lib/perl5/vendor_perl/5.26.1/DBIx/Class/DeploymentHandler/DeployMethod/SQL/Translator.pm line 533
[00:01:33] t/deploy.t ................................................ ok 35754 ms ( 0.01 usr 0.00 sys + 33.86 cusr 1.31 csys = 35.18 CPU)
Acceptance criteria¶
- AC1: No unhandled output from t/deploy.t
Suggestions¶
- Call
make test KEEP_DB=1 TESTS=t/deploy.t
locally to reproduce - Check which part of the code triggers the output, e.g. by checking the output of
prove -v ...
- Surround calls with according output capture code as we already have in many other cases, likely even in the same test module.
Updated by okurz 2 days ago
- Copied from action #178111: [beginner][openQA][CI] Unhandled output in t/05-scheduler-restart-and-duplicate.t added
Updated by tinita about 14 hours ago
- Status changed from New to In Progress
- Assignee set to tinita
Updated by tinita about 14 hours ago
- Status changed from In Progress to Feedback
Updated by tinita about 14 hours ago
- Related to action #163940: Warning in test: SQL::Translator::Diff::schema_diff(): producer_args is deprecated added
Updated by okurz about 1 hour ago
You mentioned the relation to #163940 but the "deprecated" message is still there. Maybe missing a maintenance update to Leap?
Actions