Support Application Insights web test alert rules via PowerShell #11787
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
|
Can one of the admins verify this patch? |
isra-fel
reviewed
May 6, 2020
| /// Gets or sets the rule AvailabilityCriteria | ||
| /// </summary> | ||
| [Parameter(ParameterSetName = AvailabilityParameterSet, Mandatory = true, HelpMessage = "Switch parameter for using availability criteria Type")] | ||
| public SwitchParameter WebTest { get; set; } |
Member
There was a problem hiding this comment.
I think -WebTestId can already differentiate the parameter set, so -WebTest can be removed.
Contributor
Author
VeryEarly
pushed a commit
to VeryEarly/azure-powershell
that referenced
this pull request
May 7, 2020
…re#11787) * Support Application Insights web test alert rules via PowerShell * Upgrade Microsoft.Azure.Management.Monitor nuget for Sql.Test * Make WebTest switch parameter optional
VeryEarly
added a commit
that referenced
this pull request
May 8, 2020
* apply SDK back * apply SDK back * update parameter for new-azapplicationinsights and create cmdlet update-azapplicationsights * create cmdlets for ApplicationInsightsLinkedStorageComponent * typo fix * export linkedStorageAccount cmdlets * use RestException to catch 'NotFound' without response body * add test case for ApplicationInsights CRUD * add test cases for linked storage account * generate help for new cmdlets * update test cases * update test cases and add record json * update test cases and add record json * upgrade applicationinsights SDK * update test cases * update help * fix help * add default parameter set for linked storage account related cmdlets * update applicationinsight sdk reference in monitor * re-record test cases * update test cases * update test case and record json * update record json * update help markdown * Update Update-AzApplicationInsights.md * Support Application Insights web test alert rules via PowerShell (#11787) * Support Application Insights web test alert rules via PowerShell * Upgrade Microsoft.Azure.Management.Monitor nuget for Sql.Test * Make WebTest switch parameter optional * update record json Co-authored-by: sivanguetta <46525261+sivanguetta@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Support WebTest availability criteria for metric alert V2
Checklist
CONTRIBUTING.mdChangeLog.mdfile(s) has been updated:ChangeLog.mdfile can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md## Upcoming Releaseheader -- no new version header should be added