docs: Added file, images and update the links #498
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Atulku-Microsoft
requested review from
Avijit-Microsoft,
Roopan-Microsoft,
Prajwal-Microsoft,
aniaroramsft,
brittneek and
Vinay-Microsoft
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR updates the documentation by adding a new guide for reusing an existing Log Analytics Workspace, updating images and links, and replacing the term “Workspace ID” with “Resource ID”.
- Added a new markdown file (re-use-log-analytics.md) with step-by-step instructions and images.
- Updated parameter descriptions in CustomizingAzdParameters.md to refer to the new guide and terminology.
Reviewed Changes
Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| documents/re-use-log-analytics.md | New file with detailed steps and images for using an existing Log Analytics Workspace. |
| documents/CustomizingAzdParameters.md | Updated parameter default value to link to the new documentation on Log Analytics Workspace. |
Comments suppressed due to low confidence (2)
documents/re-use-log-analytics.md:19
- The phrase 'Workspace ID' is still present, which may cause confusion given the updated terminology. Consider rephrasing to explicitly use 'Resource ID', for example: 'Copy the Resource ID for your Log Analytics Workspace.'
Copy Resource ID that is your Workspace ID
documents/CustomizingAzdParameters.md:24
- The default value currently provides an instructional message rather than a consistent placeholder as seen with other parameters. Consider using a placeholder (e.g., '') and include a reference to the guide for clarity.
| `AZURE_ENV_LOG_ANALYTICS_WORKSPACE_ID` | string | Guide to get your [Existing Workspace ID](/documents/re-use-log-analytics.md) | Reuses an existing Log Analytics Workspace instead of creating a new one. |
add new section
update the back link
removed extra space
Avijit-Microsoft
approved these changes
Jul 3, 2025
|
🎉 This PR is included in version 3.11.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
Does this introduce a breaking change?
Other Information
Added use-log-analytic file
Added images
Updated and corrected links
Replaced "Workspace ID" to "Resource ID"
Update DeploymentGuide
re-use-log-analytics