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

[Text Analytics] [AAD] TextAnalyticsClient LROs #analyze some documents with errors and multiple actions failing in nightly runs #18827

Closed
v-jiaodi opened this issue Nov 26, 2021 · 3 comments
Closed

[Text Analytics] [AAD] TextAnalyticsClient LROs #analyze some documents with errors and multiple actions failing in nightly runs #18827

v-jiaodi opened this issue Nov 26, 2021 · 3 comments
Assignees
@deyaaeldeen
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics test-reliability Issue that causes tests to be unreliable

Comments

@v-jiaodi
Copy link
Member

Text Analytics nightly test runs are failing with:

Error message:
Internal server error

Stack trace:
RestError: Internal server error
at compileError (/Users/runner/work/1/s/sdk/textanalytics/ai-text-analytics/src/util.ts:76:46)
at AnalyzeLro.sendInitialRequest (/Users/runner/work/1/s/sdk/textanalytics/ai-text-analytics/src/analyzeLro.ts:16:55)
at runMicrotasks ()
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at GenericPollOperation.update (/Users/runner/work/1/s/sdk/core/core-lro/src/lroEngine/operation.ts:70:22)
at LroEngine.pollOnce (/Users/runner/work/1/s/sdk/core/core-lro/src/poller.ts:315:26)

For more details check here:
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 26, 2021
@v-xuto v-xuto added Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics test-reliability Issue that causes tests to be unreliable and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Nov 26, 2021
@deyaaeldeen
Copy link
Member

This is a service-side issue and I am reluctant to change the SDK-side test for it. /cc @maorleger is there a guidance on what to do for such cases?

@maorleger
Copy link
Member

Agree that I wouldn't change the SDK-side tests for this. An Internal server error (500) is something that should be investigated on the service side - is this repeatable or inconsistent? If repeatable maybe we can reproduce it with logging enabled and send them the request-id?

@deyaaeldeen
Copy link
Member

@maorleger it is not consistent and the latest run succeeded. I will close this issue then based on our discussion.

azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue Jun 8, 2022
CodeGen from PR 18827 in Azure/azure-rest-api-specs
9db9dcb 
microsoft.storagecache 2022-05-01 (Azure#18827)

* Adds base for updating Microsoft.StorageCache from version stable/2022-01-01 to version 2022-05-01

* Updates readme

* Updates API version in new specs and examples

* Add swagger changes for 2022-05-01

* Fixup for Swagger PrettierCheck

* Add primingjob to custom-words.txt

* Add spaceallocationparameter to custom-words.txt

* Fix example body name

* Add location & async-operation headers & Rework primingJob action

* Add word primingjobid

* Minor: Change body parameter names

* Standerdize operationId
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@deyaaeldeen deyaaeldeen

Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics test-reliability Issue that causes tests to be unreliable
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@maorleger @deyaaeldeen @v-xuto @v-jiaodi