sclorg / testing-farm-as-github-action

GitHub Action to execute tests by Testing Farm and update Pull Request status
MIT License
13 stars 11 forks source link

feat: allow to provide context for non-PR events #220

Closed jamacku closed 1 month ago

jamacku commented 1 month ago

Adding new inputs:

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.27%. Comparing base (5405b03) to head (a9988b4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #220 +/- ## ========================================== + Coverage 90.27% 93.27% +3.00% ========================================== Files 10 11 +1 Lines 545 595 +50 Branches 95 112 +17 ========================================== + Hits 492 555 +63 + Misses 52 40 -12 + Partials 1 0 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jamacku commented 1 month ago

@yih-redhat, Could you please test this PR?

uses: jamacku/testing-farm-as-github-action@support-nonpr-context
with:
  #  ...

You can use new inputs: repo_owner, repo_name, pr_number and commit_sha to provide additional information when running on repository_dispatch. The commit_sha is optional when running on repository_dispatch trigger.

yih-redhat commented 1 month ago

Sure, I will test it and let you know the result.

yih-redhat commented 1 month ago

@jamacku I have tested this pr, it works as expected. PR https://github.com/osbuild/osbuild-composer/pull/4240 triggered ci test https://github.com/osbuild/rhel-edge-ci/actions/runs/10268536294, when test started, I can see a new test status created in that PR, and after test finished, the test result is also updated in that PR, perfect!

jamacku commented 1 month ago

Excellent, I didn't expect it to work on the first try. Thanks for testing :pray: @yih-redhat

@phracek Could you please take a look at the code? Thanks

yih-redhat commented 1 month ago

Yes, it works well at my first try with a small test case. Now I have started a full test with this pr with 10 test cases, and will let you know if anything wrong. You can also check the result at https://github.com/osbuild/osbuild-composer/pull/4240 (It's building now, testing-farm test will not be triggered until build finished), tfaga is configured at https://github.com/osbuild/rhel-edge-ci/blob/main/.github/workflows/rhel-95.yml

jamacku commented 1 month ago

test

phracek commented 1 month ago

[test]

jamacku commented 1 month ago

[test]

jamacku commented 1 month ago

[test]