-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Search] Fix test resource deployment #29599
base: main
Are you sure you want to change the base?
Conversation
API change check API changes are not detected in this pull request. |
79ded77
to
94cabd6
Compare
/azp run js - search - tests |
No pipelines are associated with this pull request. |
/azp run js - search-documents - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
94cabd6
to
9cfc489
Compare
/azp run js - search-documents - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
9cfc489
to
2673a9f
Compare
/azp run js - search-documents - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
2673a9f
to
186d9ee
Compare
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.
Seems like a good set of changes - I like the move to bicep as well. Skipped over anything that looked like it was already part of another PR assuming it's the same but let me know if that's not the case!
@@ -0,0 +1,8 @@ | |||
using './test-resources.bicep' |
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.
woah I have not seen this bicepparam file yet - what is it used for?
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.
I was having trouble using the New-TestResources.ps1
script, so this file is a template for contributors to use to deploy test 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.
LGTM!
186d9ee
to
9c42f32
Compare
9c42f32
to
42b37dc
Compare
/azp run js - search-documents - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
9d10c29
to
225f363
Compare
/azp run js - search-documents - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
225f363
to
5b99c25
Compare
/azp run js - search-documents - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
5b99c25
to
1ced2d5
Compare
Packages impacted by this PR
@azure/search-documents
Issues associated with this PR
Describe the problem that is addressed by this PR
Live test resource deployment hasn't been working correctly. This aims to fix that, enable semantic search for semantic search tests, and allow for easier deployment of test resources.
Designed to be reviewed on a per-commit basis.
Fixes #24531
Fixes #26490
Fixes #27400
Fixes #27669
Checklists