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

storage account failover complains about LastSyncTimeUnavailable #11292

Closed
qiaozha opened this issue May 7, 2020 · 5 comments
Closed

storage account failover complains about LastSyncTimeUnavailable #11292

qiaozha opened this issue May 7, 2020 · 5 comments
Assignees
@qiaozha
Labels
issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. Mgmt This issue is related to a management-plane library. Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@qiaozha
Copy link
Member

qiaozha commented May 7, 2020 •
edited
Loading

  • Package Name: azure-mgmt-storage
  • Package Version: 9.0.0
  • Operating System:
  • Python Version:

Describe the bug
I am trying to fix the test https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/storage/azure-mgmt-storage/tests/test_cli_mgmt_storage.py for azure-mgmt-storage Python SDK and found that if I didn't comment out the test for failover https://github.com/Azure/azure-sdk-for-python/pull/11018/files#diff-d17221e9561b5e8578495ccace549e0dR703 to run live test against the service, the Test will fail and complains about LastSyncTimeUnavailable

Does the LastSyncTimeUnavailable means different replicate zones sync up time?
Am I missing some kind of configurations to make failover work?

To Reproduce
Steps to reproduce the behavior:

  1. checkout the PR and uncomment the failover
  2. setup your python sdk enviroment
  3. run pytest -s sdk/storage/azure-mgmt-storage

Expected behavior

Screenshots
image

image

Additional context
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label May 7, 2020
@qiaozha qiaozha added Mgmt This issue is related to a management-plane library. Service Attention Workflow: This issue is responsible by Azure service team. Storage Storage Service (Queues, Blobs, Files) labels May 7, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label May 7, 2020
@ghost
Copy link

ghost commented May 7, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @xgithubtriage.

@blueww
Copy link
Member

blueww commented May 14, 2020 •
edited
Loading

We are discussing offline.
This is related with server issue, and has include feature team to the discussion.
Will update if get any conclusion.

@qiaozha qiaozha assigned qiaozha and unassigned vinjiang and blueww May 14, 2020
@qiaozha
Copy link
Member Author

qiaozha commented May 14, 2020

@zfchen95 Seems I can't assign you.

openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this issue Dec 14, 2020
@ArcturusZhang @msyyc
CodeGen from PR 11292 in Azure/azure-rest-api-specs
fa81c32 
Azure ad b2c tenant resource management APIs swagger, merge into public repo (Azure#11292)

* tenant creation azure ad b2c swagger merge into public repo

* adding custom word and formatting json files

* fixing readmes

* removing conflict with master branch

* fixing readme

* fixing go readme

* updated service name and output folder paths

* csharp reeadme for multiple versions?

* csharp reeadme for multiple versions?

* trimming non-public preview platform targets;cleaning up chsarp/go/python readmes

* reesolving custom-words conflict

* Update specification/cpim/resource-manager/readme.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* removing space before the yaml

* restoring and appending azureresourceschema.md

* Update specification/cpim/resource-manager/readme.go.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* Update specification/cpim/resource-manager/readme.go.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* api version and subscription paremeter not in method location

* python readme updates

* python readme updates

* fixing extra commas

* trying variation for go readme

* Update readme.md

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

* trying to match with azure monitor readme

Co-authored-by: Arcturus <ufo54153@gmail.com>
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this issue Dec 14, 2020
@ArcturusZhang @msyyc
CodeGen from PR 11292 in Azure/azure-rest-api-specs
790189d 
Azure ad b2c tenant resource management APIs swagger, merge into public repo (Azure#11292)

* tenant creation azure ad b2c swagger merge into public repo

* adding custom word and formatting json files

* fixing readmes

* removing conflict with master branch

* fixing readme

* fixing go readme

* updated service name and output folder paths

* csharp reeadme for multiple versions?

* csharp reeadme for multiple versions?

* trimming non-public preview platform targets;cleaning up chsarp/go/python readmes

* reesolving custom-words conflict

* Update specification/cpim/resource-manager/readme.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* removing space before the yaml

* restoring and appending azureresourceschema.md

* Update specification/cpim/resource-manager/readme.go.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* Update specification/cpim/resource-manager/readme.go.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* api version and subscription paremeter not in method location

* python readme updates

* python readme updates

* fixing extra commas

* trying variation for go readme

* Update readme.md

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

* trying to match with azure monitor readme

Co-authored-by: Arcturus <ufo54153@gmail.com>
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
msyyc added a commit that referenced this issue Dec 17, 2020
@msyyc @ArcturusZhang
[T2] azureadb2c 2020 12 15 (#15807)
f40a0ff 
* CodeGen from PR 11292 in Azure/azure-rest-api-specs
Azure ad b2c tenant resource management APIs swagger, merge into public repo (#11292)

* tenant creation azure ad b2c swagger merge into public repo

* adding custom word and formatting json files

* fixing readmes

* removing conflict with master branch

* fixing readme

* fixing go readme

* updated service name and output folder paths

* csharp reeadme for multiple versions?

* csharp reeadme for multiple versions?

* trimming non-public preview platform targets;cleaning up chsarp/go/python readmes

* reesolving custom-words conflict

* Update specification/cpim/resource-manager/readme.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* removing space before the yaml

* restoring and appending azureresourceschema.md

* Update specification/cpim/resource-manager/readme.go.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* Update specification/cpim/resource-manager/readme.go.md

Co-authored-by: Arcturus <ufo54153@gmail.com>

* api version and subscription paremeter not in method location

* python readme updates

* python readme updates

* fixing extra commas

* trying variation for go readme

* Update readme.md

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

* trying to match with azure monitor readme

Co-authored-by: Arcturus <ufo54153@gmail.com>
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>

* version,changelog

Co-authored-by: SDKAuto <sdkautomation@microsoft.com>
Co-authored-by: Arcturus <ufo54153@gmail.com>
@tasherif-msft tasherif-msft removed the Storage Storage Service (Queues, Blobs, Files) label Dec 1, 2021
@RAY-316 RAY-316 added the issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. label Dec 15, 2021
@ghost
Copy link

ghost commented Dec 15, 2021

Hi @qiaozha. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text “/unresolve” to remove the “issue-addressed” label and continue the conversation.

@ghost
Copy link

ghost commented Dec 22, 2021

Hi @qiaozha, since you haven’t asked that we “/unresolve” the issue, we’ll close this out. If you believe further discussion is needed, please add a comment “/unresolve” to reopen the issue.

@ghost ghost closed this as completed Dec 22, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@qiaozha qiaozha

Labels
issue-addressed Workflow: The Azure SDK team believes it to be addressed and ready to close. Mgmt This issue is related to a management-plane library. Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@vinjiang @blueww @qiaozha @RAY-316 @tasherif-msft