Skip to content

Gravityzone ASim parsers#13330

Open
gbarbieru wants to merge 102 commits intoAzure:masterfrom
bitdefender:gravityzone-asim-parsers
Open

Gravityzone ASim parsers#13330
gbarbieru wants to merge 102 commits intoAzure:masterfrom
bitdefender:gravityzone-asim-parsers

Conversation

@gbarbieru
Copy link
Copy Markdown
Contributor

Required items, please complete

Change(s):

  • Added support for Bitdefender GravityZone events in ASim parsers (ASim Alert Events)

Reason for Change(s):

Version Updated:

  • No

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

Before going into this topic I want to disclose that development in my team is done on Linux workstations and the available tooling and guides offered by Microsoft kinda lack in this department. Due to time constraints additional effort in making them work on Linux environments was abandoned and testing was eventually done on Microsoft Sentinel accounts via end-to-end testing.

  • KQL: Tested using smoke tests directly on a Microsoft Sentinel account. No issues so far. Failed to test locally.
  • YAML: Failed to test locally. We hope they can be tested by the github actions.

@gbarbieru gbarbieru requested review from a team as code owners December 16, 2025 13:18
@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Approval Required

This fork PR requires manual approval before automated testing can run.

For security, a maintainer must:

  1. 📝 Review the code changes carefully
  2. Verify file types - This PR should only contain .yml, .yaml, or .json files. Check for any executable scripts (.ps1, .py, .sh, .exe, etc.) which are not allowed in this context.
  3. 🏷️ Add the SafeToRun label if the changes are safe to execute

Note: If new commits are added later, simply remove and re-add the SafeToRun label.


🤖 Automated security check • Created: 2025-12-16T13:18:38.466Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Approval Required

This fork PR requires manual approval before automated testing can run.

For security, a maintainer must:

  1. 📝 Review the code changes carefully
  2. Verify file types - This PR should only contain .yml, .yaml, or .json files. Check for any executable scripts (.ps1, .py, .sh, .exe, etc.) which are not allowed in this context.
  3. 🏷️ Add the SafeToRun label if the changes are safe to execute

Note: If new commits are added later, simply remove and re-add the SafeToRun label.


🤖 Automated security check • Created: 2025-12-16T13:22:46.444Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Approval Required

This fork PR requires manual approval before automated testing can run.

For security, a maintainer must:

  1. 📝 Review the code changes carefully
  2. Verify file types - This PR should only contain .yml, .yaml, or .json files. Check for any executable scripts (.ps1, .py, .sh, .exe, etc.) which are not allowed in this context.
  3. 🏷️ Add the SafeToRun label if the changes are safe to execute

Note: If new commits are added later, simply remove and re-add the SafeToRun label.


🤖 Automated security check • Created: 2025-12-16T13:57:55.516Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Approval Required

This fork PR requires manual approval before automated testing can run.

For security, a maintainer must:

  1. 📝 Review the code changes carefully
  2. Verify file types - This PR should only contain .yml, .yaml, or .json files. Check for any executable scripts (.ps1, .py, .sh, .exe, etc.) which are not allowed in this context.
  3. 🏷️ Add the SafeToRun label if the changes are safe to execute

Note: If new commits are added later, simply remove and re-add the SafeToRun label.


🤖 Automated security check • Created: 2025-12-19T11:45:44.889Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@v-atulyadav v-atulyadav added SafeToRun This is used only for ASim parsers Fork PR Pipeline run. and removed SafeToRun This is used only for ASim parsers Fork PR Pipeline run. labels Dec 22, 2025
@v-atulyadav v-atulyadav added the SafeToRun This is used only for ASim parsers Fork PR Pipeline run. label Dec 29, 2025
@v-atulyadav v-atulyadav removed the SafeToRun This is used only for ASim parsers Fork PR Pipeline run. label Dec 29, 2025
@gbarbieru
Copy link
Copy Markdown
Contributor Author

hello. any updates on this one?
i see that the checks fail with "System.IO.IOException: No space left on device" which seem to be more a test / env error than 'my code' error

@v-atulyadav
Copy link
Copy Markdown
Collaborator

Hi @gbarbieru,
Could you please pull the latest changes from the master branch and push the updated branch again? Thanks

@gbarbieru
Copy link
Copy Markdown
Contributor Author

hi @v-atulyadav . done!

@v-atulyadav v-atulyadav added the SafeToRun This is used only for ASim parsers Fork PR Pipeline run. label Jan 15, 2026
@gbarbieru
Copy link
Copy Markdown
Contributor Author

hi @v-atulyadav !
there seems to be some problems with the checks that i'm not sure they have to do anything with my code

@rvirjoghe-bd
Copy link
Copy Markdown

The task "Run ASim Template Validation tests" seems stuck and fails

Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
@v-atulyadav
Copy link
Copy Markdown
Collaborator

Hi @gbarbieru,

Please review the above comments and act accordingly. Thanks

@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Re-approval Required

⚠️ New commits detected: This fork PR has been updated with new commits while the SafeToRun label was present.

