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.

Sign up for GitHub

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

Jump to bottom

Metrics Advisor - Move to the new proxy server based recorder #18001

Closed
jeremymeng opened this issue Oct 1, 2021 · 2 comments · Fixed by #26330
Closed

Metrics Advisor - Move to the new proxy server based recorder #18001

jeremymeng opened this issue Oct 1, 2021 · 2 comments · Fixed by #26330
Assignees
@KarishmaGhiya
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Metrics Advisor

Comments

@jeremymeng
Copy link
Member

jeremymeng commented Oct 1, 2021 •
edited
Loading

See #17994 for details
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Oct 1, 2021
@jeremymeng jeremymeng added the Client This issue points to a problem in the data-plane of the library. label Oct 1, 2021
@jeremymeng jeremymeng added this to the [2021] December milestone Oct 1, 2021
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Oct 1, 2021
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Feb 28, 2022
CodeGen from PR 18001 in Azure/azure-rest-api-specs
c453298 
[CosmosDB][2021-11-15-preview] Client Side Encryption - Added new algorithm field in KeyWrapMetadata .  (Azure#18001)

* Add new algorithm field to KeyWrapMetadata.

* Updated Client Encryption Key examples.

* fixed algorithm in example.

* fixed algo name.
@HarshaNalluru
Copy link
Member

Use this migration guide https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/test-utils/recorder/MIGRATION.md and try migrating the tests to use the new recorder.

And please reach out if you have feedback or for any blockers.

@xirzec xirzec modified the milestones: [2022] May, [2022] June May 17, 2022
@KarishmaGhiya
Copy link
Member

We have an ongoing service issue due to which all our live tests across the languages are failing. Won't be able to work on this until the live tests are green. Moving this to August for now.

@xirzec xirzec modified the milestones: 2022-08, 2022-09 Aug 9, 2022
@xirzec xirzec modified the milestones: 2022-09, 2022-12 Nov 1, 2022
@xirzec xirzec modified the milestones: 2022-12, 2023-02 Jan 11, 2023
@xirzec xirzec removed this from the 2023-02 milestone Mar 31, 2023
@xirzec xirzec mentioned this issue Jun 26, 2023
Merged
xirzec added a commit that referenced this issue Jun 27, 2023
@xirzec
[ai-metrics-advisor] Migrate to test proxy and temporarily disable te…
59fe745 
…sts (#26330)

### Packages impacted by this PR

`@azure/ai-metrics-advisor`

### Issues associated with this PR

Closes #18001

### Describe the problem that is addressed by this PR

This PR migrates `ai-metrics-advisor` to the latest version of the test
recorder so we can fully deprecate the 1.0 version. Due to the tests not
currently passing in live (as they require static resources that are
difficult to maintain), the tests are disabled for now until we can
figure out how to address this issue prior to a future release.
@github-actions github-actions bot locked and limited conversation to collaborators Sep 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@KarishmaGhiya KarishmaGhiya

Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Metrics Advisor
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ai-metrics-advisor] Migrate to test proxy and temporarily disable tests xirzec/azure-sdk-for-js
5 participants
@xirzec @jeremymeng @KarishmaGhiya @HarshaNalluru @ramya-rao-a