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

[Text Analytics] Lack text analytics configuration information about US Gov cloud in pipeline. #14790

Closed
Luyunmt opened this issue Sep 2, 2020 · 4 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics needs-team-triage Workflow: This issue needs the team to triage.

Comments

@Luyunmt
Copy link
Contributor

Luyunmt commented Sep 2, 2020

In PR #14633, tests changed to use a exist text analytics service stored in pipeline .

image

Such text analytics was prepared in azure cloud. Need to prepare such text analytics in US Gov cloud and China Cloud, and let pipeline configured properly.

BTW, a text analytics service also deployed by file "test-resources.json", but this service would not be used in test.
@jongio @danieljurek

@ghost ghost added needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Sep 2, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Sep 2, 2020
@tzhanl tzhanl removed customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Sep 2, 2020
@jsquire jsquire added Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics needs-team-triage Workflow: This issue needs the team to triage. labels Sep 2, 2020
@maririos
Copy link
Member

maririos commented Sep 2, 2020

Oh no. looks like I broke something? @danieljurek could you explain to me what this other things are that I broke?
We are working on preparing TA for 3.1-preview.2 which is currently only available in a PPE endpoint but next week will be deployed to other regions.

@danieljurek
Copy link
Member

Chatted w/ @maririos -- These tests will fail because the code in the SDK relies on features in a PPE endpoint hosted by the Text Analytics team. They would need to roll this out to production before we can support these features.

At this time the failure is "expected" because these features don't exist in the sovereign clouds.

A larger question here is that we should consider how to handle these situations where the SDK code is ahead of what versions are in production.

@maririos
Copy link
Member

maririos commented Oct 8, 2020

This behavior has been reverted. @Luyunmt could you verify again?

@Luyunmt
Copy link
Contributor Author

Luyunmt commented Oct 13, 2020

@maririos Great to know. We will verify while the test branch was Rebase.

@Luyunmt Luyunmt closed this as completed Feb 22, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics needs-team-triage Workflow: This issue needs the team to triage.
Projects
None yet
Development

No branches or pull requests

6 participants