Project

General

Profile

action #88482

coordination #58184: [saga][epic][use case] full version control awareness within openQA, e.g. user forks and branches, fully versioned test schedules and configuration settings

coordination #80372: [epic] Cleanup vars.json as initial information container between openQA worker and isotovideo

Two absolute paths concatenated to form a default needle dir when PRODUCT_DIR/needles doesn't exist

Added by ilausuch 6 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Concrete Bugs
Target version:
Start date:
2021-02-08
Due date:
% Done:

0%

Estimated time:
6.00 h
Difficulty:

Description

Observation

Sometimes a job execution fails with this error

needles_dir not found: /var/lib/openqa/share/tests/opensuse/var/lib/openqa/share/tests/opensuse/products/opensuse/needles (check vars.json?) at /usr/lib/os-autoinst/needle.pm line 330.

Is concatenating two absolute paths (https://github.com/os-autoinst/os-autoinst/blob/adbb28bc61ce4f21a55d07399eac7d48badc6b6f/needle.pm#L328) when needles directory doesn't exist.

Reproduction

Remove the directory needles from /var/lib/openqa/share/tests/opensuse/products/opensuse

Acceptance criteria

  • AC1: Needles folder has a working default or aborts if PRODUCT_DIR/needles doesn't exist

Suggestions

  • Look into logic in needle.pm
  • Improve error message to reveal relevant variables instead of hinting at vars.json i.e. NEEDLES_DIR, CASEDIR and default_needles_dir
  • Log the missing PRODUCTDIR}/needles if this affects the default nedles folder to be used here

Work-around

  • Create a folder PRODUCTDIR}/needles

Related issues

Related to openQA Project - coordination #67723: [epic] Remote openQA worker fails to run tests from openqa-clone-custom-git-refspecResolved2021-03-18

History

#1 Updated by cdywan 6 months ago

  • Subject changed from Improve the error when the needles directory doesn't exist to Two absolute paths concatenated to form a default needle dir when PRODUCT_DIR/needles doesn't exist
  • Description updated (diff)
  • Category set to Concrete Bugs
  • Status changed from New to Workable

Updated the description and title. To my mind, this path default makes no sense. The error could be more helpful, but it should be clear that this isn't just a cosmetic issue.

#2 Updated by cdywan 6 months ago

  • Description updated (diff)

#3 Updated by okurz 6 months ago

  • Target version set to future

#5 Updated by okurz 5 months ago

  • Related to coordination #67723: [epic] Remote openQA worker fails to run tests from openqa-clone-custom-git-refspec added

#6 Updated by okurz 5 months ago

  • Status changed from Workable to In Progress
  • Assignee set to Xiaojing_liu
  • Target version changed from future to Ready

oh right. Cool that you found the relation. Then let's add that as related to the existing story.

@Xiaojing_liu I added you as assignee as you are already working on this topic with https://github.com/os-autoinst/os-autoinst/pull/1627 . Feel free to unassign again in case you or we find out that your PR will not fix the original problem or not all of it.

#7 Updated by okurz 5 months ago

  • Parent task set to #80372

#8 Updated by openqa_review 5 months ago

  • Due date set to 2021-03-12

Setting due date based on mean cycle time of SUSE QE Tools

#9 Updated by cdywan 5 months ago

The PR got merged. The AC I proposed originally asks for a different solution but I'd say we can regard this as a fix. Users won't run into the error.

#10 Updated by Xiaojing_liu 5 months ago

  • Status changed from In Progress to Feedback
  • Estimated time set to 6.00 h

#11 Updated by cdywan 5 months ago

  • Description updated (diff)
  • Status changed from Feedback to Resolved

#12 Updated by Xiaojing_liu 5 months ago

Here is an example that run on OSD: https://openqa.suse.de/tests/5592282#details

#13 Updated by okurz 3 months ago

  • Due date deleted (2021-03-12)

Also available in: Atom PDF