Skip to content

Python: Fix ai connector integration tests - #13311

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

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

Conversation

@TaoChenOSU

@TaoChenOSU Tao Chen (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 Tao Chen (TaoChenOSU) self-assigned this Oct 29, 2025
@TaoChenOSU
Tao Chen (TaoChenOSU) requested a review from a team as a code owner October 29, 2025 20:26
@TaoChenOSU Tao Chen (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 Tao Chen (TaoChenOSU) mentioned this pull request Oct 29, 2025
Closed
@markwallace-microsoft

Mark Wallace (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
Evan Mattson (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
Tao Chen (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

Evan Mattson (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

Evan Mattson (@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
Evan Mattson (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
Evan Mattson (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
Evan Mattson (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 Eduard van Valkenburg (eavanvalkenburg) eavanvalkenburg approved these changes
@dmytrostruk Dmytro Struk (dmytrostruk) dmytrostruk approved these changes
@alliscode Ben Thomas (alliscode) alliscode approved these changes
@moonbox3 Evan Mattson (moonbox3) moonbox3 approved these changes
+1 more reviewer
@ekzhu Eric Zhu (ekzhu) ekzhu approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@TaoChenOSU Tao Chen (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