-
Notifications
You must be signed in to change notification settings - Fork 135
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
(fix): Change on trigger to pull_request_target in the "check-file-updates.yaml" action #1296
(fix): Change on trigger to pull_request_target in the "check-file-updates.yaml" action #1296
Conversation
Once this is merged, I can try to create a test pr by simulating a change in the apis. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## incubation #1296 +/- ##
=============================================
Coverage ? 18.91%
=============================================
Files ? 30
Lines ? 2670
Branches ? 0
=============================================
Hits ? 505
Misses ? 2103
Partials ? 62 ☔ View full report in Codecov by Sentry. |
2097a7a
to
0a7cbef
Compare
15537ec
to
58e68b8
Compare
You could also try running the action on your fork instead of merging potentially not working solution in upstream repo. |
I think this report is lying ;) |
58e68b8
to
4f3fe54
Compare
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action Signed-off-by: AJAY JAGANATHAN <[email protected]>
4f3fe54
to
6995f23
Compare
tested on a fork : AjayJagan#527 |
@zdtsw @VaishnaviHire ^ JFyi :) |
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.
approved based on the test result
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: zdtsw 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 |
f954f0b
into
opendatahub-io:incubation
@AjayJagan could you please keep first line of the commit message short enough and descriptive? First line is a bit special, it's a header of the patch. |
@ykaliuta , for sure :) |
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action (opendatahub-io#1296) Signed-off-by: AJAY JAGANATHAN <[email protected]>
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action (opendatahub-io#1296) Signed-off-by: AJAY JAGANATHAN <[email protected]> Signed-off-by: Wen Zhou <[email protected]>
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action (opendatahub-io#1296) Signed-off-by: AJAY JAGANATHAN <[email protected]> Signed-off-by: Wen Zhou <[email protected]>
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action (opendatahub-io#1296) Signed-off-by: AJAY JAGANATHAN <[email protected]> Signed-off-by: Wen Zhou <[email protected]>
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action (opendatahub-io#1296) Signed-off-by: AJAY JAGANATHAN <[email protected]> (cherry picked from commit f954f0b)
… config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the action (opendatahub-io#1296) Signed-off-by: AJAY JAGANATHAN <[email protected]> (cherry picked from commit f954f0b)
(fix): Change on trigger to
pull_request_target
in the "Check config and readme updates / Ensure generated files are included (pull_request)" action to fix "Resource not accessible by integration" error while running the actionDescription
The "Check config and readme updates / Ensure generated files are included (pull_request)" action fails when trying to comment on the pr.
There are 2 cases which might cause this
pull-requests: write
permission.Read and Write
permission in the repo.But the above 2 cases are satisfied for opendatahub-operator.
So the last possible case is to add change
on: pull_request
toon: pull_request _target
(as per https://github.com/thollander/actions-comment-pull-request?tab=readme-ov-file#permissions). This does not ensure that it will work but this is going to be a trial.Also, the same workflow was able to run in my repo: AjayJagan#517
Merge criteria