Project

General

Profile

Actions

action #178114

closed

coordination #127031: [saga][epic] openQA for SUSE customers

coordination #165393: [epic] Improved code coverage in openQA

[beginner][openQA][CI] Unhandled output in t/deploy.t

Added by okurz about 1 month ago. Updated 30 days ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Regressions/Crashes
Target version:
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.

Related issues 2 (0 open2 closed)

Related to openQA Project (public) - action #163940: Warning in test: SQL::Translator::Diff::schema_diff(): producer_args is deprecated size:SResolvedokurz2024-07-15

Actions
Copied from openQA Project (public) - action #178111: [beginner][openQA][CI] Unhandled output in t/05-scheduler-restart-and-duplicate.tResolvedtinita

Actions
Actions #1

Updated by okurz about 1 month ago

  • Copied from action #178111: [beginner][openQA][CI] Unhandled output in t/05-scheduler-restart-and-duplicate.t added
Actions #2

Updated by tinita about 1 month ago

  • Status changed from New to In Progress
  • Assignee set to tinita
Actions #3

Updated by tinita about 1 month ago

  • Status changed from In Progress to Feedback
Actions #4

Updated by tinita about 1 month ago

  • Related to action #163940: Warning in test: SQL::Translator::Diff::schema_diff(): producer_args is deprecated size:S added
Actions #5

Updated by okurz about 1 month ago

You mentioned the relation to #163940 but the "deprecated" message is still there. Maybe missing a maintenance update to Leap?

Actions #6

Updated by tinita 30 days ago

  • Status changed from Feedback to Resolved

The related ticket was opened again now, so I can resolve this one

Actions

Also available in: Atom PDF