Skip to content

Websites: Add new snapshots cmdlets #5996

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

Merged
merged 5 commits into from
Apr 28, 2018

Conversation

nking-1
Copy link

@nking-1 nking-1 commented Apr 19, 2018

Description

Adds two new cmdlets. Get-AzureRmWebAppSnapshot returns the snapshots available for a web app. Restore-AzureRmWebAppSnapshot restores a snapshot to a web app.

This branch is rebased on top of pending PR #5967 because it needs the new SDK.

Checklist

Recorded snapshots tests again

Regenerated Restore-AzureRmWebAppSnapshot and filled in help examples.

Refactored RestoreAzureWebAppSnapshot cmdlet. ResourceGroup and Name parameters specify target app.

Rewrote RestoreAzureRmWebAppSnapshot cmdlet help.
@markcowl
Copy link
Member

@Nking92 You will need to fix the issues outlined here: https://azuresdkci.westus2.cloudapp.azure.com/job/powershell/4012/artifact/src/Package/ValidateHelpIssues.csv

pull the latest, and fix merge conflicts

@markcowl markcowl changed the base branch from preview to release-6.0.0 April 26, 2018 19:05
@@ -300,7 +300,6 @@
<Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" />
</ItemGroup>
Copy link
Member

@markcowl markcowl Apr 27, 2018

Choose a reason for hiding this comment

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

You need to add your new recorded mock files here and ensure they are set to CopyAlways

Copy link
Member

Choose a reason for hiding this comment

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

This is the reason for the ci failure

@markcowl
Copy link
Member

@markcowl markcowl merged commit 3a2c7f8 into Azure:release-6.0.0 Apr 28, 2018
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.

3 participants