Skip to content

gitea github-runner test #6765

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ChristopherHX
Copy link

@ChristopherHX ChristopherHX commented May 7, 2025

This verifies github_runner scaler compatbility with https://github.com/go-gitea/gitea (review pending for v1.25.0)

Cloned the github_runner e2e test to run against a local Gitea instance of go-gitea/gitea#33964

  • Removed the cancel run
    • this e2e deploys a Gitea Server for testing into another namespace by default
  • Replaced the workflow id by file names, Gitea currently do not support numeric ids of these endpoints
  • Verifies GitHub Api Compatibility of Gitea
  • github_runner can be tested without github.com
  • added vendor/k8s.io/client-go/tools/portforward to allow the test controller to make Gitea Api calls
    • within kubernetes this is not used

Checklist

  • Tests have been added
  • A PR is opened to update the documentation on (repo) (if applicable)
    • Once Gitea side is merged and this test still passes
  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Use official Gitea images for test

Further enhancements

  • add a gitea_runner scaler
  • allow gitea-global and gitea-user scope in github_runner
  • implement paging when rate limiting is not enforced like on Gitea
  • make Gitea Api calls within kubernetes instead of port forwarding

Relates to #6488, go-gitea/gitea#33964

Signed-off-by: Christopher Homberger <[email protected]>
@ChristopherHX ChristopherHX force-pushed the gitea-github-runner-keda-test branch from 39fff4e to 5317bf5 Compare May 7, 2025 11:05
Signed-off-by: Christopher Homberger <[email protected]>
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.

1 participant