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

agw-7 check for AppGw deployed in multiple Zones #417

Merged
merged 3 commits into from
Mar 28, 2024
Merged

agw-7 check for AppGw deployed in multiple Zones #417

merged 3 commits into from
Mar 28, 2024

Conversation

ReneHezser
Copy link
Contributor

Overview/Summary

Creates agw-7 recommendation for zone-redundant Application Gateway deployment

image

Related Issues/Work Items

This PR fixes/adds/changes/removes

  1. add agw-7 recommendation with resource graph query

Breaking Changes

none

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.)

@ReneHezser ReneHezser requested review from a team as code owners March 28, 2024 16:43
DaFitRobsta
DaFitRobsta approved these changes Mar 28, 2024
View reviewed changes
Copy link
Contributor

@DaFitRobsta DaFitRobsta 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 approved these changes Mar 28, 2024
View reviewed changes
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 ffefd2a into Azure:main Mar 28, 2024
4 checks passed
@ReneHezser ReneHezser deleted the agw-7 branch April 8, 2024 05:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@DaFitRobsta DaFitRobsta DaFitRobsta approved these changes

@ejhenry ejhenry ejhenry approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ReneHezser @ejhenry @DaFitRobsta