Skip to content

fix: Duplicated Query - b60ae773-9917-4bca-8a42-7cb45365a917.kql#720

Merged
tksh164 merged 1 commit into
mainfrom
rosanto-fix-resourceType
Apr 23, 2025
Merged

fix: Duplicated Query - b60ae773-9917-4bca-8a42-7cb45365a917.kql#720
tksh164 merged 1 commit into
mainfrom
rosanto-fix-resourceType

Conversation

@rod-reis
Copy link
Copy Markdown
Contributor

This query was duplicated with this df0ff862-814d-45a3-95e4-4fad5a244ba6 from Compute/VirtualMachines and it was also incorrectly associated with Subscriptions.

Removing it to avoid issues in CxO.

There is already a query that captures this regardless of if it is specialized workload or not.
Copilot AI review requested due to automatic review settings April 22, 2025 21:54
@rod-reis rod-reis requested review from a team as code owners April 22, 2025 21:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR aims to remove a duplicated query that was causing mis-association issues with Subscriptions, thereby avoiding potential conflicts in CxO. The key changes include:

  • Removal of the duplicated query from the SAP KQL file.
  • Replacement of the query code with a placeholder comment.
Comments suppressed due to low confidence (1)

azure-specialized-workloads/sap/kql/b60ae773-9917-4bca-8a42-7cb45365a917.kql:1

  • [nitpick] The placeholder comment 'under-development' may lead to ambiguity regarding the intended state of this file. Consider either removing the file entirely if it is no longer needed or updating the comment to clearly indicate that the duplicated query has been removed.
// under-development

@kpoineal kpoineal changed the title Fix - Duplicated Query - b60ae773-9917-4bca-8a42-7cb45365a917.kql fix: Duplicated Query - b60ae773-9917-4bca-8a42-7cb45365a917.kql Apr 23, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Bug 🐞 Something isn't working label Apr 23, 2025
@tksh164 tksh164 merged commit 99c757e into main Apr 23, 2025
8 of 9 checks passed
@tksh164 tksh164 deleted the rosanto-fix-resourceType branch April 23, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug 🐞 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants