Skip to content
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

PR to trigger enterprise XRT pipeline using github actions #7794

Merged
merged 6 commits into from
Nov 15, 2023
Merged

PR to trigger enterprise XRT pipeline using github actions #7794

merged 6 commits into from
Nov 15, 2023

Conversation

manikandan-xilinx
Copy link
Collaborator

This PR will trigger a XRT GitHub actions pipeline in GitHub enterprise for every PR created here in XRT github.com.
We are doing this to test the stability of using GitHub actions pipeline for XRT

@manikandan-xilinx
Copy link
Collaborator Author

https://github.com/Xilinx/XRT/actions/runs/6818933395/job/18545437738?pr=7794 is failing because we have added secrets and it seems like secrets cannot be accessed during PR eg: (pypa/gh-action-pypi-publish#49). So I believe after merge it should be fine

@manikandan-xilinx
Copy link
Collaborator Author

manikandan-xilinx commented Nov 10, 2023

@stsoe @maxzhen Could you pls help review this.

@gbuildx
Copy link
Collaborator

gbuildx commented Nov 10, 2023

Build Passed!

Signed-off-by: msubrama <msubrama@xilinx.com>
@gbuildx
Copy link
Collaborator

gbuildx commented Nov 10, 2023

Build failed :(

@manikandan-xilinx
Copy link
Collaborator Author

retest this please.

@gbuildx
Copy link
Collaborator

gbuildx commented Nov 13, 2023

Build Passed!

-H "Accept: application/vnd.github+json" \
-H "Authorization: Bearer ${{ secrets.CI_DOMAIN_TOKEN }}" \
"https://${{ secrets.CI_DOMAIN_URL }}/api/v3/repos/${{ secrets.CI_DOMAIN_OWNER }}/${{ secrets.CI_DOMAIN_REPO }}/actions/workflows/${{ secrets.CI_DOMAIN_WORKFLOW }}/dispatches" \
-d '{"ref": "main", inputs: { branch_name: ${{ github.event.pull_request.ref }} }}'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing new line

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed it! thanks

@gbuildx
Copy link
Collaborator

gbuildx commented Nov 14, 2023

Build failed :(

@gbuildx
Copy link
Collaborator

gbuildx commented Nov 15, 2023

Build Passed!

@stsoe stsoe merged commit bc0b302 into Xilinx:master Nov 15, 2023
stsoe pushed a commit that referenced this pull request Nov 17, 2023
* creating a repo dispatch

* using Ubuntu as label

* Signing off for DCO

Signed-off-by: msubrama <msubrama@xilinx.com>

* Adding a new line

* Fix for call repo mirror worflow

Github action workflows are not able to access secrets for PR raised from forked repos. This PR fixes that issue for this workflow

#7794

---------

Signed-off-by: msubrama <msubrama@xilinx.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.

3 participants