Project

General

Profile

Actions

action #52247

closed

autoinst-log.txt opened with wrong path

Added by apachelogger almost 5 years ago. Updated almost 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Regressions/Crashes
Target version:
Start date:
2019-05-28
Due date:
% Done:

0%

Estimated time:

Description

austoinst-log.txt is currently opened as though it was in . but really it is in ./testresults/

basetest.pm contains

open(my $AUTOINSTLOG, "<", "autoinst-log.txt") || die('Could not open autoinst-log.txt for reading');

which is meant to open the log file created in bmwqemu.pm

        $logger = Mojo::Log->new(level => 'debug', path => catfile(result_dir, 'autoinst-log.txt'));

but as far as I can tell there is no chdir into result_dir anywhere, so the CWD at the time of the open() is still the same CWD as it was when the logger was initialized, and that is one directory above autoinst-log.txt. Printing getcwd just before the open call confirms as much.

Current master now always trip over the incorrect open call and die.


Related issues 1 (0 open1 closed)

Related to openQA Project - action #52235: isotovideo -d brokenResolveddheidler2019-05-28

Actions
Actions #1

Updated by coolo almost 5 years ago

  • Category set to Regressions/Crashes
  • Target version set to Current Sprint

Well, the regular case is not calling the Mojo::Log statement - the worker runs isotovideo -d and channels the output itself to ./autoinst.log.txt - so the open is correct.

So you only notice the problem if calling isotovideo yourself without -d, so the Mojo::Log statement needs to remove the catfile, easy fix

Actions #2

Updated by szarate almost 5 years ago

Actions #3

Updated by dheidler almost 5 years ago

  • Status changed from New to In Progress
  • Assignee set to dheidler
Actions #4

Updated by dheidler almost 5 years ago

  • Status changed from In Progress to Feedback

PR https://github.com/os-autoinst/os-autoinst/pull/1159 takes care of this issue as well now.

Actions #5

Updated by okurz almost 5 years ago

There was some fallout. See #52235#note-10 for details about the "fix" by partial revert. We can continue in reopened #45011

Actions #6

Updated by okurz almost 5 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF