Skip to content
This repository has been archived by the owner on Mar 30, 2023. It is now read-only.

Use resource group to get details for azure-repository storage account #229

Merged
merged 4 commits into from
Oct 4, 2018

Conversation

russcam
Copy link
Contributor

@russcam russcam commented Oct 3, 2018

Use resource group to get details for azure-repository storage account. This was received as feedback from Microsoft on the last template submission.

This commit updates the parameters required to configure azure repository plugin with an
existing storage account in another resource group. Instead of passing the storage account
key and endpoint suffix, these values can be determined with the storage account name and
resource group.

key and suffix are removed and replaced with resourceGroup.

This now requires that the storage account and resoruce group exist in the same
subscription. The implementation could be expanded in future to accept a subscriptionId,
if necessary.

Closes #215

This commit updates the parameters required to configure azure repository plugin with an
existing storage account in another resource group. Instead of passing the storage account
key and endpoint suffix, these values can be determined with the storage account name and
resource group.

key and suffix are removed and replaced with resourceGroup.

Note that this now requires that the storage account and resoruce group exist in the same
subscription. The implementation could be expanded in future to accept a subscriptionId,
if necessary.

Closes #215
@russcam russcam requested a review from Mpdreamz October 3, 2018 03:52
@russcam russcam added the 6.4 label Oct 3, 2018
@russcam
Copy link
Contributor Author

russcam commented Oct 3, 2018

Run

npm run links -- --branch master

before merging

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM 👍 much safer and also easier for the user.

@russcam russcam merged commit 4ef21bc into master Oct 4, 2018
@russcam russcam deleted the feature/repository-azure-rg branch October 16, 2018 04:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use resource group to get details for azure-repository storage account
2 participants