FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci(ai): Route AI integration tests through OpenRouter by sentry-junior[bot] · Pull Request #6979 · getsentry/sentry-python · GitHub

ci(ai): Route AI integration tests through OpenRouter - #6979

Merged
HazAT merged 3 commits into
masterfrom
ci/ai-openrouter-api-key
Jul 31, 2026
Merged

ci(ai): Route AI integration tests through OpenRouter#6979
HazAT merged 3 commits into
masterfrom
ci/ai-openrouter-api-key

Conversation

sentry-junior Bot commented Jul 31, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Wire the AI integration harness caller to OpenRouter so Anthropic-compatible tests no longer need a direct Anthropic key.

Action input

.github/workflows/ai-integration-test.yml now passes openrouter-api-key: ${{ secrets.OPENROUTER_API_KEY }} instead of anthropic-api-key. The harness remains pinned to the merged OpenRouter migration revision (318143d106d4a847dfac3804bb2663d1b852cf28) for pipeline stability and supply-chain safety.

Companion: getsentry/testing-ai-sdk-integrations#154

Secrets

This workflow uses the AI Integrations Tests environment. Add OPENROUTER_API_KEY there (environment secret), not only as a plain repo secret, then ANTHROPIC_API_KEY can be removed after this ships.

Pass openrouter-api-key instead of anthropic-api-key and pin the
testing-ai-sdk-integrations action to the OpenRouter migration commit.

Companion: getsentry/testing-ai-sdk-integrations#154
Co-Authored-By: Daniel Griesser <dgriesser@sentry.io>

github-actions Bot commented Jul 31, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Codecov Results 📊

94982 passed | ⏭️ 6343 skipped | Total: 101325 | Pass Rate: 93.74% | Execution Time: 347m 38s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2513 uncovered lines.
❌ Project coverage is 89.88%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.91%    89.88%    -0.03%
==========================================
  Files          193       193         —
  Lines        24832     24832         —
  Branches      8912      8912         —
==========================================
+ Hits         22326     22319        -7
- Misses        2506      2513        +7
- Partials      1417      1417         —

Generated by Codecov Action

Drop the commit pin for the internal AI integration harness action.

Co-Authored-By: Daniel Griesser <dgriesser@sentry.io>
HazAT marked this pull request as ready for review July 31, 2026 18:17
HazAT requested a review from a team as a code owner July 31, 2026 18:17

ericapisani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

The changeover to the openrouter api key looks fine but I think we should undo the change that was introduced to what version of testing-ai-sdk-integrations we test against in the pipeline

Co-Authored-By: Daniel Griesser <dgriesser@sentry.io>
HazAT merged commit 10b4f5b into master Jul 31, 2026
145 checks passed
HazAT deleted the ci/ai-openrouter-api-key branch July 31, 2026 19:08
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
Wire the AI integration harness caller to OpenRouter so
Anthropic-compatible tests no longer need a direct Anthropic key.

**Action input**

`.github/workflows/ai-integration-test.yml` now passes
`openrouter-api-key: ${{ secrets.OPENROUTER_API_KEY }}` instead of
`anthropic-api-key`. The harness remains pinned to the merged OpenRouter
migration revision (`318143d106d4a847dfac3804bb2663d1b852cf28`) for
pipeline stability and supply-chain safety.

Companion:
getsentry/testing-ai-sdk-integrations#154

**Secrets**

This workflow uses the `AI Integrations Tests` environment. Add
`OPENROUTER_API_KEY` there (environment secret), not only as a plain
repo secret, then `ANTHROPIC_API_KEY` can be removed after this ships.

---------

Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: Daniel Griesser <dgriesser@sentry.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL