Skip to content

Updated AzureActivity.json#796

Merged
shainw merged 2 commits into
Azure:masterfrom
samikroy:master
Sep 24, 2020
Merged

Updated AzureActivity.json#796
shainw merged 2 commits into
Azure:masterfrom
samikroy:master

Conversation

@samikroy

@samikroy samikroy commented Jun 26, 2020

Copy link
Copy Markdown
Contributor

Reason : Making Caller pill similar to ResourceGroup to get data from Log Analytics workspace instead of resource group.This will be enable users with limited permission only to workspace to deploy the workbook, instead the resource group.

image

Reason : Making Caller pill similar to ResourceGroup to get data from Log Analytics workspace instead of resource group.This will be enable users with limited permission only to workspace to deploy the workbook, instead the resource group.
Comment thread Workbooks/AzureActivity.json
Updated workbooksMetadata for AzureActivity workbook fix for making Caller pill similar to ResourceGroup to get data from Log Analytics workspace instead of resource group.This will be enable users with limited permission only to workspace to deploy the workbook, instead the resource group..
@samikroy

Copy link
Copy Markdown
Contributor Author

@Amitbergman Could you please help with the review,

@samikroy

Copy link
Copy Markdown
Contributor Author

@shainw - Could you please help with review.

@shainw

shainw commented Sep 24, 2020

Copy link
Copy Markdown
Contributor

@shainw - Could you please help with review.

I pinged @Amitbergman to Approve and then we can complete this one.

@Amitbergman Amitbergman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good to me

@shainw shainw merged commit ccae1fb into Azure:master Sep 24, 2020
@samikroy

Copy link
Copy Markdown
Contributor Author

Thank you @Amitbergman & @shainw for your help here.

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.

3 participants