-
Notifications
You must be signed in to change notification settings - Fork 91
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip running acceptance tests in GH workflows for PR that does not change SBO nor the testing framework #1150
Skip running acceptance tests in GH workflows for PR that does not change SBO nor the testing framework #1150
Conversation
/hold |
61c51f9
to
db30a9b
Compare
/retest |
1 similar comment
/retest |
49275e8
to
5645687
Compare
e93ea2f
to
fe421ef
Compare
/unhold |
… testing framework Signed-off-by: Pavel Macík <pavel.macik@gmail.com>
Some questions:
|
Yes, this PR only affects GH Actions, Prow jobs (OpenShift CI) are separated in openshift/release#28875 So, until the above PR is merged, the Prow jobs would run as before. |
Codecov Report
@@ Coverage Diff @@
## master #1150 +/- ##
=======================================
Coverage 57.84% 57.84%
=======================================
Files 31 31
Lines 2697 2697
=======================================
Hits 1560 1560
Misses 976 976
Partials 161 161 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sadlerap The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Pavel Macík pavel.macik@gmail.com
Changes
Currently, the acceptance tests are executed for any and each PR - even for those that for example affects only the docs. In such cases there is no need to actually execute the acceptnace tests as the resulting SBO images as well as the testing framework (incl. scenarios) are exactly the same as those already tested and merged in master.
Not running the acceptance tests for PRs that do not inflict any testable changes would save time significantly to merge such PRs as well as saving resources
Ref: https://issues.redhat.com/browse/APPSVC-1116
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
included if any changes are user facing
included if any functionality added or changed. For bugfixes please include tests that can catch regressions
@external-feedback
tag.