Skip to content

Fix AMBA extended policy sync path handling#1311

Open
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-syncambaextendedpolicies
Open

Fix AMBA extended policy sync path handling#1311
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-syncambaextendedpolicies

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 24, 2026

Fixes #1309

AMBA extended policy sync was missing generated definitions on non-Windows runners because the file filters and path parsing assumed backslash-only paths. This update makes the extended-policy sync path handling cross-platform and adds coverage for syncing the secondary AMBA repository.

  • Cross-platform extended policy discovery

    • Updated AMBA extended-policy file matching to recognize both /policy/ and \policy\ path separators.
    • Updated subpath extraction to parse nested service paths on Windows and Linux consistently.
  • Regression coverage

    • Added a Pester test that runs Sync-ALZPolicyFromLibrary with -Type AMBA -SyncAMBAExtendedPolicies.
    • Verifies an extended policy from the AMBA repo is materialized into the expected EPAC policyDefinitions/AMBA/... location.

Example:

Sync-ALZPolicyFromLibrary `
  -DefinitionsRootFolder .\Definitions `
  -Type AMBA `
  -PacEnvironmentSelector "epac-dev" `
  -SyncAMBAExtendedPolicies

Copilot AI changed the title [WIP] Fix SyncAMBAExtendedPolicies not syncing extended definitions Fix AMBA extended policy sync path handling May 24, 2026
Copilot AI requested a review from anwather May 24, 2026 08:24
@anwather anwather marked this pull request as ready for review May 24, 2026 23:47
@anwather anwather enabled auto-merge (squash) May 24, 2026 23:48
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.

-SyncAMBAExtendedPolicies not syncing any extended definitions

2 participants