Skip to content

Major refactoring: name changes, adding more cmdlets and fixins some bugs #1969

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 21 commits into from
Mar 23, 2016

Conversation

gucalder
Copy link
Contributor

No description provided.

@gucalder
Copy link
Contributor Author

Last failure was in some other team's tests. Merging and checking in again to see if that resolves the issue.

@gucalder
Copy link
Contributor Author

Still failing one test in another project.

@gucalder
Copy link
Contributor Author

Failing this way: D:\workspace\powershell\AzurePowershell.test.targets(104,5): error : Microsoft.Azure.Commands.Sql.Test.ScenarioTests.ElasticPoolCrudTests.TestElasticPoolMetricGet [FAIL] [D:\workspace\powershell\build.proj]

@markcowl
Copy link
Member

@gucalder this normally indicates an assembly load issue in the SQL cmdlets - does your PR update an assembly used by SQL? If so, have you updated the appropriate assembly reference?

@gucalder
Copy link
Contributor Author

I changes the name of the CmdLet called in the Sql test from Get-Metrics to Get-AzureRmMetric

@gucalder
Copy link
Contributor Author

Another test is failing:

Microsoft.Azure.Commands.AzureBackup.Test.ScenarioTests.AzureBackupPolicyTests.SetProtectionPolicyTests
System.Management.Automation.ActionPreferenceStopException : The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Unable to find a matching HTTP request for URL 'GET /Subscriptions/f5303a0b-fae4-4cdb-b44d-0e4c032dde26/resources?$filter=resourceType%20eq%20'Microsoft.Backup/BackupVault'&$top=100'. Calling method ListAsync().

That call does not seem like related to us. I am merging from dev.

@gucalder
Copy link
Contributor Author

The tests passed.

@hovsepm
Copy link
Contributor

hovsepm commented Mar 22, 2016

@hovsepm
Copy link
Contributor

hovsepm commented Mar 23, 2016

@hovsepm hovsepm merged commit 19817cc into Azure:dev Mar 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants