Skip to content

Conversation

@akhilthomas011
Copy link

Purpose

  • This PR fixes the below bugs in the lab ai-agent-service:
  1. Added a property named foundryProjectId in extendedAIServicesConfig array
  2. Changed Bicep outputs for connection IDs to use foundryProjectId instead of full resource path.
  3. Updated notebook to reference bicep output for Bing Search Connection ID instead of the existing SDK reference to align with other use cases.
  4. Fixed openapi_tools variable typo.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone https://github.com/akhilthomas011/AI-Gateway
cd [AI-Gateway]
git checkout [fix-conn-ids]
npm install
  • Test the code
Run the notebook in the location AI-Gateway\labs\ai-agent-service\main-v2.bicep

What to Check

Verify that the following are valid:

  • All uses cases are executed without the error: project_connection_error; Connection id /subscriptions/361e9bb5-a5de-4ac8-8d6f-4f0349859032/resourceGroups/lab-v2-ai-agent-service/providers/Microsoft.CognitiveServices/accounts/foundry1-jnnkiqe7dd5h2/projects/ai-agent-service/connections/WeatherAPI was not found in the list of the provided connections.

Other Information

Changed Bicep outputs for connection IDs to use foundryProjectId instead of full resource path. Updated notebook to reference bing_search_connection_id and fixed tool variable name. Added foundryProjectId output in foundry.bicep for consistency.
Changed Bicep outputs for connection IDs to use foundryProjectId instead of full resource path. Updated notebook to reference bing_search_connection_id and fixed tool variable name. Added foundryProjectId output in foundry.bicep for consistency.
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