action #33892
closedcoordination #10316: [epic] Better command line options extending "client"
Allow the client to trigger a build for a jobgroup
Description
User story¶
As a test developer, I want to trigger or retrigger a build from a jobgroup, to ease posting jobs into openQA
Acceptance criteria¶
- AC1: openQA::Client is able to trigger a build only for the specified jobgroup
- AC2: Builds that were triggered in the normal way, are not obsoleted or canceled
- AC3: Builds for a jobgroup that were previously scheduled can be handled in a similar fashion as other builds (Parameters, et al)
Updated by okurz over 6 years ago
I guess one can also describe it as restricted to a job group because the job group is merely a filtered part of a complete build spanning none to many job groups.
Updated by szarate over 6 years ago
- Assignee deleted (
szarate)
I haven't had too much chance to work on this.
Part #1: is to add simple re/trigger:
- Query the Job Group and find the last job ids
- Use TESTS parameter in isos post added here to trigger tests only for that jobgroup.
MM jobs need to be tested aswell.
Having a proper table for isos would be desirable, as this could help to better trace events, and simplify the needs of passing more and more parameters.
Updated by mkittler over 6 years ago
The idea would be:
- Create an API route to get relevant job ideas. It would return a JSON array like this:
[{id: 1}, {id: 2}, ...]
. This route would be similar to the test results overview and should preferably share the code. - Allow the client to restart all jobs returned by the new route.
@szarate Not sure why you always use the word 'trigger' and want to use the POST isos
route. I talked with @coolo and this seems to be simply about restarting some jobs filtered by some criteria like on the test results overview page.
Updated by szarate over 6 years ago
@mkittler: Because that's how I was seeing it, specially if you take MM jobs into account.
Triggering jobs (isos posts) and retriggering/restarting which might be the part that coolo pointed you to.
Update de AC if you see it fit :)
Updated by mkittler over 6 years ago
- Status changed from New to In Progress
Updated by coolo over 6 years ago
hmm, now reading the user story - looks like we talked about a different problem. Oops :-)
Updated by szarate over 6 years ago
@coolo Happens :P but it's not entirely bad
Updated by mkittler over 6 years ago
Yet another PR: https://github.com/os-autoinst/openQA/pull/1653
Updated by szarate over 6 years ago
- Status changed from In Progress to Resolved
- Target version changed from Current Sprint to Done
Implemented feature complies with the AC. :)