QA » openQA Project

status quo¶

• The "restart" route is used when one restarts a job via the web UI and therefore also the route I suggest to use when using the REST-API directly.
• The "duplicate" route also exists but is at this point ironically almost a duplicate of the "restart" route.
• The "duplicate" route is still used (see audit log on o3 and OSD, filter for event:job_duplicate).
  • The worker uses the "duplicate" route when a job has been aborted because the worker service is stopped.

acceptance criteria¶

• AC0: Make the "duplicate" route an alias for the "restart" route using the same code internally.
• AC1: Backwards compatibility of any route must not be broken. (Except for the event currently emitted by the "duplicate" route. It is not used and may be dropped/replaced. The parameter dup_type_auto can likely be ignored as well because it doesn't seem so have any effect anymore.)

historical remarks¶

According to the commits 1c7de4b2ce and 6b48bd8a72 "restart" was there before. According to git show 6b48bd8a72:lib/OpenQA/modules/Scheduler.pm "restart" meant at the time aborting the job and setting it back to scheduled. Only "duplicate" would create a new job. That explains why "duplicate" was added in the first place.

notes¶

• The specific functions I'm talking about are OpenQA::WebAPI::Controller::API::V1::Job::restart and OpenQA::WebAPI::Controller::API::V1::Job::duplicate.