-
Notifications
You must be signed in to change notification settings - Fork 4k
Azure RT PR for Sprint 44 #91
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
Conversation
Conflicts: src/ResourceManager/StreamAnalytics/Commands.StreamAnalytics/Microsoft.Azure.Commands.StreamAnalytics.dll-Help.xml
… to the build folder
…o dev Conflicts: src/ServiceManagement/Compute/Commands.ServiceManagement.Test/FunctionalTests/ServiceManagementCmdletTestHelper.cs src/ServiceManagement/Compute/Commands.ServiceManagement.Test/FunctionalTests/VMTemplateTests.cs
Console.WriteLine("Error occurred during cleaning up reserved ips.."); | ||
} | ||
} | ||
//vmPowershellCmdlets = new ServiceManagementCmdletTestHelper(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove commented code if it's not used.
fix vm image verification
Address CR
@@ -76,8 +76,8 @@ public void CleanupTest() | |||
{ | |||
} | |||
|
|||
[TestMethod] | |||
[TestCategory(Category.Functional)] | |||
//[TestMethod] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do not comment the code, please remove the tests altogether
Re-activate unit tests.
vmPowershellCmdlets.RunPSScript("Get-AzureService | Remove-AzureService -Force"); | ||
Utilities.RetryActionUntilSuccess(() => vmPowershellCmdlets.RemoveAzureVNetConfig(), "in use", 5, 30); | ||
} | ||
//if (vnetConfig.Count > 0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still commented code
Revert changes made for Set-AzureRoleDefinition and New-AzureRoleDefinition command let
No description provided.