Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove Qdrant from webapi scripts and update default package zip filename #1118

Merged
merged 2 commits into from
May 19, 2023

Conversation

adrianwyatt
Copy link
Contributor

Motivation and Context

Qdrant deployed to Azure is not currently secured.

Description

  • Remove Qdrant from cloud deployment while securing work progresses
  • Update payload package zipfile name.

@adrianwyatt adrianwyatt requested a review from glahaye May 19, 2023 23:00
@adrianwyatt adrianwyatt self-assigned this May 19, 2023
@adrianwyatt adrianwyatt added PR: ready for review All feedback addressed, ready for reviews samples labels May 19, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) May 19, 2023 23:38
@adrianwyatt adrianwyatt merged commit 224d2cf into microsoft:main May 19, 2023
@adrianwyatt adrianwyatt deleted the noqdrant branch May 22, 2023 17:36
shawncal pushed a commit to shawncal/semantic-kernel that referenced this pull request Jul 6, 2023
…name (microsoft#1118)

### Motivation and Context
Qdrant deployed to Azure is not currently secured.

### Description
- Remove Qdrant from cloud deployment while securing work progresses
- Update payload package zipfile name.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
…name (microsoft#1118)

### Motivation and Context
Qdrant deployed to Azure is not currently secured.

### Description
- Remove Qdrant from cloud deployment while securing work progresses
- Update payload package zipfile name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready for review All feedback addressed, ready for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants