Skip to content

[formrecognizer] update recordings, add delay, switch to westcentralus region#11916

Merged
kristapratico merged 5 commits intoAzure:masterfrom
kristapratico:try-region
Jun 9, 2020
Merged

[formrecognizer] update recordings, add delay, switch to westcentralus region#11916
kristapratico merged 5 commits intoAzure:masterfrom
kristapratico:try-region

Conversation

@kristapratico
Copy link
Contributor

No description provided.

@kristapratico kristapratico requested a review from iscai-msft as a code owner June 9, 2020 16:54
@kristapratico kristapratico changed the title try westus2 region [formrecognizer] update recordings, add delay, switch to westcentralus region Jun 9, 2020
@kristapratico kristapratico self-assigned this Jun 9, 2020
@kristapratico kristapratico added Client This issue points to a problem in the data-plane of the library. Cognitive Services FormRecognizer labels Jun 9, 2020
Copy link
Contributor

@iscai-msft iscai-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

feel free to disreegard my coomment

)
time.sleep(60) # current ask until race condition bug fixed
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be worth it to only do time.sleep for live tests, and to have the cog service account preparer take in a parameter for whether to sleep after creation so not all calls to it have the time.sleep?

@kristapratico kristapratico merged commit 8d54b11 into Azure:master Jun 9, 2020
@kristapratico kristapratico deleted the try-region branch June 9, 2020 22:34
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Jun 11, 2020
…into have_azure_key_credential_policy_take_kwargs

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  typing fix (Azure#11985)
  remove test code that works around service bug (fixed) (Azure#11961)
  Refactor SharedTokenCacheCredential exception handling (Azure#11962)
  add the previous versions of azure-servicebus to the regression exclusion list (Azure#11957)
  Increment package version after release of azure_ai_formrecognizer (Azure#11954)
  Remove redundant await_prepared_test (Azure#11930)
  update changelog date (Azure#11934)
  Sync eng/common directory with azure-sdk-tools repository (Azure#11925)
  [formrecognizer] update recordings, add delay, switch to westcentralus region (Azure#11916)
  Increment package version after release of azure_identity (Azure#11931)
  Resolve Breaking Aggregate Reports (Azure#11825)
  Increment package version after release of azure_ai_textanalytics (Azure#11928)
  update error msg if not supported (Azure#11915)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer Cognitive Services

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants