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

add event hub recommendation evhns-1 #107

Merged
merged 2 commits into from
Oct 10, 2023
Merged

add event hub recommendation evhns-1 #107

merged 2 commits into from
Oct 10, 2023

Conversation

ejhenry
Copy link
Contributor

@ejhenry ejhenry commented Oct 6, 2023 •
edited
Loading

Overview/Summary

AB#29816
Adding Event Hub service and recommendation for zone redundancy (evhns-1)

This PR fixes/adds/changes/removes

  1. Recommendation evhns-1 for zone redundancy
  2. ARG query for evhns-1

Breaking Changes

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

lachaves
lachaves approved these changes Oct 10, 2023
View reviewed changes
Copy link
Contributor

@lachaves lachaves left a comment

Choose a reason for hiding this comment

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

Reviewed the changes, ran hugo server on the branch, everything working fine, all looks good.

@lachaves lachaves merged commit d0ef576 into Azure:main Oct 10, 2023
4 checks passed
@ejhenry ejhenry deleted the eventHubZoneRedundant branch October 11, 2023 23:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lachaves lachaves lachaves approved these changes

@rodrigosantosms rodrigosantosms Awaiting requested review from rodrigosantosms

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

Successfully merging this pull request may close these issues.
2 participants
@ejhenry @lachaves