action #162035
Updated by livdywan 7 months ago
## 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. PR author receives timely feedback/support * **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. https://github.com/os-autoinst/openQA/pull/5175 is merged ## Suggestions * Read exactly what the PR "Use needles from correct ref of NEEDLES_DIR" is about * See https://github.com/os-autoinst/openQA/pull/5175 * 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