Commit 44ad298
Revert: ship automation rules as manual Deploy-to-Azure, not a packaged playbook
Automation rules are Microsoft.SecurityInsights/automationRules, not Logic
App playbooks (Microsoft.Logic/workflows). Packaging them as a Playbook
content type produced a broken 'Playbook template' in Content Hub that fails
to load (the portal tries to parse a Logic App trigger/connectors and can't).
Confirmed no solution in the repo packages automation rules (0 hits across
all Package/mainTemplate.json). The established pattern - e.g. Check Point
Cyberint Alerts - ships the automation-rules ARM template under Playbooks/
but keeps it OUT of the data file, delivering it via a one-click
Deploy to Azure button instead.
Changes:
- Remove the Playbooks entry from the solution data file so the automation
rules are no longer packaged as a (broken) playbook content template.
- Remove the now-unused PlaybookName parameter from azuredeploy.json.
- Keep the automation-rules ARM template + Deploy-to-Azure button under
Playbooks/MicrosoftADTierModelAutomationRules/ (Check Point pattern).
- Repackage: mainTemplate.json now cleanly ships 19 analytic rules + 1
workbook (0 automationRules). Version held at 3.0.0.
Verified in lab: Content Hub shows 19 analytic templates + 1 workbook, an
empty Playbook templates tab (no failed-to-load error), and the 5 automation
rules deploy correctly via the button. Full validation: ARM-TTK 48 passed,
15 passed / 0 failed.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 722d3473-56e9-4c4a-be7a-b8eccec306dc1 parent f72960c commit 44ad298
5 files changed
Lines changed: 381 additions & 827 deletions
File tree
- Solutions/Microsoft Active Directory Tier Model
- Data
- Package
- Playbooks/MicrosoftADTierModelAutomationRules
Lines changed: 0 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | 31 | | |
35 | 32 | | |
36 | 33 | | |
| |||
Binary file not shown.
Lines changed: 1 addition & 29 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
386 | 386 | | |
387 | 387 | | |
388 | 388 | | |
389 | | - | |
390 | | - | |
391 | | - | |
392 | | - | |
393 | | - | |
394 | | - | |
395 | | - | |
396 | | - | |
397 | | - | |
398 | | - | |
399 | | - | |
400 | | - | |
401 | | - | |
402 | | - | |
403 | | - | |
404 | | - | |
405 | | - | |
406 | | - | |
407 | | - | |
408 | | - | |
409 | | - | |
410 | | - | |
411 | | - | |
412 | | - | |
413 | | - | |
414 | | - | |
415 | | - | |
416 | | - | |
417 | 389 | | |
418 | 390 | | |
419 | 391 | | |
| |||
0 commit comments