Skip to content

Conversation

@ffarinha-msft
Copy link
Contributor

Submitted via Learn Editor.

@prmerger-automator
Copy link
Contributor

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

@ffarinha-msft ffarinha-msft marked this pull request as ready for review January 25, 2024 11:34
@ffarinha-msft
Copy link
Contributor Author

The purpose of this PR is to add a note on the only public documentation that I could find about sync triggers that in order for this operation to be successful it is important the function app can connect to the storage as well as the same is able to call it self, as this last one is a common configuration issue when using Private Endpoints.

Sync triggers operation failures can cause deployment issues.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 4327ca9:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-functions/functions-deployment-technologies.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit e574c06:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-functions/functions-deployment-technologies.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented Jan 25, 2024

@ggailey777

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment 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 Jan 25, 2024
@ffarinha-msft
Copy link
Contributor Author

Hi @ggailey777 ,

Kindly checking if you have been able to review this PR

@American-Dipper
Copy link
Contributor

sent email and Teams msg to author

@American-Dipper
Copy link
Contributor

email and Teams msg to author

@AnnaMHuff
Copy link
Contributor

PR review team emailed author today

@v-dirichards
Copy link
Contributor

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards
Copy link
Contributor

@ggailey777

Can you review this old PR and determine whether it needs to be closed or merged?

@MicrosoftDocs/public-repo-pr-review-team

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a documentation note about network connectivity requirements for sync trigger operations in Azure Functions. The change emphasizes the importance of maintaining proper connectivity when network restrictions are applied.

  • Adds an important note about connectivity requirements for sync trigger operations
  • Clarifies network configuration needs when using VNET Integration or Private Endpoints

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@prmerger-automator
Copy link
Contributor

@ffarinha-msft : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f4cd352:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-functions/functions-deployment-technologies.md ✅Succeeded

For more details, please refer to the build report.

@ffarinha-msft
Copy link
Contributor Author

@ggailey777

kindly reaching out to check if you had the chance to review this old PR

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.

6 participants