Skip to content

Conversation

@swatimadhukar
Copy link

[Azure Backup] Bug Fix to display ResourceGroupName in Get-AzureRMBackupVault cmdlet

@azurecla
Copy link

Hi @swatimadhukar, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Samuel Anudeep and others added 10 commits August 20, 2015 20:08
RI from release into dev1
Fixing Bug 4265568. Signed off by SwatiM.
Bug #4266426: Removing Default Parameter set from RegisterContainer c…
RI from dev1 into dev1-anudeeb
…er. Re-recording reg and unreg test cases to remove hacks in the test cases.
Bugfixes and removing hacks. Signed off by SureshT.
FI from dev1 into release
@dragonfly91
Copy link

Adding a few more bug fixes. Taking dependency on DLL. Will update nuget once it's published. Links to PRs in other repos: https://github.com/Azure/hydra-specs-pr/pull/1105, Azure/azure-sdk-for-net#1390

@pcgeek86
Copy link
Contributor

Thanks for proactively fixing this bug. Very useful to have ResourceGroupName on all emitted ARM resources.

Cheers,
Trevor Sullivan
Microsoft MVP: PowerShell

@ogail
Copy link
Contributor

ogail commented Aug 24, 2015

@swatimadhukar why adding Microsoft.Azure.Management.BackupServicesManagement.dll? Please remove it and use NuGet package instead.

Samuel Anudeep added 5 commits August 25, 2015 19:15
RI from dev1 into dev1-anudeeb
Fix for Bug 4308340. Signed off by SureshT. Will incorporate comments in a separate PR.
Commented unused code.
FI from dev1 into release
@dragonfly91
Copy link

  1. Removed the DLL and taken dependency on nuget pkg.
  2. Some bug fixes

markcowl added a commit that referenced this pull request Aug 25, 2015
Adding ResourceGroupName in output of Get-AzureRMBackupVault
@markcowl markcowl merged commit 45ef9da into Azure:dev Aug 25, 2015
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.

10 participants