docs(enterprise): document Windows admin-tier lifecycle script path - #2651
docs(enterprise): document Windows admin-tier lifecycle script path#2651HughChaw (Hughhhhcoder) wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Enterprise “Lifecycle Scripts” documentation to include the Windows admin-tier policy directory location, aligning the docs with the platform-specific behavior of lifecycle script discovery.
Changes:
- Documented the Windows admin-tier policy path (
C:\ProgramData\APM\policy.d\*.json) alongside the existing Linux/macOS path. - Updated the admin-tier path references across the tier overview, file format section, discovery notes, trust model, and analytics example.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
@microsoft-github-policy-service agree |
f8a053c to
ff898ba
Compare
|
Updated the branch onto the latest main and resolved the documentation conflict while preserving the current lifecycle-script trust and kill-switch wording. Validation:
|
|
The latest pull-request workflow runs are completed with |
TL;DR
The Lifecycle Scripts doc only referenced the Linux admin-tier policy directory (
/etc/apm/policy.d/*.json). This PR adds the Windows path (C:\ProgramData\APM\policy.d\*.json) so Windows fleet admins can find where admin-tier lifecycle scripts live.Closes #2621.
Problem
_get_policy_scripts_dir()insrc/apm_cli/core/lifecycle_scripts.pyresolves the admin-tier policy directory per-platform:But the docs page only documented the POSIX path, leaving Windows administrators without the correct location.
Approach
Updated every reference to the admin-tier path in
docs/src/content/docs/enterprise/lifecycle-scripts.mdto mention both platforms:Validation
npm ci && npm run buildindocs/— 123 pages built, all internal links valid:No code changes; no tests affected.