Skip to content

[Storage] Support Point in Time Restore#11080

Merged
wyunchi-ms merged 60 commits intoAzure:AzureRM.Storage.Locationfrom
wastoresh:pitr
Feb 11, 2020
Merged

[Storage] Support Point in Time Restore#11080
wyunchi-ms merged 60 commits intoAzure:AzureRM.Storage.Locationfrom
wastoresh:pitr

Conversation

@blueww
Copy link
Copy Markdown
Member

@blueww blueww commented Feb 10, 2020 •
edited
Loading

Description

The design in https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/466
Almost pass review

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:
    • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
    • the markdown help files have been regenerated using the commands listed here

anusapan and others added 30 commits January 22, 2020 15:37
@anusapan
Manage device cmdlets
3d22201
@anusapan
Merge branch 'master' of https://github.com/Azure/azure-powershell in…
4701411 
…to manage-device
@anusapan
Fixed build issues
0ab71e2
@saisujithreddym
adding support for firewall policy on vnet firewall
c76e5cd
@anusapan
Remove preview dependencies
353d860
@anusapan
Merge branch 'master' of https://github.com/Azure/azure-powershell in…
964fdcf 
…to manage-device
@saisujithreddym
adding tests
86084c4
@saisujithreddym
cleanup
31f62d3
@saisujithreddym
test fix
31fb3ee
@saisujithreddym
resolving merge conflicts
05d4b3b
@saisujithreddym
clean up
ac104f4
@saisujithreddym
removing the test
436c4e1
@saisujithreddym
clean up
48c06d4
@saisujithreddym
vnet policy changes
c5234aa
@saisujithreddym
clean up
7705e0f
@VeryEarly
Merge pull request Azure#11013 from saisujithreddym/vnetpolicynrp
c07c5fa 
Firewall Policy support on vnet firewall
@yangyuan
Fix error message when server returns 202
6be6a8e
@yangyuan
Update SDK to 7.0.0.
2013dfc
@yangyuan
Update change log
bd2f76f
@yangyuan
Update style
c4b4169
@digavino
Initial cmdlet to get managed rule definitions
232f6ba
@digavino
Add test cases for new cmdlet
39a573a
@digavino
Update help
6cfaf10
@sivanguetta
Support a new parameter called -ActionGroupId in the Add-AzMetricAler…
e414ef8 
…tRuleV2 command
@sivanguetta
Update ChangeLog.Md
2edcb90
@digavino
Update changelog
1daa987
@digavino
Fix test
0080ad8
@digavino
Merge branch 'master' of https://github.com/Azure/azure-powershell in…
7c802c3 
…to waf-get-managed-rules
@anusapan
Re-record the test session
82c2683
@theheatDK
Update Get-AzLog.md
35a9b94 
The -CorrelationId parameter is not a required parameter.
dingmeng-xue and others added 2 commits February 11, 2020 00:11
@dingmeng-xue
Add new version of common packages
468e8f2
@VeryEarly
Merge pull request Azure#11045 from sivanguetta/master
b10da39 
Support a new parameter called -ActionGroupId in the Add-AzMetricAlertRuleV2 command in Az.Monitor
dingmeng-xue and others added 15 commits February 11, 2020 00:39
@dingmeng-xue
Add new version of common packages
3638a46
@dingmeng-xue
Polished changelog.md
a348e2e
@dingmeng-xue
Corrected tense
1af2e9c
@theheatDK
Update Get-AzLog.md
5e8f60b 
Added "This parameter is required." again.
@theheatDK
Update ChangeLog.md
728f68c
@anusapan
Update parameter Name to IotHubName
420a731
@VeryEarly
Merge pull request Azure#11084 from dingmeng-xue/release-2020-02-18
812c204 
Polished changelog.md
@VeryEarly
Update Get-AzLog.md
8ec6123
@VeryEarly
Update Get-AzLog.md
9898ed9
@VeryEarly
Merge branch 'master' into patch-10
5ecb67e
@VeryEarly
Merge pull request Azure#10960 from anusapan/manage-device
10b3462 
[IoT] Add support to manage devices in an Iot Hub.
@VeryEarly
Merge pull request Azure#10959 from yuanyang-microsoft/yuanyang/async…
fba96af 
…_delete

Az.CognitiveServices - Update SDK to 7.0, Improve error message
@wyunchi-ms
Merge pull request Azure#11078 from Azure/tvuylsteke-patch-1
0d00e9d 
Adding info on IANAPrivateRanges
@VeryEarly
Merge pull request Azure#11068 from theheatDK/patch-10
57c3343 
Small documentation change to Get-AzLog.md
@dingmeng-xue
Merge pull request Azure#11079 from dingmeng-xue/telemetry
370d85b 
Add SubscriptionId, TenantId, and execution time into client telemetry
@blueww blueww requested a review from wyunchi-ms February 11, 2020 05:53
@blueww
Copy link
Copy Markdown
Member Author

blueww commented Feb 11, 2020

@wyunchi-ms
The test failed with 214 network cases failed.
But seems I don't have any change on this. And the cases pass on my local machine.
Would you please help to look how to fix it?

Following is error log:

System.Management.Automation.RuntimeException : Test failed due to a non-empty error stream. First error : PowerShell Error Record: The specified module 'D:\a\1\s\artifacts\Debug\Az.Sql\Az.Sql.psd1' was not loaded because no valid module file was found in any module directory. [D:\a\1\s\build.proj]
Exception:System.IO.FileNotFoundException: The specified module 'D:\a\1\s\artifacts\Debug\Az.Sql\Az.Sql.psd1' was not loaded because no valid module file was found in any module directory.

@wyunchi-ms wyunchi-ms merged commit 8bf6e2a into Azure:AzureRM.Storage.Location Feb 11, 2020
@blueww blueww mentioned this pull request Aug 28, 2020
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wyunchi-ms wyunchi-ms wyunchi-ms approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.