Skip to content

Conversation

@sambitratha
Copy link
Contributor

Description
Enabled Alternate Location Restore for SQL/SAPHANA protected items

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@yungezz
Copy link
Member

yungezz commented Feb 20, 2021

hi @qwordy could you pls help to review? thanks

target_item = show_protectable_instance(cmd, protectable_items_client, resource_group_name, vault_name,
target_server_name, target_server_type,
workload_type, container_name)
if target_container_name is None:
Copy link
Member

Choose a reason for hiding this comment

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

the change is ok, while code could be refined.

@yungezz
Copy link
Member

yungezz commented Feb 20, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@qwordy
Copy link
Member

qwordy commented Feb 20, 2021

Code owner review required

@sambitratha
Copy link
Contributor Author

Hi @fengzhou-msft could you please review this PR?

@sambitratha
Copy link
Contributor Author

Hi @fengzhou-msft i've resolved your comment. please restart the azure pipelines and approve. Thanks

@yungezz
Copy link
Member

yungezz commented Feb 23, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@fengzhou-msft fengzhou-msft merged commit d2a2152 into Azure:dev Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants