Skip to content

Update Start-AzureRmRecoveryServicesAsrTestFailoverCleanupJob.md #7511

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 1 commit into from
Oct 16, 2018
Merged

Update Start-AzureRmRecoveryServicesAsrTestFailoverCleanupJob.md #7511

merged 1 commit into from
Oct 16, 2018

Conversation

maorteg
Copy link
Contributor

@maorteg maorteg commented Oct 10, 2018

The example 2 "Clean up Test Failover using Recovery Plan" code is incorrect. I tested this on my lab and confirmed that correct syntax should be < Start-AzureRmRecoveryServicesAsrTestFailoverCleanupJob -RecoveryPlan $recoveryPlan -Comment "testing done" >.

Please, update the documentation accordingly.

Description

Checklist

The example 2 "Clean up Test Failover using Recovery Plan" code is incorrect. I tested this on my lab and confirmed that correct syntax should be  < Start-AzureRmRecoveryServicesAsrTestFailoverCleanupJob -RecoveryPlan $recoveryPlan -Comment "testing done" >.

Please, update the documentation accordingly.
@adxsdkps
Copy link
Collaborator

Can one of the admins verify this patch?

@cormacpayne
Copy link
Member

@azuresdkci add to whitelist

@cormacpayne
Copy link
Member

@maorteg Hey Marvin, thanks for the fix! 😀

@viverm Hey Vipin, would you mind taking a look at this PR when you have the chance?

@viverm
Copy link
Contributor

viverm commented Oct 16, 2018

@maorteg Thanks for PR.

@cormacpayne can we merge this PR for next release.

Change is in help file.

@cormacpayne cormacpayne merged commit 1dc601d into Azure:preview Oct 16, 2018
@maorteg maorteg deleted the patch-1 branch November 5, 2018 23:34
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.

5 participants