Skip to content

[barracuda] Upgrade ECS to 8.0.0#2381

Closed
r00tu53r wants to merge 5 commits intoelastic:mainfrom
r00tu53r:feat/ecs-8-0-barracuda
Closed

[barracuda] Upgrade ECS to 8.0.0#2381
r00tu53r wants to merge 5 commits intoelastic:mainfrom
r00tu53r:feat/ecs-8-0-barracuda

Conversation

@r00tu53r
Copy link
Copy Markdown
Contributor

Automated PR.

Upgrades ECS to 8.0.0.

@r00tu53r r00tu53r added the enhancement New feature or request label Dec 24, 2021
@r00tu53r r00tu53r requested a review from adriansr December 24, 2021 05:55
@r00tu53r r00tu53r self-assigned this Dec 24, 2021
@elasticmachine
Copy link
Copy Markdown

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@r00tu53r r00tu53r requested review from a team and leehinman December 24, 2021 05:57
@r00tu53r r00tu53r mentioned this pull request Dec 24, 2021
@r00tu53r r00tu53r linked an issue Dec 24, 2021 that may be closed by this pull request
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Dec 24, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-01-24T16:19:58.116+0000

  • Duration: 23 min 17 sec

  • Commit: 64c8837

Test stats 🧪

Test Results
Failed 0
Passed 14
Skipped 0
Total 14

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@r00tu53r r00tu53r removed a link to an issue Dec 25, 2021
name: process.pid
- external: ecs
name: process.ppid
name: process.parent.pid
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Breaking change?

name: barracuda
title: Barracuda Logs
version: 0.7.1
version: 0.7.2
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is arguably 0.8.0 given the breaking change in parent PID mapping.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes! Scripting error. I'll fix this. Thanks for catching it :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

All rsa2elk integrations (including this one) will have another round of modifications after updates.

@adriansr
Copy link
Copy Markdown
Contributor

Replaced by #2574

@adriansr adriansr closed this Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants