-
Notifications
You must be signed in to change notification settings - Fork 22
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
Cherry-pick self hosted to master #45
Cherry-pick self hosted to master #45
Conversation
Changes made to configure the self-hosted-runner modified: .github/workflows/configurable-benchmark-test.yaml
Signed-off-by: hershd23 <[email protected]>
The commit automates below steps: 1. Create a CNCF CIL machine and register it as a self-hosted runner 2. Run SMP benchmarks on the self-hosted runner 3. Stop and remove the CNCF CIL machine and self-hosted runner Signed-off-by: Huang Xin <[email protected]>
Signed-off-by: Huang Xin <[email protected]>
Signed-off-by: Huang Xin <[email protected]>
@leecalcote Can you merge this? And then two github secrets should be added to let it work.
I haven't the permission to add above two secrets in this repo, so can you add them or give me a accessible role? |
@leecalcote @hershd23 Can you review this and give some advice about above permission issue? Thanks. |
Oh, I missed this PR earlier. Yes, we do have a service/robot account ( |
This folder in the community drive has a GitHub Process doc, which will be helpful reference and to which the new CIF token can be noted - https://drive.google.com/drive/folders/1O7TPo1FYkEYcUhQPx6hAzxPho7pWgtIp |
@leecalcote Thanks for giving me the privileges and I can add "CNCF_CIL_TOKEN" in this repo. I raised a PR #46 to change PAT name used in scripts to "GH_ACCESS_TOKEN", can you merge it too? |
Awesome. You bet. |
Description
This PR fixes #44
Notes for Reviewers
Signed commits