Add Check Point Cyberint Alerts bi-directional sync playbooks (v3.1.0)#13790
Add Check Point Cyberint Alerts bi-directional sync playbooks (v3.1.0)#13790klevitskiy wants to merge 39 commits intoAzure:masterfrom
Conversation
|
@microsoft-github-policy-service agree company="Check Point" |
…/cyberint-ioc-connector # Conflicts: # Solutions/Check Point Cyberint Alerts/Analytic Rules/CPEMAlertIngestionAnomaly.yaml # Solutions/Check Point Cyberint Alerts/Package/createUiDefinition.json # Solutions/Check Point Cyberint Alerts/Package/mainTemplate.json # Solutions/Check Point Cyberint Alerts/Parsers/CPEMAlerts.yaml # Solutions/Check Point Cyberint Alerts/Playbooks/Sync/CPEM_InboundSync/azuredeploy.json # Solutions/Check Point Cyberint Alerts/Playbooks/Sync/CPEM_ManualStatusUpdate/azuredeploy.json # Solutions/Check Point Cyberint Alerts/Playbooks/Sync/CPEM_ManualStatusUpdate/readme.md # Solutions/Check Point Cyberint Alerts/Playbooks/Sync/CPEM_OutboundSync/azuredeploy.json # Solutions/Check Point Cyberint Alerts/Playbooks/Sync/CPEM_OutboundSync/readme.md # Solutions/Check Point Cyberint IOC/Data Connectors/CyberintArgosIOCLogs_ccp/CyberintArgosIOCLogs_PollingConfig.json # Solutions/Check Point Cyberint IOC/Package/mainTemplate.json
|
@v-maheshbh Hi, thank you for your assistance! All checks have passed, could you please review the PR when you get a chance? |
|
@v-maheshbh Hi, sorry to bug you again. Thanks for your help so far! All checks have passed - could you take a look at the PR when you have a moment? |
|
@v-maheshbh Hi, thank you for the recommendation. I made the single PR for review instead of two. Could you please review the PR when you get a chance? |
|
Hi @klevitskiy Kindly resolve the branch conflicts. Thanks! |
|
Hi @klevitskiy Kindly add both black‑and‑white preview images inside the workbook preview images folder. Thanks! |
…ture/cyberint-clean # Conflicts: # .script/tests/detectionTemplateSchemaValidation/ValidConnectorIds.json
@v-maheshbh Hi, I've added required screenshots. Let me know if anything else need to be added there
|
Hi @klevitskiy kindly refer below solution for more details: Thanks! |
@v-maheshbh Hi, Thank you for your patience and endless support! I've updated the workbook preview screenshots. To be honest, I'm not quite following your comment, the link shows 2 screenshots (black and white appearances), which matches what I attached. This PR has been open for a month now. I'd really appreciate it if we could wrap it up soon. Thanks again! |
|
Hi @klevitskiy
Thanks! |
@v-maheshbh Hi, you are looking at the wrong folder
|
|
Hi @klevitskiy
Create this folder here and add images. hope you get my point. or give me branch access. Thanks! |
@v-maheshbh Hi, thank you again for your patience and the helpful hint. I've created new folders and uploaded the screenshots
|
|
Hi @klevitskiy Kindly grant me branch access to resolve the validation error. Thanks! |
|
@v-maheshbh Hi, I've granted you access to the branch. Let me know if you have any issues |
Remove leftover merge-conflict markers and duplicate block in .script/tests/detectionTemplateSchemaValidation/ValidConnectorIds.json, normalize the JSON array formatting, and append missing connectors (CheckPointCyberintAlerts, CheckPointCyberintIOC). Ensures the file is a valid, de-duplicated list of connector IDs.
|
@v-maheshbh Thanks for the help! What's left to get this PR ready for merge? |
|
Hi @klevitskiy The data file includes a total of 9 playbooks; however, one playbook is not reflected in the mainTemplate.
Thanks! |
@v-maheshbh Hi, fixed it! Hopefully it looks correct now |
Kindly repackage the solution so that the solution name changes are correctly reflected in the mainTemplate.json. |
Done! ✅ |
Thanks! |
|
@v-maheshbh Hi, thanks for pointing me on that! Fixed |











Change(s):
Reason for Change(s):
Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: