-
Notifications
You must be signed in to change notification settings - Fork 94
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
✨ Trigger clusteradm e2e after post images. #209
✨ Trigger clusteradm e2e after post images. #209
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #209 +/- ##
=======================================
Coverage 59.69% 59.69%
=======================================
Files 128 128
Lines 13247 13247
=======================================
Hits 7908 7908
Misses 4598 4598
Partials 741 741
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
with: | ||
token: ${{ secrets.OCM_BOT_PAT }} | ||
repository: open-cluster-management-io/clusteradm | ||
event-type: ocm_changes |
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.
can we also send the commit sha, so clusteradm will know which commit triggers?
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.
Sure, the sha and ref added in the next line.
Signed-off-by: xuezhaojun <zxue@redhat.com>
c1a2769
to
7bdfbde
Compare
steps: | ||
- uses: peter-evans/repository-dispatch@v2 | ||
with: | ||
token: ${{ secrets.OCM_BOT_PAT }} |
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.
Hi, @mikeshng does OCM_BOT_PAT
have the write permission on Contents
? The permission is needed to send a repository-dispatch event.
/assign @qiujian16 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: qiujian16, xuezhaojun 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 |
Related PR
open-cluster-management-io/clusteradm#352