For security, a maintainer must:

  1. 📝 Review the latest commits carefully for any security concerns
  2. Verify file types - Ensure new commits only contain .yml, .yaml, or .json files. Reject if any executable scripts (.ps1, .py, .sh, .exe, etc.) are included.
  3. 🏷️ Remove the SafeToRun label
  4. 🏷️ Re-add the SafeToRun label if the new commits are safe

This simple process ensures that all commits have been properly reviewed before testing with repository secrets.


🤖 Automated security check • Updated: 2026-04-27T08:07:36.572Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@gbarbieru
Copy link
Copy Markdown
Contributor Author

@v-atulyadav @yummyblabla hello! after implementing the suggested changes the filter seems to no longer work as expected:

image

the additionalFields filed no longer contains data, no mater the value of the 'pack' parameter.

i've also observed that not all entries contain the TimeGenerated value. is this normal ? (filtering by time seems to work correctly)

@microsoft-github-policy-service
Copy link
Copy Markdown

@gbarbieru the command you issued was incorrect. Please try again.

Examples are:

@ agree

and

@ agree company="your company"

Comment thread Parsers/ASimAlertEvent/Parsers/ASimAlertEventBitdefenderGravityZone.yaml Outdated
@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Re-approval Required

⚠️ New commits detected: This fork PR has been updated with new commits while the SafeToRun label was present.

For security, a maintainer must:

  1. 📝 Review the latest commits carefully for any security concerns
  2. Verify file types - Ensure new commits only contain .yml, .yaml, or .json files. Reject if any executable scripts (.ps1, .py, .sh, .exe, etc.) are included.
  3. 🏷️ Remove the SafeToRun label
  4. 🏷️ Re-add the SafeToRun label if the new commits are safe

This simple process ensures that all commits have been properly reviewed before testing with repository secrets.


🤖 Automated security check • Updated: 2026-04-28T08:03:46.382Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Re-approval Required

⚠️ New commits detected: This fork PR has been updated with new commits while the SafeToRun label was present.

For security, a maintainer must:

  1. 📝 Review the latest commits carefully for any security concerns
  2. Verify file types - Ensure new commits only contain .yml, .yaml, or .json files. Reject if any executable scripts (.ps1, .py, .sh, .exe, etc.) are included.
  3. 🏷️ Remove the SafeToRun label
  4. 🏷️ Re-add the SafeToRun label if the new commits are safe

This simple process ensures that all commits have been properly reviewed before testing with repository secrets.


🤖 Automated security check • Updated: 2026-04-28T09:57:59.280Z
Learn more: GitHub Security Lab - Preventing PWN Requests

@gbarbieru
Copy link
Copy Markdown
Contributor Author

fixed the pack & timegenerated issue
image

@github-actions
Copy link
Copy Markdown
Contributor

🔒 Security Re-approval Required

⚠️ New commits detected: This fork PR has been updated with new commits while the SafeToRun label was present.

For security, a maintainer must:

  1. 📝 Review the latest commits carefully for any security concerns
  2. Verify file types - Ensure new commits only contain .yml, .yaml, or .json files. Reject if any executable scripts (.ps1, .py, .sh, .exe, etc.) are included.
  3. 🏷️ Remove the SafeToRun label
  4. 🏷️ Re-add the SafeToRun label if the new commits are safe

This simple process ensures that all commits have been properly reviewed before testing with repository secrets.


🤖 Automated security check • Updated: 2026-04-28T12:08:21.920Z
Learn more: GitHub Security Lab - Preventing PWN Requests

let parser=(pack:bool=false){
union isfuzzy=true
vimAlertEventEmpty,
ASimAlertEventBitdefenderGravityZone (disabled=(ASimBuiltInDisabled or ('ExcludeASimAlertEventBitdefenderGravityZone' in (DisabledParsers)))),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please pass pack as a parameter here as well

{
union isfuzzy=true
vimAlertEventEmpty,
vimAlertEventBitdefenderGravityZone (starttime=starttime, endtime=endtime, ipaddr_has_any_prefix=ipaddr_has_any_prefix, hostname_has_any=hostname_has_any, username_has_any=username_has_any, attacktactics_has_any=attacktactics_has_any, attacktechniques_has_any=attacktechniques_has_any, threatcategory_has_any=threatcategory_has_any, alertverdict_has_any=alertverdict_has_any, eventseverity_has_any=eventseverity_has_any, disabled=(vimBuiltInDisabled or ('ExcludevimAlertBitdefenderGravityZone' in (DisabledParsers)))),
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please pass pack as a parameter here.

| where module == "new-incident"
| extend d = data
| extend
EventStartTime = start_time,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It looks like EventStartTime = start_time and EventEndTime is the same for all the tables and can be extended at the end.. Same for the vim parser.

@yummyblabla
Copy link
Copy Markdown
Collaborator

Afterwards, please run the script "DirectoryOfAzureSentinel\.script\kqlFuncYaml2Arm.ps1" in the root folder of the repo to have it generate the necessary ARM templates as "Convert kql function yaml to ARM template" can't run on forked repos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ASIM SafeToRun This is used only for ASim parsers Fork PR Pipeline run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants