Alan-Jowett / CoPilot-For-Consensus / 21656314354
78%

DEFAULT BRANCH: main
Ran
Jobs 27
Files 160
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 77.85%. Remained the same
21656314354

push

github

web-flow 
fix(azure): remove partition_key from replace_item() call (#1122)

* fix(azure): remove partition_key from replace_item() call

azure-cosmos 4.9.0's replace_item() does NOT have a partition_key parameter,
unlike read_item() and delete_item(). Passing partition_key causes it to
leak through **kwargs to Session.request(), raising:
'TypeError: Session.request() got an unexpected keyword argument partition_key'

The SDK infers the partition key from the document body's 'id' field, so we
simply remove the partition_key parameter from the call.

Fixes embedding service failures in Azure deployments.

Signed-off-by: Alan Jowett <alanjo@microsoft.com>

* fix: enable test_update_document on emulator - it passes with the fix

The test was incorrectly skipped because we thought the emulator had a
compatibility issue with replace_item. Actually, the issue was our bug:
passing partition_key to replace_item() which doesn't support it.

With the fix from this PR, the test passes on both emulator and real Cosmos DB.
All 17 integration tests now pass.

Signed-off-by: Alan Jowett <alanjo@microsoft.com>

* fix: address PR review comments

1. Updated troubleshooting doc to reflect correct root cause (API inconsistency,
   not version-specific regression)
2. Removed unnecessary azure-cosmos <4.14.0 version constraint from setup.py
   since the actual fix is removing partition_key from replace_item()
3. Updated history, prevention, and timeline sections to be accurate

Signed-off-by: Alan Jowett <alanjo@microsoft.com>

---------

Signed-off-by: Alan Jowett <alanjo@microsoft.com>

8755 of 11246 relevant lines covered (77.85%)

0.87 hits per line

Coverage Regressions

Lines Coverage ∆ File
19
79.66
 0.0% adapters/copilot_storage/copilot_storage/azure_cosmos_document_store.py
Jobs
ID Job ID Ran Files Coverage
1 copilot_vectorstore_integration - 21656314354.1 8
33.08
 GitHub Action Run
2 copilot_logging - 21656314354.2 6
92.64
 GitHub Action Run
3 copilot_chunking - 21656314354.3 2
91.91
 GitHub Action Run
4 copilot_error_reporting - 21656314354.4 5
75.34
 GitHub Action Run
5 embedding - 21656314354.5 2
81.42
 GitHub Action Run
6 copilot_consensus - 21656314354.6 3
95.48
 GitHub Action Run
7 copilot_storage_integration - 21656314354.7 8
33.3
 GitHub Action Run
8 summarization - 21656314354.8 2
91.22
 GitHub Action Run
9 copilot_archive_fetcher - 21656314354.9 9
68.81
 GitHub Action Run
10 copilot_storage - 21656314354.10 8
75.38
 GitHub Action Run
11 ingestion - 21656314354.11 5
79.85
 GitHub Action Run
12 copilot_metrics - 21656314354.12 7
72.95
 GitHub Action Run
13 copilot_schema_validation - 21656314354.13 7
85.23
 GitHub Action Run
14 copilot_summarization - 21656314354.14 8
90.61
 GitHub Action Run
15 chunking - 21656314354.15 2
82.75
 GitHub Action Run
16 copilot_schema_validation_integration - 21656314354.16 7
42.05
 GitHub Action Run
17 copilot_embedding - 21656314354.17 7
91.81
 GitHub Action Run
18 copilot_auth - 21656314354.18 12
64.82
 GitHub Action Run
19 orchestrator - 21656314354.19 6
86.25
 GitHub Action Run
20 parsing - 21656314354.20 7
81.9
 GitHub Action Run
21 copilot_archive_fetcher_integration - 21656314354.21 9
47.46
 GitHub Action Run
22 reporting - 21656314354.22 2
84.48
 GitHub Action Run
23 copilot_message_bus - 21656314354.23 13
74.62
 GitHub Action Run
24 copilot_vectorstore - 21656314354.24 8
74.09
 GitHub Action Run
25 copilot_draft_diff - 21656314354.25 6
90.98
 GitHub Action Run
26 copilot_config - 21656314354.26 33
63.43
 GitHub Action Run
27 copilot_message_bus_integration - 21656314354.27 13
30.05
 GitHub Action Run
Source Files on build 21656314354