-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[7.17](backport #39261) Update filebeat pipeline to match Jenkins steps #39507
Conversation
* updated filebeat pipeline * updated pre-command * updated timeout for python tests * restored original timeout * pr fixes (cherry picked from commit a878b49)
/test |
This pull request has not been merged yet. Could you please review and merge it @oakrizan? 🙏 |
This pull request is now in conflicts. Could you fix it? 🙏
|
…nto mergify/bp/7.17/pr-39261
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
left two minor suggestions that don't warrant another review cycle
Proposed commit message
In scope of https://github.com/elastic/ingest-dev/issues/3254 - updated filebeat pipeline to match same steps & commands as in Jenkins.
Jenkins vs. Buildkite comparison: https://gist.github.com/oakrizan/c45f1554b3f7ffbe153e244a248ce07d
BK builds: https://buildkite.com/elastic/filebeat/builds?branch=elastic%3Amergify%2Fbp%2F7.17%2Fpr-39261
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs
This is an automatic backport of pull request #39261 done by [Mergify](https://mergify.com).