Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Entra ID support for auto config #2572

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

TheovanKraay
Copy link
Contributor

@TheovanKraay TheovanKraay commented Mar 25, 2025

Add support for Entra ID in the auto config for Azure Cosmos DB vector store support.
This PR also adds a few fixes/enhancements and corresponding tests:

  • fix to ensure getMetadata() returns the metadata fields and values
  • allow metadata field to be defined as the Cosmos DB partition key (and not just /id)
  • allow choice between gateway and direct mode in autoconfig

@ilayaperumalg
Copy link
Member

@TheovanKraay Thanks for the PR! Could you complete the DCO check (by signing off your commit: git commit --amend --signoff) and fix the PR build failure (there is a format issue)?

@TheovanKraay TheovanKraay force-pushed the cosmos-db-entra-id-support branch 2 times, most recently from 9e35834 to 86dccae Compare March 25, 2025 21:34
@TheovanKraay
Copy link
Contributor Author

Thank @ilayaperumalg I've done as you requested. Let me know if there are other issues blocking merge/release?

@TheovanKraay
Copy link
Contributor Author

Hi @ilayaperumalg and all - any eta on when this could be merged and released? I have some other fixes/enhancements I want to make behind this one, but doing them in small increments. Thanks!

@TheovanKraay TheovanKraay force-pushed the cosmos-db-entra-id-support branch 2 times, most recently from 918a8c9 to e988ca0 Compare March 28, 2025 22:04
@TheovanKraay
Copy link
Contributor Author

@ilayaperumalg since the PR is not merged yet, I added another small fix to expose metadata fields in response. I have some more changes planned but will hold off now until this is merged. Hoping it can be pushed soon. Thanks again.

Signed-off-by: Theo van Kraay <theo.van@microsoft.com>
…van@microsoft.com>

Signed-off-by: Theo van Kraay <theo.van@microsoft.com>
Signed-off-by: Theo van Kraay <theo.van@microsoft.com>
Signed-off-by: Theo van Kraay <theo.van@microsoft.com>
@TheovanKraay TheovanKraay force-pushed the cosmos-db-entra-id-support branch from 712e67e to fd959fd Compare March 29, 2025 18:43
@markpollack markpollack added this to the 1.0.0-M7 milestone Mar 31, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants