minke.yu/sglang
1
0
Fork 0
Code Issues Pull Requests Actions 13 Packages Projects Releases Wiki Activity

Fix re-trigger actor of CI rate limit (#13136)

Browse Source
This commit is contained in:
Liangsheng Yin
2025-11-12 16:10:27 +08:00
committed by GitHub
parent 018123b5b0
commit ad8d24c39e
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/pr-test.yml
+8 -5
View File
@@ -52,10 +52,13 @@ jobs:
github-token: ${{ secrets.GITHUB_TOKEN }} github-token: ${{ secrets.GITHUB_TOKEN }}
script: | script: |
const HOURS = 2; const HOURS = 2;
const actor = context.actor;
const owner = context.repo.owner; const owner = context.repo.owner;
const repo = context.repo.repo; const repo = context.repo.repo;
const eventName = context.eventName; const eventName = context.eventName;
const curRun = await github.rest.actions.getWorkflowRun({
owner, repo, run_id: context.runId
});
const triggeringActor = curRun.data.triggering_actor?.login || context.actor;
async function hasHighPermission(username) { async function hasHighPermission(username) {
try { try {
@@ -68,8 +71,8 @@ jobs:
} }
} }
if (await hasHighPermission(actor)) { if (await hasHighPermission(triggeringActor)) {
core.info(`Triggering user '${actor}' has high permission. No rate limit applied.`); core.info(`Triggering user '${triggeringActor}' has high permission. No rate limit applied.`);
return; return;
} }
@@ -88,12 +91,12 @@ jobs:
const recentFound = runs.find((run) => { const recentFound = runs.find((run) => {
if (String(run.id) === String(context.runId)) return false; if (String(run.id) === String(context.runId)) return false;
if (new Date(run.created_at) < cutoff) return false; if (new Date(run.created_at) < cutoff) return false;
return (run.actor?.login === actor) || (run.triggering_actor?.login === actor); return (run.actor?.login === triggeringActor) || (run.triggering_actor?.login === triggeringActor);
}); });
if (recentFound) { if (recentFound) {
core.setFailed( core.setFailed(
`User '${actor}' already triggered '${context.workflow}' via '${eventName}' at ${recentFound.created_at}. ` + `User '${triggeringActor}' already triggered '${context.workflow}' via '${eventName}' at ${recentFound.created_at}. ` +
`Please wait ${HOURS} hours before triggering again.` `Please wait ${HOURS} hours before triggering again.`
); );
} else { } else {