Skip to content

Python: Fix ai connector integration tests#13311

Merged
moonbox3 merged 8 commits into
mainfrom
taochen/python-fix-ai-connector-integration-tests
Oct 30, 2025
Merged

Python: Fix ai connector integration tests#13311
moonbox3 merged 8 commits into
mainfrom
taochen/python-fix-ai-connector-integration-tests

Conversation

@TaoChenOSU

@TaoChenOSU TaoChenOSU commented Oct 29, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Motivation and Context

Vertex AI and Bedrock integration tests have been failing.

Description

  1. This PR fixes the vertex ai integration tests by upgrading the Google AI SDK version.
  2. This PR fixes the Bedrock integration tests by upgrading the models.

Contribution Checklist

@TaoChenOSU TaoChenOSU self-assigned this Oct 29, 2025
@TaoChenOSU TaoChenOSU requested a review from a team as a code owner October 29, 2025 20:26
@TaoChenOSU TaoChenOSU added python Pull requests for the Python Semantic Kernel ai connector Anything related to AI connectors labels Oct 29, 2025
@github-actions github-actions Bot changed the title Fix ai connector integration tests Python: Fix ai connector integration tests Oct 29, 2025
@TaoChenOSU TaoChenOSU mentioned this pull request Oct 29, 2025
Closed
@markwallace-microsoft

markwallace-microsoft commented Oct 29, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
connectors/ai/google/vertex_ai/services
   utils.py640100% 
   vertex_ai_chat_completion.py120496%123, 153, 182, 184
   vertex_ai_text_completion.py64296%95, 117
   vertex_ai_text_embedding.py400100% 
TOTAL27298456083% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
3759 23 💤 0 ❌ 0 🔥 1m 41s ⏱️

@moonbox3 moonbox3 added this pull request to the merge queue Oct 29, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 30, 2025
@TaoChenOSU TaoChenOSU enabled auto-merge October 30, 2025 00:18
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Oct 30, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 30, 2025
@moonbox3

moonbox3 commented Oct 30, 2025

Copy link
Copy Markdown
Collaborator

@copilot bump the Python's pyproject.toml weaviate extra to 4.17.0

Copilot AI mentioned this pull request Oct 30, 2025
Merged

Copilot AI commented Oct 30, 2025

Copy link
Copy Markdown
Contributor

@moonbox3 I've opened a new pull request, #13312, to work on those changes. Once the pull request is ready, I'll request review from you.

@moonbox3
Python: Fix AI connector integration tests and bump weaviate dependen…
0c8f26f 
…cy (#13312)

Vertex AI and Bedrock integration tests were failing due to outdated
dependencies and deprecated models.

**Changes:**
- Upgraded Google AI SDK version to fix Vertex AI integration tests
- Updated Bedrock integration test models to current supported versions
- Bumped `weaviate-client` minimum version from `>=4.10` to `>=4.17.0`
(removed `!=4.16.7` exclusion)

<!-- START COPILOT CODING AGENT TIPS -->
---

✨ Let Copilot coding agent [set things up for
you](https://github.com/microsoft/semantic-kernel/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot)
— coding agent works faster and does higher quality work when set up for
your repo.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: moonbox3 <35585003+moonbox3@users.noreply.github.com>
@moonbox3 moonbox3 added this pull request to the merge queue Oct 30, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 30, 2025
@moonbox3 moonbox3 enabled auto-merge October 30, 2025 04:19
@moonbox3 moonbox3 added this pull request to the merge queue Oct 30, 2025
Merged via the queue into main with commit 1f8fea2 Oct 30, 2025
27 checks passed
@moonbox3 moonbox3 deleted the taochen/python-fix-ai-connector-integration-tests branch October 30, 2025 04:29
@github-project-automation github-project-automation Bot moved this to Sprint: Done in Semantic Kernel Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eavanvalkenburg eavanvalkenburg eavanvalkenburg approved these changes
@dmytrostruk dmytrostruk dmytrostruk approved these changes
@alliscode alliscode alliscode approved these changes
@moonbox3 moonbox3 moonbox3 approved these changes
+1 more reviewer
@ekzhu ekzhu ekzhu approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@TaoChenOSU TaoChenOSU

Labels

ai connector Anything related to AI connectors python Pull requests for the Python Semantic Kernel

Projects

Semantic Kernel
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@TaoChenOSU @markwallace-microsoft @moonbox3 @ekzhu @eavanvalkenburg @dmytrostruk @alliscode