Project

General

Profile

action #16362

tests spill out a lot of warnings and fail if PRJDIR is not set

Added by okurz over 5 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Concrete Bugs
Target version:
Start date:
2017-01-31
Due date:
% Done:

100%

Estimated time:
Difficulty:

Description

observation

Use of uninitialized value in index at /usr/lib/os-autoinst/needle.pm line 45.
Use of uninitialized value $bmwqemu::vars{"PRJDIR"} in addition (+) at /usr/lib/os-autoinst/needle.pm line 46.
Use of uninitialized value in index at /usr/lib/os-autoinst/needle.pm line 45.
Use of uninitialized value $bmwqemu::vars{"PRJDIR"} in addition (+) at /usr/lib/os-autoinst/needle.pm line 46.
Use of uninitialized value in index at /usr/lib/os-autoinst/needle.pm line 45.
Use of uninitialized value $bmwqemu::vars{"PRJDIR"} in addition (+) at /usr/lib/os-autoinst/needle.pm line 46.

steps to reproduce

expected result

  • probably "PRJDIR" should have some sensible default OR
  • fail in a more explanatory manner
  • example vars.json should be working
  • Make sure example vars.json from documentation is tested, e.g. either test the content of the README.md or do not include in the README but only reference an example file, e.g. the one from the full stack test

History

#1 Updated by oholecek over 5 years ago

Do I understand it correctly this occurs only when running example vars.json. Or do you see this on some production instance? OpenQA::Worker::Engines::isotovideo::engine_workit adds this to vars.json, but maybe I missed something.

I'm for adding this to example vars.json and add check for presence to isotovideo. Can be?

#2 Updated by okurz over 5 years ago

yes, when working with isotovideo standalone on any vars.json that does not include the PRJDIR variable. I prefer to have the necessary settings in the vars.json as few as possible so that it makes sense to run it also locally standalone on a vars.json file. PRJDIR can be defined as mandatory if that makes sense for that purpose but maybe PRJDIR, CASEDIR, PRODUCTDIR should be revisited if all of them are needed or if sensible defaults can be given.

#3 Updated by oholecek about 5 years ago

  • Status changed from New to In Progress
  • Target version set to Milestone 6
  • % Done changed from 0 to 90

#4 Updated by oholecek about 5 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 90 to 100

Also available in: Atom PDF