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

Docs5156/azure logging updates #22654

Merged
merged 6 commits into from
May 13, 2024
Merged

Docs5156/azure logging updates #22654

merged 6 commits into from
May 13, 2024

Conversation

buraizu
Copy link
Contributor

@buraizu buraizu commented Apr 13, 2024

What does this PR do? What is the motivation?

  • Update manual setup and blob storage logging instructions
  • Add subsections for clearer organization
  • Add a relevant screenshot

Merge instructions

  • Please merge after reviewing

Additional notes

@buraizu buraizu requested a review from a team as a code owner April 13, 2024 00:06
@github-actions github-actions bot added Images Images are added/removed with this PR Guide Content impacting a guide labels Apr 13, 2024
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@buraizu buraizu added the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 14, 2024
@buraizu buraizu removed the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 25, 2024
Copy link
Contributor

@estherk15 estherk15 left a comment

Choose a reason for hiding this comment

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

Looks good!

content/en/logs/guide/azure-logging-guide.md Outdated Show resolved Hide resolved
@buraizu buraizu merged commit 8d44f1a into master May 13, 2024
13 checks passed
@buraizu buraizu deleted the docs5156/azure-logging-updates branch May 13, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Guide Content impacting a guide Images Images are added/removed with this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants