Skip to content

docs: Added file and images and updated links#197

Merged
Roopan-Microsoft merged 7 commits intomainfrom
psl-re-use-fileupdate
Jul 4, 2025
Merged

docs: Added file and images and updated links#197
Roopan-Microsoft merged 7 commits intomainfrom
psl-re-use-fileupdate

Conversation

@Thanusree-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

Updating the documents

Does this introduce a breaking change?

  • Yes
  • No

Other Information

Added use-log-analytic file
Added images
Updated and corrected links
Replaced "Workspace ID" to "Resource ID"

Copy link
Copy Markdown
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 new guide for reusing an existing Log Analytics Workspace and updates parameter documentation to reference that guide.

  • Introduce re-use-log-analytics.md with step-by-step instructions and screenshots.
  • Update CustomizingAzdParameters.md to link the new guide and clarify the Log Analytics parameter.
  • Replace “Workspace ID” phrasing and correct links.

Reviewed Changes

Copilot reviewed 2 out of 5 changed files in this pull request and generated 1 comment.

File Description
docs/re-use-log-analytics.md New guide on configuring an existing Log Analytics workspace.
docs/CustomizingAzdParameters.md Updated default value row to link to the reuse-log-analytics guide and fixed link paths.
Comments suppressed due to low confidence (3)

docs/re-use-log-analytics.md:19

  • [nitpick] Rephrase for clarity, e.g., "Copy the Resource ID; this serves as your Workspace ID."
Copy Resource ID that is your Workspace ID

docs/re-use-log-analytics.md:28

  • Ensure consistent casing for "ID" (use "") and remove the extra spaces before "obtained."
Replace `<Existing Log Analytics Workspace Id>` with the value   obtained from Step 3.

docs/CustomizingAzdParameters.md:21

  • The default value column now contains a link rather than a literal default; consider moving this link to the description column or specifying a default value, and use a relative path (e.g., ../docs/re-use-log-analytics.md).
| `AZURE_ENV_LOG_ANALYTICS_WORKSPACE_ID` | string  | Guide to get your [Existing Workspace ID](/docs/re-use-log-analytics.md) | Reuses an existing Log Analytics Workspace instead of provisioning a new one.         |

Go to https://portal.azure.com

### 2. Search for Log Analytics
In the search bar at the top, type "Log Analytics workspaces" and click on it and click on the workspace you want to use.
Copy link

Copilot AI Jun 25, 2025

Choose a reason for hiding this comment

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

[nitpick] The instruction repeats "click": simplify to "In the search bar, type 'Log Analytics workspaces', select it, then choose the workspace you want to use."

Suggested change
In the search bar at the top, type "Log Analytics workspaces" and click on it and click on the workspace you want to use.
In the search bar at the top, type "Log Analytics workspaces", select it, then choose the workspace you want to use.

Copilot uses AI. Check for mistakes.
Updated links and removed space
Added a collapsible section.
@Roopan-Microsoft Roopan-Microsoft merged commit 8702976 into main Jul 4, 2025
5 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-re-use-fileupdate branch July 4, 2025 12:37
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.8.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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants