Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

fixed typo and updated KQL issues #409

Merged
merged 1 commit into from
Mar 27, 2024
Merged

fixed typo and updated KQL issues #409

merged 1 commit into from
Mar 27, 2024

Conversation

seanluce
Copy link
Contributor

Overview/Summary

Found a small typo changed "zone" -> "region".

Updated and tested KQL for ANF-6 and ANF-7. Previous KQL was showing positive results instead of negative results.

Related Issues/Work Items

n/a

  • To associate a GitHub Issue, use a key word preceded with the GitHub issue number.
  • To associate an ADO Work Item, use the key word AB# succeeded with the ADO Work Item ID.

This PR fixes/adds/changes/removes

  1. one small typo to _index
  2. anf-6.kql
  3. anf-7.kql

Breaking Changes

n/a

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Performed testing and provided evidence (e.g. screenshot of output) for any changes associated to ARG queries and/or scripts
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@seanluce seanluce requested a review from a team as a code owner March 27, 2024 19:46
@seanluce
Copy link
Contributor Author

Tested KQL for previously submitted ANF-6
image

@seanluce
Copy link
Contributor Author

Tested KQL for previously submitted ANF-7
image

Copy link
Contributor

@ejhenry ejhenry left a comment

Choose a reason for hiding this comment

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

LGTM

@ejhenry ejhenry merged commit 0446c42 into Azure:main Mar 27, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants