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

Updated with latest APRL recommendations. #79

Merged
merged 69 commits into from
Oct 17, 2023
Merged

Updated with latest APRL recommendations. #79

merged 69 commits into from
Oct 17, 2023

Conversation

jimays-avila
Copy link
Contributor

@jimays-avila jimays-avila commented Sep 13, 2023

Overview/Summary

ARG queries Output Standardization for Automation

AB#31152
AB#31316
AB#31317
AB#31318
AB#31319
AB#31320
AB#30178

This PR fixes/adds/changes/removes

  1. Changed all kql queries to match standardization for automation

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
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

Copy link
Contributor

@rodrigosantosms rodrigosantosms left a comment

Choose a reason for hiding this comment

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

PLease see the required changes for each file/line.

@jimays-avila
Copy link
Contributor Author

@rodrigosantosms can you please review the changes.

@jimays-avila
Copy link
Contributor Author

I made the necessary changes to align with the naming convention of the folders and files and kql queries.

oZakari and others added 2 commits October 6, 2023 15:50
* VM queries added (#61)

Co-authored-by: Luis Alfonso Chaves <[email protected]>
Co-authored-by: Rodrigo Santos <[email protected]>

* Added SQLDB-7 recommendation (#77)

Co-authored-by: Rodrigo Santos <[email protected]>

* Updating vnet-2 query (#71)

Co-authored-by: Luis Alfonso Chaves <[email protected]>
Co-authored-by: Rodrigo Santos <[email protected]>

* vm-updates (#80)

Co-authored-by: Rodrigo Santos <[email protected]>

* Updating vmss - 9-18-2023-1 (#83)

Co-authored-by: Rodrigo Santos <[email protected]>

* updates to LB, VMs and VMSSs (#84)

Co-authored-by: Rodrigo Santos <[email protected]>

* major-updates-sep-2023 (#85)

Co-authored-by: Rodrigo Santos <[email protected]>

* Bug Fix: Abbrev. names to support CAF guidance (#81)

* Bug Fix-Fix sorting to prioritize title vs weight when applicable (#82)

Co-authored-by: Rodrigo Santos <[email protected]>

* Revert back to prioritize content weight (#89)

* Query Bug Fixes and Project standardization (#93)

Co-authored-by: Rodrigo Santos <[email protected]>

* Attempt to get Traffic manager and app gateway queries to display again (#92)

Co-authored-by: Rodrigo Santos <[email protected]>

* Bug Fixes-VM-4 and VM-7 Resource Graph Queries (#87)

* Updating Web App and App Servcices which were not created with Hugo (#96)

Co-authored-by: Rodrigo Santos <[email protected]>

* updating web-app (#97)

Co-authored-by: Rodrigo Santos <[email protected]>

* Adding recommendationId to Traffic Manager queries (#98)

Co-authored-by: Rodrigo Santos <[email protected]>

* Moving web-app from draft to published (#99)

Co-authored-by: Rodrigo Santos <[email protected]>

* Major Update - Adding Categories (#100)

Co-authored-by: Rodrigo Santos <[email protected]>

* RedisAvailabilityZones (#101)

* added application insights rule ins-1 (#104)

Co-authored-by: Eric Henry <[email protected]>

* add SignalR recommendation sigr-1 (#102)

Co-authored-by: Luis Alfonso Chaves <[email protected]>

* Update typo in appi-1.kql (#106)

* General formatting fixes for app gateway queries

---------

Co-authored-by: René Hézser <[email protected]>
Co-authored-by: Luis Alfonso Chaves <[email protected]>
Co-authored-by: Rodrigo Santos <[email protected]>
Co-authored-by: maheshbenke <[email protected]>
Co-authored-by: Rodrigo Santos <[email protected]>
Co-authored-by: Eric Henry <[email protected]>
Co-authored-by: Onder yildirim <[email protected]>
@oZakari
Copy link
Contributor

oZakari commented Oct 9, 2023

Currently working with @jimays-avila on resolving merge conflicts.

Copy link
Contributor Author

@jimays-avila jimays-avila left a comment

Choose a reason for hiding this comment

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

Are there any other changes that need to be made

@oZakari oZakari closed this Oct 17, 2023
@oZakari oZakari reopened this Oct 17, 2023
@oZakari oZakari removed the request for review from rodrigosantosms October 17, 2023 19:18
@oZakari oZakari merged commit dd8c4de into Azure:main Oct 17, 2023
3 of 4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants