-
Notifications
You must be signed in to change notification settings - Fork 90
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
docs: aisearch resiliency recommendations #603
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Copilot <[email protected]>
…695-8f90-9a44544fdfa4.kql Co-authored-by: Copilot <[email protected]>
…621-98b3-c877c65cb195.kql Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (2)
azure-resources/CognitiveServices/AIsearch/recommendations.yaml:11
- The word 'potentialBenefits' is incorrectly placed within the description text and should be removed.
longDescription: | Availability zones are used when adding multiple replicas to your search service. Each replica is assigned to a different zone in the region. If there are more replicas than zones, they are distributed as evenly as possible across the available zones. potentialBenefits: High Availability
azure-resources/CognitiveServices/AIsearch/kql/b376281d-bfec-4695-8f90-9a44544fdfa4.kql:7
- The use of
strcat
is redundant when concatenating a single property. It should be replaced with the property itself.
| project recommendationId = "b376281d-bfec-4695-8f90-9a44544fdfa4", name, id, param1 = strcat(properties['replicaCount']) , param2 = strcat(location)
@ejhenry please review, this PR is just pending one final review. thanks |
azure-resources/Search/searchServices/kql/dff62efe-c3a3-4621-98b3-c877c65cb195.kql
Outdated
Show resolved
Hide resolved
- name: Reliability in Azure AI Search | ||
url: "https://learn.microsoft.com/en-us/azure/search/search-reliability#multiple-services-in-separate-geographic-regions" | ||
|
||
- description: Use Azure Traffic Manager to route search requests via geo-located clients for failover redundancy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This recommendation might make more sense in the AI specialized workload section as it seems to be targeting Traffic Manager for a specific AI scenario. As a specialized workload recommendation, the targeted resource can be a subscription. In this current location, the targeted resources for this recommendation could only be Microsoft.Search/searchServices resources.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved the recommendation here azure-specialized-workloads/ai/recommendations.yaml
Head branch was pushed to by a user without write access
Head branch was pushed to by a user without write access
Head branch was pushed to by a user without write access
…af0-eb51eed5352e.kql
Head branch was pushed to by a user without write access
@@ -0,0 +1,2 @@ | |||
// Azure Resource Graph Query |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Azure Resource Graph Query |
recommendationTypeId: null | ||
recommendationControl: HighAvailability | ||
recommendationImpact: Medium | ||
recommendationResourceType: Microsoft.Search/searchServices |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
recommendationResourceType: Microsoft.Search/searchServices | |
recommendationResourceType: Microsoft.Subscription/subscriptions |
Overview/Summary
AI Search resiliency: Enable AZ support in AI Search by configuring multiple replicas to your search service
AI Search Resiliency: Enable Multi Region deployments for AI Search
Related Issues/Work Items
https://dev.azure.com/CSUSolEng/Accelerator%20-%20AI/_workitems/edit/38367
https://dev.azure.com/CSUSolEng/Accelerator%20-%20AI/_workitems/edit/38366
main
branch