Skip to content

feat: workflow to run tests#778

Merged
harshiv-26 merged 11 commits intomainfrom
harshivg/gateway-2289-run-test-for-updation-pr-on-models-repo
Apr 22, 2026
Merged

feat: workflow to run tests#778
harshiv-26 merged 11 commits intomainfrom
harshivg/gateway-2289-run-test-for-updation-pr-on-models-repo

Conversation

@harshiv-26
Copy link
Copy Markdown
Collaborator

@harshiv-26 harshiv-26 commented Apr 21, 2026

Note

Medium Risk
Adds a comment-triggered CI path that can start external TrueFoundry jobs and uses repo secrets; while author checks and input sanitization reduce risk, misconfiguration could still lead to unintended job triggers or cost.

Overview
Adds a new GitHub Actions workflow (test-changed-models.yml) that listens for /test-models PR comments from trusted users, checks out the PR head separately, and triggers TrueFoundry gateway-test-job-v2 runs.

Introduces trigger_changed_model_tests.py to git diff the PR’s tip commit for changed providers/**/*.yaml, group changes by provider/model with basic path validation, and invoke tfy trigger job once per provider while reporting the number of triggered runs back to the workflow. Also updates .gitignore to ignore /scripts/ at the repo root.

Reviewed by Cursor Bugbot for commit bffff4f. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread .github/scripts/trigger_changed_model_tests.py Outdated
Comment thread .github/scripts/trigger_changed_model_tests.py
Comment thread .github/scripts/trigger_changed_model_tests.py
Comment thread .github/scripts/trigger_changed_model_tests.py
Comment thread .github/workflows/test-changed-models.yml
Comment thread .github/workflows/test-changed-models.yml
Comment thread .github/scripts/trigger_changed_model_tests.py
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2514e73. Configure here.

Comment thread .github/scripts/trigger_changed_model_tests.py Outdated
@harshiv-26 harshiv-26 enabled auto-merge (squash) April 22, 2026 12:43
@harshiv-26 harshiv-26 disabled auto-merge April 22, 2026 12:44
@harshiv-26 harshiv-26 merged commit 5b627c8 into main Apr 22, 2026
3 checks passed
@harshiv-26 harshiv-26 deleted the harshivg/gateway-2289-run-test-for-updation-pr-on-models-repo branch April 22, 2026 13:03
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