Sentinel-2024-04-01-preview-AdvancedAutomationRulesConditionsForIncidentLabels#29246
Merged
loriatarms merged 4 commits intoAzure:dev-Sentinel-securityinsights-Microsoft.SecurityInsights-2024-04-01-previewfrom Jun 9, 2024
Conversation
Next Steps to Merge✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge. |
Swagger Generation Artifacts
|
Generated ApiView
|
loriatarms
changed the base branch from
main
to
dev-Sentinel-securityinsights-Microsoft.SecurityInsights-2024-04-01-preview
This was referenced May 28, 2024
AzureRestAPISpecReview
added
resource-manager
BreakingChangeReviewRequired
AzureRestAPISpecReview
removed
TypeSpec
loriatarms
changed the title
loriatarms
changed the title
Contributor
Author
|
/azp run |
|
Commenter does not have sufficient privileges for PR 29246 in repo Azure/azure-rest-api-specs |
xuhumsft
added
the
WaitForARMFeedback
rkmanda
added
the
ARMSignedOff
openapi-pipeline-app
bot
removed
the
WaitForARMFeedback
loriatarms
merged commit 1644507
into
Azure:dev-Sentinel-securityinsights-Microsoft.SecurityInsights-2024-04-01-preview
xuhumsft
added a commit
that referenced
this pull request
Sep 26, 2024
…preview (#28437) * Adds base for updating Microsoft.SecurityInsights from version preview/2024-01-01-preview to version 2024-04-01-preview * Updates readme * Updates API version in new specs and examples * Release sentinel 2024 04 01 preview business applications agents (#28504) * businessApplicationAgents * prettier * businessApplicationAgents * add displayName to be required (#28720) * Add breaking changes from last stable version (#28904) * remove 4 values from enum (#28931) * [Enrichment API] Remove deprecated api (#29059) * remove deprecated api * delete examples * remove old enrichment.json reference --------- Co-authored-by: Camilo Gonzalez Tabares <camilog@microsoft.com> * update the pattern for bookmark (#29100) * update bizapps (#29122) * update bizapps * Update BusinessApplicationAgents.json * update name for SDK * update resource name for SDK * update agentsystem * Sentinel-2024-04-01-preview-AdvancedAutomationRulesConditionsForIncidentLabels (#29246) Support automation rules with conditions on incident labels as a collection * Add suppressions for AvoidAdditionalProperties (#29278) * Update readme.md Add suppressions for data connector * Add suppressions * add suppressions (#29432) Co-authored-by: Offir Shvartz <ofshvart@microsoft.com> * Update dataConnectors.json (#29394) * update suffix to port (#29372) * Added x-ms-pageable and NextList to GetRecommendations (#29465) * Fix readonly and additionalProperties in OnDemand (#29595) * fix readonly and additionalProperties in OnDemand * update reason for suppressing AvoidAdditionalProperties * fix prettier * Fix Lro in OnDemand (#29603) * Fix Lro Error in OnDemand * fix prettier * updated settings file (#29658) * updated settings file * fix model issues * ran prettier * Update Entities Controllers Lint Exceptions (#29462) * Update EntityQueryTemplates.json * Update EntityQueries.json * Update Entities.json * Fix File Import Failures (#29515) * fix failures * add 200 response option * update from PR comments * Fileimports fix (#29733) * fix failures * add 200 response option * update from PR comments * update cloud error * Fileimports fix (#29786) * fix failures * add 200 response option * update from PR comments * update cloud error * update error response * update --------- Co-authored-by: xuhumsft <116764429+xuhumsft@users.noreply.github.com> * Remove EnvironmentVariable from Enum (#29949) * Watchlist fix lint failures (#30191) * Watchlist fix lint failures * fix lint * lint * provisioning states * lint again * model validation * update suppression format (#30252) * Dev sentinel securityinsights microsoft.security insights 2024 04 01 preview (#30302) * update suppression format * update ThreatIntelligenceQuery in the readme to suppress AvoidAdditionalProperties * Update readme.md (#30313) * adding comments to the readme file (#30316) * Update BusinessApplicationAgents.json * update readme to fix Lint Diff Error (#30340) * update readme to fix Lint Diff Error * fix format * remove ---- * fix line break * Revert "remove ----" This reverts commit d6842b5. * suppress the correct file (#30375) * update common-types/resource-management/v3/types.json to common-types/resource-management/v5/types.json to resolve Go-SDK issue (#30556) * TI Query small fixes (#30567) * Add blockchain to latest profile * Add additional types * Adds base for updating Microsoft.SecurityInsights from version preview/2022-04-01-preview to version 2022-05-01-preview * Updates readme * Updates API version in new specs and examples * . * changes * . * . --------- Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com> Co-authored-by: Anat Gilenson <53407600+anat-gilenson@users.noreply.github.com> * update resource in common to v5 (#30570) * fix ModelValidationError (#30571) * fix go-sdk error (#30699) * fix go-sdk error * remove odata parameters in post --------- Co-authored-by: Dvir Naim <106969883+dvir-ms@users.noreply.github.com> Co-authored-by: ityankel <47292630+ityankel@users.noreply.github.com> Co-authored-by: Camilo Gonzalez <cgs.camilogonzalez@gmail.com> Co-authored-by: Camilo Gonzalez Tabares <camilog@microsoft.com> Co-authored-by: loriatarms <105870291+loriatarms@users.noreply.github.com> Co-authored-by: sagamzu <52034287+sagamzu@users.noreply.github.com> Co-authored-by: Offir Shvartz <offirsh@yahoo.com> Co-authored-by: Offir Shvartz <ofshvart@microsoft.com> Co-authored-by: Itiel Maimon <itiel.maimon@gmail.com> Co-authored-by: Laith Hisham <49263932+laithhisham@users.noreply.github.com> Co-authored-by: Vaspate <103540155+Vaspate@users.noreply.github.com> Co-authored-by: NoaGoren-Microsoft <160013798+NoaGoren-Microsoft@users.noreply.github.com> Co-authored-by: rheabansal <93624991+rheabansal@users.noreply.github.com> Co-authored-by: nbatyrbb <115049492+nbatyrbb@users.noreply.github.com> Co-authored-by: Rotem2dvir <rotemdvir@microsoft.com> Co-authored-by: moranraz <59017169+moranraz@users.noreply.github.com> Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com> Co-authored-by: Anat Gilenson <53407600+anat-gilenson@users.noreply.github.com>
cheukchuen
pushed a commit
that referenced
this pull request
Jan 24, 2025
…preview (#28437) * Adds base for updating Microsoft.SecurityInsights from version preview/2024-01-01-preview to version 2024-04-01-preview * Updates readme * Updates API version in new specs and examples * Release sentinel 2024 04 01 preview business applications agents (#28504) * businessApplicationAgents * prettier * businessApplicationAgents * add displayName to be required (#28720) * Add breaking changes from last stable version (#28904) * remove 4 values from enum (#28931) * [Enrichment API] Remove deprecated api (#29059) * remove deprecated api * delete examples * remove old enrichment.json reference --------- Co-authored-by: Camilo Gonzalez Tabares <camilog@microsoft.com> * update the pattern for bookmark (#29100) * update bizapps (#29122) * update bizapps * Update BusinessApplicationAgents.json * update name for SDK * update resource name for SDK * update agentsystem * Sentinel-2024-04-01-preview-AdvancedAutomationRulesConditionsForIncidentLabels (#29246) Support automation rules with conditions on incident labels as a collection * Add suppressions for AvoidAdditionalProperties (#29278) * Update readme.md Add suppressions for data connector * Add suppressions * add suppressions (#29432) Co-authored-by: Offir Shvartz <ofshvart@microsoft.com> * Update dataConnectors.json (#29394) * update suffix to port (#29372) * Added x-ms-pageable and NextList to GetRecommendations (#29465) * Fix readonly and additionalProperties in OnDemand (#29595) * fix readonly and additionalProperties in OnDemand * update reason for suppressing AvoidAdditionalProperties * fix prettier * Fix Lro in OnDemand (#29603) * Fix Lro Error in OnDemand * fix prettier * updated settings file (#29658) * updated settings file * fix model issues * ran prettier * Update Entities Controllers Lint Exceptions (#29462) * Update EntityQueryTemplates.json * Update EntityQueries.json * Update Entities.json * Fix File Import Failures (#29515) * fix failures * add 200 response option * update from PR comments * Fileimports fix (#29733) * fix failures * add 200 response option * update from PR comments * update cloud error * Fileimports fix (#29786) * fix failures * add 200 response option * update from PR comments * update cloud error * update error response * update --------- Co-authored-by: xuhumsft <116764429+xuhumsft@users.noreply.github.com> * Remove EnvironmentVariable from Enum (#29949) * Watchlist fix lint failures (#30191) * Watchlist fix lint failures * fix lint * lint * provisioning states * lint again * model validation * update suppression format (#30252) * Dev sentinel securityinsights microsoft.security insights 2024 04 01 preview (#30302) * update suppression format * update ThreatIntelligenceQuery in the readme to suppress AvoidAdditionalProperties * Update readme.md (#30313) * adding comments to the readme file (#30316) * Update BusinessApplicationAgents.json * update readme to fix Lint Diff Error (#30340) * update readme to fix Lint Diff Error * fix format * remove ---- * fix line break * Revert "remove ----" This reverts commit d6842b5. * suppress the correct file (#30375) * update common-types/resource-management/v3/types.json to common-types/resource-management/v5/types.json to resolve Go-SDK issue (#30556) * TI Query small fixes (#30567) * Add blockchain to latest profile * Add additional types * Adds base for updating Microsoft.SecurityInsights from version preview/2022-04-01-preview to version 2022-05-01-preview * Updates readme * Updates API version in new specs and examples * . * changes * . * . --------- Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com> Co-authored-by: Anat Gilenson <53407600+anat-gilenson@users.noreply.github.com> * update resource in common to v5 (#30570) * fix ModelValidationError (#30571) * fix go-sdk error (#30699) * fix go-sdk error * remove odata parameters in post --------- Co-authored-by: Dvir Naim <106969883+dvir-ms@users.noreply.github.com> Co-authored-by: ityankel <47292630+ityankel@users.noreply.github.com> Co-authored-by: Camilo Gonzalez <cgs.camilogonzalez@gmail.com> Co-authored-by: Camilo Gonzalez Tabares <camilog@microsoft.com> Co-authored-by: loriatarms <105870291+loriatarms@users.noreply.github.com> Co-authored-by: sagamzu <52034287+sagamzu@users.noreply.github.com> Co-authored-by: Offir Shvartz <offirsh@yahoo.com> Co-authored-by: Offir Shvartz <ofshvart@microsoft.com> Co-authored-by: Itiel Maimon <itiel.maimon@gmail.com> Co-authored-by: Laith Hisham <49263932+laithhisham@users.noreply.github.com> Co-authored-by: Vaspate <103540155+Vaspate@users.noreply.github.com> Co-authored-by: NoaGoren-Microsoft <160013798+NoaGoren-Microsoft@users.noreply.github.com> Co-authored-by: rheabansal <93624991+rheabansal@users.noreply.github.com> Co-authored-by: nbatyrbb <115049492+nbatyrbb@users.noreply.github.com> Co-authored-by: Rotem2dvir <rotemdvir@microsoft.com> Co-authored-by: moranraz <59017169+moranraz@users.noreply.github.com> Co-authored-by: Mark Cowlishaw <markcowl@microsoft.com> Co-authored-by: Anat Gilenson <53407600+anat-gilenson@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.
ARM (Control Plane) API Specification Update Pull Request
Tip
Overwhelmed by all this guidance? See the
Getting helpsection at the bottom of this PR description.Note
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.
Click here to see the details of Step 1, Breaking Changes review
If you are in purview of Step 1 of the diagram, follow the Breaking Changes review process.
IMPORTANT! This applies even if you believe your PR was mislabeled, for any reason, including tool failure.
Click here to see the details of Step 2, ARM review
See https://aka.ms/azsdk/pr-arm-review.
Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.
Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.
Additional information
Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the
Generated ApiViewcomment added to this PR. You can use ApiView to show API versions diff.Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.
Getting help
Purpose of this PRandDue diligence checklist.Next Steps to Mergecomment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.and https://aka.ms/ci-fix.
queuedstate, please add a comment with contents/azp run.This should result in a new comment denoting a
PR validation pipelinehas started and the checks should be updated after few minutes.