Project

General

Profile

action #124961

openQA restarts user_cancelled jobs with RETRY=N (N>0) size:M

Added by ph03nix 3 months ago. Updated 3 months ago.

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

0%

Estimated time:
Difficulty:

Description

Observation

When cancelling a job with RETRY=1 (or any number larger than 0), openQA automatically restarts the job. This is expected for any failed state, but not for cancellation

Acceptance criteria

  • AC1*: Jobs cancelled by users are not restarted automatically (even when RETRY is used). What happens to other cancelled jobs may still remain undefined/suboptimal.

Out of scope

  • Fixing the mess you can see in #110458

Steps to reproduce

  • Create job with RETRY=1
  • Cancel the job

Impact / problem

When cancelling jobs with RETRY, we need to cancel N jobs.

Notes

History

#1 Updated by cdywan 3 months ago

  • Category set to Concrete Bugs
  • Target version set to Ready

Thank you for reporting this, Felix!

I could be wrong, but I would take this to be a regression. And even if it's not, it would be an issue whenever RETRY is being used. And since we encourage RETRY to be used in many cases, I think this is something we should really look into. And maybe not even that difficult to fix.

#2 Updated by mkittler 3 months ago

  • Subject changed from openQA restarts user_cancelled jobs with RETRY=N (N>0) to openQA restarts user_cancelled jobs with RETRY=N (N>0) size:M
  • Description updated (diff)
  • Status changed from New to Workable

#3 Updated by jbaier_cz 3 months ago

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

#4 Updated by jbaier_cz 3 months ago

https://github.com/os-autoinst/openQA/pull/5021 should make USER_CANCELLED jobs ok regarding the RETRY

#5 Updated by jbaier_cz 3 months ago

  • Status changed from In Progress to Feedback

#6 Updated by jbaier_cz 3 months ago

  • Status changed from Feedback to Resolved

With https://github.com/os-autoinst/openQA/pull/5024 also merged, we can consider this resolved.

Also available in: Atom PDF