Skip to content

ci: fix slack report job — use AWS runner and authenticated GitHub API calls#46448

Merged
ydshieh merged 2 commits into
mainfrom
use_runner
Jun 5, 2026
Merged

ci: fix slack report job — use AWS runner and authenticated GitHub API calls#46448
ydshieh merged 2 commits into
mainfrom
use_runner

Conversation

@ydshieh
Copy link
Copy Markdown
Collaborator

@ydshieh ydshieh commented Jun 5, 2026

Summary

  • Switch slack-report job from ubuntu-22.04 to aws-general-8-plus runner to avoid GitHub Actions IPs being blocked by the WAF IP reputation list when uploading to HuggingFace Hub
  • Add actions/setup-python step since AWS runners don't have Python pre-installed
  • Pass GITHUB_TOKEN to the script and use it in GitHub API requests (/commits, /pulls) to avoid unauthenticated rate limiting (60 req/hr → 5000 req/hr)
  • Use .get() for API response field access so a rate-limit error response doesn't crash the CI run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2026

CI Dashboard: View test results in Grafana

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@ydshieh ydshieh merged commit b1ac534 into main Jun 5, 2026
41 checks passed
@ydshieh ydshieh deleted the use_runner branch June 5, 2026 14:55
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.

2 participants