Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update set-up-zone-redundancy-availability-zones.md #123634

Merged
merged 2 commits into from
Jul 9, 2024

Conversation

sheppyj
Copy link
Contributor

@sheppyj sheppyj commented Jul 5, 2024

Trying to remove the ambiguity from the following Consideration when deploying Zone Redundant Standard Logic Apps;

  • You can enable availability zone redundancy only at creation time using Azure portal. No programmatic tool support, such as Azure PowerShell or Azure CLI, currently exists to enable availability zone redundancy.

The above reads to me (and some of my colleagues) that you can only deploy a zone redundant Standard Logic App/App Service if its done via the Azure Portal ONLY which isn't the case. App Service Plans can be deployed via all manner of CLI tools but Zone Redundancy cannot be enabled post creation. I feel like this consideration should be in fact worded like this;

  • You can enable availability zone redundancy only at creation time. No programmatic tool support, such as Azure PowerShell or Azure CLI, currently exists to enable availability zone redundancy post creation.

Hopefully this removes some of the confusion!

To further demonstrate that the original could be confusing I found an old issue where a user was having the exact same thoughts I was;
Azure/logicapps#887

Trying to remove the ambiguity from the following Consideration when deploying Zone Redundant Standard Logic Apps;

* You can enable availability zone redundancy *only at creation time using Azure portal*. No programmatic tool support, such as Azure PowerShell or Azure CLI, currently exists to enable availability zone redundancy.

The above reads to me (and some of my colleagues) that you can only deploy a zone redundant Standard Logic App/App Service if its done via the Azure Portal ONLY which isn't the case. App Service Plans can be deployed via all manner of CLI tools but Zone Redundancy cannot be enabled post creation. I feel like this consideration should be in fact worded like this;

* You can enable availability zone redundancy *only at creation time*. No programmatic tool support, such as Azure PowerShell or Azure CLI, currently exists to enable availability zone redundancy post creation.

Hopefully this removes some of the confusion!
Copy link
Contributor

@sheppyj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit 6cf0484:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/set-up-zone-redundancy-availability-zones.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Jul 5, 2024

@ecfan
Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Jul 5, 2024
Change "post" to "after" for clarity

Co-authored-by: Esther Fan <ecfan@users.noreply.github.com>
Copy link
Contributor

Learn Build status updates of commit c217d9f:

✅ Validation status: passed

File Status Preview URL Details
articles/logic-apps/set-up-zone-redundancy-availability-zones.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@sheppyj
Copy link
Contributor Author

sheppyj commented Jul 9, 2024

Thank you @ecfan I have updated with a slight amendment!

Copy link
Contributor

@ecfan ecfan left a comment

Choose a reason for hiding this comment

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

Thank you!

@ecfan
Copy link
Contributor

ecfan commented Jul 9, 2024

#sign-off

@Jak-MS Jak-MS merged commit 31d0e2e into MicrosoftDocs:main Jul 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants