Project

General

Profile

Actions

action #162035

open

coordination #58184: [saga][epic][use case] full version control awareness within openQA

coordination #88561: [epic] Extend needle version control handling

Support development of "Use needles from correct ref of NEEDLES_DIR" https://github.com/os-autoinst/openQA/pull/5175 size:M

Added by okurz 4 months ago. Updated 3 days ago.

Status:
Workable
Priority:
High
Assignee:
Category:
Feature requests
Target version:
Start date:
2024-06-10
Due date:
% Done:

0%

Estimated time:

Description

Motivation

It can be confusing to not see candidate needles on the web UI that were not available at the time the test was being executed.

Acceptance criteria

  • AC1: The candidates view on the test details pages shows candidate needles of the version that was used when the test was executed.
  • AC2: Temporary needles are cleaned up (instantly or on a schedule)
  • AC3: The needle editor still shows the last candidates. One might optionally show there the candidates at the time the test ran as well.

Suggestions

  • Read exactly what the PR "Use needles from correct ref of NEEDLES_DIR" is about
  • Checkout #154783-10 and further comments for what's missing
  • Review the code and make suggestions or add changes yourself to get it merged
  • Make sure test coverage is appropriate (PR does not look like it atm.)
  • If any work becomes too much work then split out into separate explicit tickets and provide explicit feedback to the PR author about what is expected from them and what can be expected from us
  • Saying no to a feature is always an option too
  • Consider a NEEDLES_DIR being a different repo/remote entirely (e.g. a pull request) (although this is not strictly required it would be good to ensure we could also support it)
  • Read #56789 again

Related issues 2 (2 open0 closed)

Related to openQA Project - action #56789: New needles from git repository not working with openqa-clone-custom-git-refspecBlockedokurz2019-09-11

Actions
Related to openQA Project - action #157159: Show version of candidate needles the test ran on in the "diff view" on the test details pageNew

Actions
Actions #1

Updated by okurz 4 months ago

  • Tracker changed from coordination to action
  • Target version changed from Tools - Next to Ready
Actions #2

Updated by okurz 4 months ago

  • Subject changed from Support development of https://github.com/os-autoinst/openQA/pull/5175 to Support development of "Use needles from correct ref of NEEDLES_DIR" https://github.com/os-autoinst/openQA/pull/5175 size:M
  • Description updated (diff)
  • Status changed from New to Workable
Actions #3

Updated by livdywan 3 months ago

I would guess there were more urgent tickets preventing someone from picking this up within our usual 30 days. If you took a look and had open questions, might be worth discussing again, though.

Actions #4

Updated by okurz about 2 months ago

  • Priority changed from Normal to High
Actions #5

Updated by tinita about 2 months ago

  • Assignee set to tinita
Actions #6

Updated by tinita about 2 months ago

  • Status changed from Workable to In Progress
Actions #7

Updated by openqa_review about 2 months ago

  • Due date set to 2024-09-03

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

Actions #8

Updated by tinita about 1 month ago

  • Status changed from In Progress to Feedback

I tried out the pull request locally and added some notes: https://github.com/os-autoinst/openQA/pull/5175
Still need to understand all of it...

Actions #9

Updated by livdywan about 1 month ago

  • Related to action #56789: New needles from git repository not working with openqa-clone-custom-git-refspec added
Actions #10

Updated by livdywan about 1 month ago

  • Description updated (diff)
Actions #11

Updated by tinita about 1 month ago

  • Related to action #157159: Show version of candidate needles the test ran on in the "diff view" on the test details page added
Actions #12

Updated by tinita about 1 month ago

  • Status changed from Feedback to Workable

It looks like I can't work on it in the very near future, so putting it back to Workable for now

Actions #13

Updated by tinita about 1 month ago

  • Assignee deleted (tinita)
Actions #14

Updated by livdywan about 1 month ago

  • Due date deleted (2024-09-03)
Actions #15

Updated by livdywan about 1 month ago

  • Status changed from Workable to In Progress
  • Assignee set to livdywan

Let's see if I can drive this. To start out I'm going to check if the OP had a chance to review the acceptance criteria we came up with.

Actions #16

Updated by livdywan about 1 month ago

  • Status changed from In Progress to Blocked

Waiting for a response for now

Actions #17

Updated by livdywan 24 days ago

livdywan wrote in #note-16:

Waiting for a response for now

No response so far. Might just need to wait for Scott to make time for it.

Actions #18

Updated by livdywan 17 days ago

Waiting for a response for now

No response so far. Might just need to wait for Scott to make time for it.

Asked once more.

Actions #19

Updated by livdywan 10 days ago

  • Status changed from Blocked to Workable

Scott confirmed the AC's we came up with but seems to have no time to work on this for now. The PR has some conflicts. Maybe this can be considered unblocked.

Actions #20

Updated by ybonatakis 3 days ago

Let's wait for Liv to return from vacation next week!?!

Actions

Also available in: Atom PDF