Project

General

Profile

action #108323

Subroutine consoles::sshVirtsh::has redefined at .../Class/Accessor.pm

Added by tinita 3 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Concrete Bugs
Target version:
Start date:
2022-03-14
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

When looking into #108266 I saw this warning in the logfile
https://openqa.suse.de/tests/8294410/logfile?filename=autoinst-log.txt

Subroutine consoles::sshVirtsh::has redefined at /usr/lib/perl5/vendor_perl/5.26.1/Class/Accessor.pm line 35.
    Class::Accessor::import("Class::Accessor", "antlers") called at /usr/lib/os-autoinst/consoles/sshVirtsh.pm line 13
    consoles::sshVirtsh::BEGIN() called at /usr/lib/os-autoinst/consoles/sshVirtsh.pm line 13
...

I think this was introduced in 43543e0a7ab3acaf7310e4cf6efcb6675469d37e by adding

use Mojo::Base 'consoles::sshXtermVt', -signatures;

Now we have Mojo::Base and Class::Accessor defining accessors.

Also in consoles/VNC.pm we have

use Mojo::Base 'Class::Accessor::Fast', -signatures;

Related issues

Related to openQA Project - action #99663: Use more perl signatures - os-autoinst size:MWorkable2021-10-01

Copied from openQA Infrastructure - action #108266: grenache: script_run() commands randomly time out since server room moveNew2022-03-14

History

#1 Updated by tinita 3 months ago

  • Copied from action #108266: grenache: script_run() commands randomly time out since server room move added

#2 Updated by okurz 3 months ago

  • Related to action #99663: Use more perl signatures - os-autoinst size:M added

#3 Updated by mkittler 3 months ago

  • Assignee set to mkittler

#4 Updated by mkittler 3 months ago

  • Status changed from New to Feedback

#5 Updated by mkittler 3 months ago

  • Status changed from Feedback to Resolved

Works in production, e.g. the warning is gone in https://openqa.suse.de/tests/8334727/logfile?filename=autoinst-log.txt (but was present in previous jobs of that scenario, e.g. https://openqa.suse.de/tests/8284935/logfile?filename=autoinst-log.txt).

Also available in: Atom PDF