Commit 97ae703
authored
CHORE: Test/fix models api endpoint (#529)
### Work Item / Issue Reference
<!--
IMPORTANT: Please follow the PR template guidelines below.
For mssql-python maintainers: Insert your ADO Work Item ID below
For external contributors: Insert Github Issue number below
Only one reference is required - either GitHub issue OR ADO Work Item.
-->
<!-- mssql-python maintainers: ADO Work Item -->
>
[AB#44172](https://sqlclientdrivers.visualstudio.com/c6d89619-62de-46a0-8b46-70b92a84d85e/_workitems/edit/44172)
<!-- External contributors: GitHub Issue -->
> GitHub Issue: #<ISSUE_NUMBER>
-------------------------------------------------------------------
### Summary
This pull request updates the way the GitHub Models API is called in
both the GitHub Actions workflow and the local test script. The main
focus is on switching to the new API endpoint and updating the model
identifier to the correct format.
**API Endpoint and Model Updates:**
* Changed the API endpoint from `models.inference.ai.azure.com` to
`models.github.ai/inference` in both
`.github/workflows/issue-triage.yml` and `test-triage-local.js` to use
the new GitHub Models service.
[[1]](diffhunk://#diff-4eea6c17da2d79103bcb247f5af47a37abfd43faf9032ea844ddf6c3e18dc81cR29-R37)
[[2]](diffhunk://#diff-8920b09e9baab77cdbb09013f8a8e579f466b8148f194e026a94c10d703aea2eL34-R34)
* Updated the model identifier from `"gpt-4.1"` to `"openai/gpt-4.1"` in
both files to match the expected model naming convention.
[[1]](diffhunk://#diff-4eea6c17da2d79103bcb247f5af47a37abfd43faf9032ea844ddf6c3e18dc81cL44-R46)
[[2]](diffhunk://#diff-8920b09e9baab77cdbb09013f8a8e579f466b8148f194e026a94c10d703aea2eL43-R43)
**Environment Variable Handling:**
* Added the `GITHUB_TOKEN` environment variable to the `Triage Analysis`
step in the workflow to ensure the token is available for API calls.
<!-- Insert your summary of changes below. Minimum 10 characters
required. -->
<!--
### PR Title Guide
> For feature requests
FEAT: (short-description)
> For non-feature requests like test case updates, config updates ,
dependency updates etc
CHORE: (short-description)
> For Fix requests
FIX: (short-description)
> For doc update requests
DOC: (short-description)
> For Formatting, indentation, or styling update
STYLE: (short-description)
> For Refactor, without any feature changes
REFACTOR: (short-description)
> For release related changes, without any feature changes
RELEASE: #<RELEASE_VERSION> (short-description)
### Contribution Guidelines
External contributors:
- Create a GitHub issue first:
https://github.com/microsoft/mssql-python/issues/new
- Link the GitHub issue in the "GitHub Issue" section above
- Follow the PR title format and provide a meaningful summary
mssql-python maintainers:
- Create an ADO Work Item following internal processes
- Link the ADO Work Item in the "ADO Work Item" section above
- Follow the PR title format and provide a meaningful summary
-->1 parent 070b250 commit 97ae703
2 files changed
Lines changed: 7 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
| 29 | + | |
| 30 | + | |
29 | 31 | | |
30 | 32 | | |
31 | 33 | | |
32 | 34 | | |
33 | 35 | | |
34 | 36 | | |
35 | | - | |
| 37 | + | |
36 | 38 | | |
37 | 39 | | |
38 | 40 | | |
| |||
41 | 43 | | |
42 | 44 | | |
43 | 45 | | |
44 | | - | |
| 46 | + | |
45 | 47 | | |
46 | 48 | | |
47 | 49 | | |
| |||
292 | 294 | | |
293 | 295 | | |
294 | 296 | | |
295 | | - | |
| 297 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | | - | |
| 34 | + | |
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
| 43 | + | |
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| |||
0 commit comments