Skip to content

Conversation

@sholzhauer-es
Copy link
Contributor

@sholzhauer-es sholzhauer-es commented Jul 15, 2025

Proposed commit message

Adding agentless support for Anomali integration

Checklist

Author's Checklist

  • ensure kibana ui behaves properly

How to test this PR locally

Need credentials for anomali to test.

Related issues

Screenshots

image

@sholzhauer-es sholzhauer-es added the enhancement New feature or request label Jul 15, 2025
@sholzhauer-es sholzhauer-es requested a review from a team as a code owner July 15, 2025 08:24
@narph
Copy link
Contributor

narph commented Jul 15, 2025

@sholzhauer-es , thanks for contributing to the integrations repo, we are slowly introducing integrations to agentless with phase 2 coming up next #14186. ti_anomali is not yet on the list as we need to do further testing and validate agentless is supported. cc @cpascale43

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Jul 15, 2025
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@botelastic
Copy link

botelastic bot commented Aug 14, 2025

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Aug 14, 2025
@mohitjha-elastic
Copy link
Collaborator

@sholzhauer-es From the Security Service team’s side, I have tested the ti_anomali agentless deployment support on ECH 8.18.0 and later versions, and it appears to be working as expected.
Testing on the Serverless environment and ECH 9.1.* has not yet been completed due to some issue.

Meanwhile, could you please look into resolving the build issue?

@botelastic botelastic bot removed the Stalled label Aug 19, 2025
@sholzhauer-es
Copy link
Contributor Author

@mohitjha-elastic perfect. I'll dive into the build failure and try and resolve it.

@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Aug 19, 2025

🚀 Benchmarks report

Package ti_anomali 👍(0) 💚(0) 💔(2)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
intelligence 9433.96 6134.97 -3298.99 (-34.97%) 💔
threatstream 8695.65 3861 -4834.65 (-55.6%) 💔

To see the full report comment with /test benchmark fullreport

@sholzhauer-es
Copy link
Contributor Author

@mohitjha-elastic the build and merge conflicts have been resolved.

@mohitjha-elastic
Copy link
Collaborator

@elastic/security-service-integrations Could someone from the team please review the PR?

sholzhauer-es and others added 3 commits August 21, 2025 08:46
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
@elastic-sonarqube
Copy link

Copy link
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

LGTM, but please wait for @alaudazzi for review of the doc change.

@sholzhauer-es
Copy link
Contributor Author

@alaudazzi have you had a chance to look at this?

@botelastic
Copy link

botelastic bot commented Oct 10, 2025

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Oct 10, 2025
@efd6 efd6 requested a review from alaudazzi October 14, 2025 22:41
@botelastic botelastic bot removed the Stalled label Oct 14, 2025
@botelastic
Copy link

botelastic bot commented Nov 13, 2025

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Nov 13, 2025
Copy link
Contributor

@alaudazzi alaudazzi left a comment

Choose a reason for hiding this comment

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

LGTM

@botelastic botelastic bot removed the Stalled label Nov 14, 2025
@mohitjha-elastic
Copy link
Collaborator

@sholzhauer-es Can you please resolve the conflicts to proceed with this PR?

@sholzhauer-es
Copy link
Contributor Author

sholzhauer-es commented Nov 20, 2025

@mohitjha-elastic and @elastic/security-service-integrations I fixed the conflict.

@elasticmachine
Copy link

💚 Build Succeeded

History

@sholzhauer-es
Copy link
Contributor Author

@mohitjha-elastic Am i good to merge?

@mohitjha-elastic mohitjha-elastic merged commit 911c74c into elastic:main Nov 25, 2025
8 checks passed
@mohitjha-elastic
Copy link
Collaborator

@sholzhauer-es I merged the PR. Thanks!

@elastic-vault-github-plugin-prod

Package ti_anomali - 2.2.0 containing this change is available at https://epr.elastic.co/package/ti_anomali/2.2.0/

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

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:ti_anomali Anomali Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants