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

[test][mgmt] arm-loadtesting test are failing now due to recording mismatch #27159

Closed
jeremymeng opened this issue Sep 19, 2023 · 4 comments · Fixed by #27168
Closed

[test][mgmt] arm-loadtesting test are failing now due to recording mismatch #27159

jeremymeng opened this issue Sep 19, 2023 · 4 comments · Fixed by #27168
Assignees
Labels
Mgmt This issue is related to a management-plane library. test-reliability Issue that causes tests to be unreliable

Comments

@jeremymeng
Copy link
Member

Seems now we are sending <application/json> in header. maybe something changes in swagger? Also the ci pipeline for arm-loadtesting has been running for a while.

  1. arm-loadtesting
[node-tests]   6 passing (4s)
[node-tests]   1 failing
[node-tests]
[node-tests]   1) Load Testing Resource Operations
[node-tests]        patch resource:
[node-tests]      RecorderError: Unable to find a record for the request GET https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/myjstest/providers/Microsoft.LoadTestService/loadTests/loadtestsResource?api-version=2022-12-01
[node-tests] Header differences:
[node-tests]     <Content-Type> is absent in record, value <application/json>
[node-tests] Body differences:
[node-tests]
[node-tests]       at Recorder.handleTestProxyErrors (/workspaces/js-main/sdk/test-utils/recorder/src/recorder.ts:431:13)
[node-tests]       at Object.sendRequest (/workspaces/js-main/sdk/test-utils/recorder/src/recorder.ts:456:14)
[node-tests]       at processTicksAndRejections (node:internal/process/task_queues:95:5)
[node-tests]       at async Object.sendRequest (/workspaces/js-main/sdk/core/core-rest-pipeline/src/policies/redirectPolicy.ts:39:24)
[node-tests]       at async Object.sendRequest (/workspaces/js-main/sdk/core/core-rest-pipeline/src/policies/retryPolicy.ts:58:22)
[node-tests]       at async Object.sendRequest (/workspaces/js-main/sdk/core/core-client/src/deserializationPolicy.ts:90:24)
[node-tests]       at async Object.sendRequest (/workspaces/js-main/sdk/core/core-rest-pipeline/src/policies/bearerTokenAuthenticationPolicy.ts:189:20)
[node-tests]       at async LoadTestClient.sendOperationRequest (/workspaces/js-main/sdk/core/core-client/src/serviceClient.ts:207:27)
@jeremymeng jeremymeng added Mgmt This issue is related to a management-plane library. test-reliability Issue that causes tests to be unreliable labels Sep 19, 2023
@qiaozha qiaozha assigned MaryGao and kazrael2119 and unassigned qiaozha Sep 20, 2023
@qiaozha
Copy link
Member

qiaozha commented Sep 20, 2023

Adding Mary as she is responsible for issues in this week.

@kazrael2119
Copy link
Contributor

I think push the recordings rerun loadtesting cases may resolve this issue

@kazrael2119
Copy link
Contributor

Hi @jeremymeng ,the pr is nerged now, could you help confirm whether this issue is fixed? thanks

@jeremymeng
Copy link
Member Author

@kazrael2119 thanks for the fix! CI is passing so it should be good

@github-actions github-actions bot locked and limited conversation to collaborators Dec 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Mgmt This issue is related to a management-plane library. test-reliability Issue that causes tests to be unreliable
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants