Skip to content

Conversation

@sgaud-quic
Copy link
Contributor

As part of QLI 2.0 Kernel CI/CD below should be flow :

  1. Dev raises PR
  2. Kernel checker is triggered
    3.If kernel checker is passed, and maintainer approves PR then, Kernel CI/CD i.e sync, build test should trigger
    4.If kernel checker fails, developer should fix the issue and repush the PR which will trigger kernel checker again

To achieve this, post a check run event for kernel checker in pass case for QLI 2.0 branch.
Based on check run event and approvals Kernel CI/CD will be triggered from Coral

Also introduce a workflow which will help override the kernel-checker in case of False positive

As part of QLI 2.0 Kernel CI/CD below should be flow :
1. Dev raises PR
2. Kernel checker is triggered
3. If kernel checker is passed, and maintainer approves PR then,
   Kernel CI/CD i.e sync, build test should trigger
4. If kernel checker fails, developer should fix the issue and
   repush the PR which will trigger kernel checker again

To achieve this, post a check run event for kernel checker in pass case
for QLI 2.0 branch.
Based on check run event and approvals Kernel CI/CD will be triggered from
Coral

Signed-off-by: Salendarsingh Gaud <sgaud@qti.qualcomm.com>
In case the failure from Kernel checker can't be fixed,
maintainer should have some way to override this, so that
CI/CD can be triggered.

Introdue a workflow, which can be triggered manually by
maintainer with PR number, repo name and reason for override.
This will generate a check_run event irrespective of the checker
results.

Maintainer can trigger override workflow either by UI by going to link :
https://github.com/qualcomm-linux/kernel-config/actions

Or by CLI with below :

curl -X POST \
  -H "Accept: application/vnd.github+json" \
  -H "Authorization: Bearer <PAT>" \
  -H "X-GitHub-Api-Version: 2022-11-28" \
  https://api.github.com/repos/qualcomm-linux/kernel-config/actions/workflows/checker_override.yml/dispatches \
  -d '{
        "ref": "main",
        "inputs": {
          "pr_number": "<PR number>",
          "repo": "<repository > e.g qualcomm-linux/kernel",
		  "reason": "<Reason for override>"
        }
      }'

Signed-off-by: Salendarsingh Gaud <sgaud@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant