-
Notifications
You must be signed in to change notification settings - Fork 392
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mandiant Advantage Integration initial implementation #8390
Conversation
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
packages/ti_mandiant_advantage/data_stream/threat_intelligence/fields/agent.yml
Show resolved
Hide resolved
/test |
Currently the build is failing because there's no entry in codeowners. Once that gets added, we should get a 'real' run of CI |
…. Also add package to CODEOWNERS.
Thanks for the review! I believe I have addressed your comments and added the CODEOWNERS entry. This should be ready for another review. |
/test |
🌐 Coverage report
|
...mandiant_advantage/data_stream/threat_intelligence/elasticsearch/ingest_pipeline/default.yml
Outdated
Show resolved
Hide resolved
packages/ti_mandiant_advantage/data_stream/threat_intelligence/agent/stream/httpjson.yml.hbs
Show resolved
Hide resolved
...mandiant_advantage/data_stream/threat_intelligence/elasticsearch/ingest_pipeline/default.yml
Show resolved
Hide resolved
@miguelhdezsob CI is looking good. Just a couple final comments and we should be good to go :) |
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Package ti_mandiant_advantage - 0.1.0 containing this change is available at https://epr.elastic.co/search?package=ti_mandiant_advantage |
Summary of PR
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Pull the code locally and run
elastic-package test
. You might also want to build and manually install the integration if needed.Related issues
Screenshots