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.

Sign up for GitHub

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

Jump to bottom

Update backup-azure-diagnostic-events.md #124822

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Update backup-azure-diagnostic-events.md #124822

merged 1 commit into from
Dec 18, 2024

Conversation

azarboon
Copy link
Contributor

I've found a gap in the documentation. Following info is scattered around the web and different pages. I've added it to doc for further clarity:

Recovery Services vaults can send diagnostic logs to a storage account located in the same region. They can also send these logs to a Log Analytics workspace that may be situated in either the same or a different region.

@prmerger-automator PRMerger Automator
Copy link
Contributor

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

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit f62d7c6:

✅ Validation status: passed

File Status Preview URL Details
articles/backup/backup-azure-diagnostic-events.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented Nov 11, 2024

@AbhishekMallick-MS

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 Nov 11, 2024
@AbhishekMallick-MS
Copy link
Contributor

AbhishekMallick-MS commented Dec 6, 2024 •
edited
Loading

Hi @azarboon -Thanks for your updates. We will get back after receiving confirmation from Product group.

@v-ccolin
Copy link
Contributor

I sent an email to the content owner today.

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

@AbhishekMallick-MS
Copy link
Contributor

Hi @v-ccolin - awaiting PG review, will get back soon. Thanks!

AbhishekMallick-MS
AbhishekMallick-MS approved these changes Dec 18, 2024
View reviewed changes
Copy link
Contributor

@AbhishekMallick-MS AbhishekMallick-MS left a comment

Choose a reason for hiding this comment

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

Approved with PG confirmation.

@AbhishekMallick-MS
Copy link
Contributor

Hi @v-ccolin - please merge the PR. Thanks #sign-off

@v-dirichards v-dirichards merged commit e25e256 into MicrosoftDocs:main Dec 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AbhishekMallick-MS AbhishekMallick-MS AbhishekMallick-MS approved these changes

Assignees

@AbhishekMallick-MS AbhishekMallick-MS

Labels
aq-pr-triaged tracking label for the PR review team azure-backup/svc Change sent to author qualifies-for-auto-merge ready-to-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@azarboon @Court72 @AbhishekMallick-MS @v-ccolin @v-dirichards