action #129730
closedcoordination #58184: [saga][epic][use case] full version control awareness within openQA
coordination #48641: [epic] Trigger openQA tests in pull requests of any product github pull request
Adapt http://open.qa/docs/#_running_openqa_jobs_as_ci_checks for the use of github pull_request_target size:M
Description
Motivation¶
We found a nice way to trigger openQA tests within GitHub workflows but so far https://github.com/os-autoinst/os-autoinst-distri-example/blob/7a7293d6ba34d91d1428ba7261a13b83e31faad0/.github/workflows/openqa.yml#L5 uses "pull_request" meaning that a fork would need to define their own openQA credentials. Instead we should use "pull_request_target" based on the example of https://github.com/os-autoinst/opensuse-jobgroups/blob/master/.github/workflows/ci.yml
Acceptance criteria¶
- AC1: PRs like https://github.com/os-autoinst/os-autoinst-distri-example/pull/19 can successfully execute openQA based CI checks based on the github workflows triggered openQA tests
Suggestions¶
- Consider the work of #128360
- Use the example of https://github.com/os-autoinst/opensuse-jobgroups/blob/master/.github/workflows/ci.yml and adapt both http://open.qa/docs/#_running_openqa_jobs_as_ci_checks as well as our https://github.com/os-autoinst/os-autoinst-distri-example examples as applicable
Updated by okurz over 1 year ago
- Copied from action #128360: Supporting fork based development model size:M added
Updated by mkittler over 1 year ago
- Status changed from New to In Progress
- Assignee set to mkittler
Updated by mkittler over 1 year ago
- Status changed from In Progress to Feedback
Updated by okurz over 1 year ago
- Subject changed from Adapt http://open.qa/docs/#_running_openqa_jobs_as_ci_checks for the use of github pull_request_target to Adapt http://open.qa/docs/#_running_openqa_jobs_as_ci_checks for the use of github pull_request_target size:M
Updated by mkittler over 1 year ago
- Status changed from Feedback to Resolved
It has already been enabled via https://github.com/os-autoinst/os-autoinst-distri-example/pull/24 before so I've just closed https://github.com/os-autoinst/os-autoinst-distri-example/pull/25.
https://github.com/os-autoinst/openQA/pull/5198 has been merged as well. So this can be considered resolved.