Skip to content
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

[8.14](backport #39087) Migrate packetbeat to static Buildkite pipeline #39106

Closed
wants to merge 22 commits into from

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Apr 22, 2024

Proposed commit message

This commit refactors the packetbeat pipeline to the pipeline selector
approach as laid out in #38783.

Related issues

Screenshots

Logs

tetianakravchenko and others added 18 commits April 18, 2024 10:19
…torage-blob-go dependencies (#38861)

* filebeat/azure/azureeventhub: upgrade azure azure-event-hubs-go and azure-storage-blob-go dependencies

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>

* add the changelog record

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>

---------

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
Migrate x-pack filebeat to single pipeline

Signed-off-by: Alexandros Sapranidis <alexandros@elastic.co>
Use mono repo plugin for xpack/heartbeat

Relates: elastic/ingest-dev#3072

---------

Co-authored-by: Dimitrios Liappis <dimitrios.liappis@gmail.com>
- Syslog reader and processor are owned by elastic/sec-deployment-and-devices
* fixed issues in heartbeat pipeline

* pr fixes
Co-authored-by: Pierre HILBERT <pierre.hilbert@elastic.co>
- In 8.13, the netflow was accidentally removed from the list of inputs
for Windows. This restores netflow as an input under Windows.
Made with ❤️️ by updatecli

Co-authored-by: apmmachine <apmmachine@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* updated common.sh

* updated cnetral pipleine with winlogbeat

* fixed typo
PR#38913 had a bug for the extended tests group:
it wouldn't allow to run steps in the extended
group when not using a PR (e.g. merge commits).

This commit allows all steps in the extended
group to run when the trigger is not a PR.
In preparation for the migration to a static pipeline for packetbeat,
this commit removes the Buildkite PR Bot trigger.

Relates elastic/ingest-dev#3072
* migrate x-pack/libbeat to static

* Update pull-requests.json

* change paths

* Update pull-requests.json

* change conditions for group-steps

* change the pipeline.yml

* work with remarks

* work with remarks

* remove libbeat and x-pack/libbeat changesets from the common.sh

* remove libbeat and x-pack/libbeat changesets from the common.sh

* fix merge issues

* Update .buildkite/x-pack/pipeline.xpack.libbeat.yml

Co-authored-by: Dimitrios Liappis <dimitrios.liappis@gmail.com>

* apply common.sh from main and cleanup it

* change paths

---------

Co-authored-by: Dimitrios Liappis <dimitrios.liappis@gmail.com>
* updated common.sh

* fixed typo

* updated common.sh

* added changeset ps1 script and updated xpack-winlogbeat pipeline

* pr fixes

* handle backslashes in changeset directories
This commit refactors the packetbeat pipeline to the pipeline selector
approach as laid out in #38783.

Relates elastic/ingest-dev#3072
This commit refactors the packetbeat pipeline to the pipeline selector
approach as laid out in #38783.

Relates https://github.com/elastic/ingest-dev/issues/3072

(cherry picked from commit 8f8f313)

# Conflicts:
#	.buildkite/pipeline.yml
#	.buildkite/scripts/common.sh
@mergify mergify bot added backport conflicts There is a conflict in the backported pull request labels Apr 22, 2024
@mergify mergify bot requested a review from a team as a code owner April 22, 2024 09:42
Copy link
Contributor Author

mergify bot commented Apr 22, 2024

Cherry-pick of 8f8f313 has failed:

On branch mergify/bp/8.14/pr-39087
Your branch is up to date with 'origin/8.14'.

You are currently cherry-picking commit 8f8f3137f2.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   .buildkite/hooks/pre-command
	modified:   .buildkite/packetbeat/pipeline.packetbeat.yml
	deleted:    .buildkite/scripts/generate_packetbeat_pipeline.sh

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   .buildkite/pipeline.yml
	both modified:   .buildkite/scripts/common.sh

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 22, 2024
@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 22, 2024

💚 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: 2024-04-22T11:07:04.862+0000

  • Duration: 13 min 49 sec

Test stats 🧪

Test Results
Failed 0
Passed 3
Skipped 0
Total 3

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Copy link
Contributor Author

mergify bot commented Apr 22, 2024

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b mergify/bp/8.14/pr-39087 upstream/mergify/bp/8.14/pr-39087
git merge upstream/8.14
git push upstream mergify/bp/8.14/pr-39087

@mergify mergify bot requested review from a team as code owners April 22, 2024 10:57
@mergify mergify bot removed the request for review from a team April 22, 2024 10:57
@mergify mergify bot requested review from rdner and faec April 22, 2024 10:57
@dliappis dliappis removed conflicts There is a conflict in the backported pull request needs_team Indicates that the issue/PR needs a Team:* label labels Apr 22, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 22, 2024
@dliappis dliappis added Team:Ingest-EngProd and removed needs_team Indicates that the issue/PR needs a Team:* label labels Apr 22, 2024
@dliappis
Copy link
Contributor

closing, in favor of #39108

@dliappis dliappis closed this Apr 22, 2024
@mergify mergify bot deleted the mergify/bp/8.14/pr-39087 branch April 22, 2024 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants