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

migrate libbeat pipeline #37725

Merged
merged 62 commits into from
Feb 7, 2024
Merged

Conversation

sharbuz
Copy link
Contributor

@sharbuz sharbuz commented Jan 24, 2024

What is the problem this PR solves?

Jenkins->Buildkite pipelines migration

How does this PR solve the problem?

  1. Migrate the libbeat pipeline
    https://buildkite.com/elastic/beats-libbeat/builds?branch=sharbuz%3Amigrate-libbeat-pipeline
  2. Update the common conditions/rules for starting the pipelines (excluded the part from the main pipeline)

Examples of the builds:
Metricbeat: https://buildkite.com/elastic/beats-metricbeat/builds/1992
Libbeat: https://buildkite.com/elastic/beats-libbeat/builds/700

Related issues

https://github.com/elastic/ingest-dev/issues/1693

@sharbuz sharbuz added enhancement Team:Elastic-Agent Label for the Agent team backport-7.17 Automated backport to the 7.17 branch with mergify backport-v8.12.0 Automated backport with mergify labels Jan 24, 2024
@sharbuz sharbuz requested review from v1v, mrodm and jlind23 January 24, 2024 15:28
@sharbuz sharbuz self-assigned this Jan 24, 2024
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jan 24, 2024
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
.buildkite/libbeat/pipeline.libbeat.yml Outdated Show resolved Hide resolved
@sharbuz sharbuz removed request for mrodm and jlind23 January 24, 2024 19:31
Siarhei Harbuz added 3 commits January 25, 2024 16:26
@sharbuz sharbuz removed the Team:Elastic-Agent Label for the Agent team label Jan 25, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 25, 2024
@botelastic
Copy link

botelastic bot commented Jan 25, 2024

This pull request doesn't have a Team:<team> label.

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
@sharbuz sharbuz enabled auto-merge (squash) February 6, 2024 10:32
@sharbuz
Copy link
Contributor Author

sharbuz commented Feb 6, 2024

The pipeline will finish with errors because of:
#37873
#37870
@v1v suggested using different PRs for that.
All other remarks have been fixed.
CC @dliappis

@sharbuz sharbuz disabled auto-merge February 6, 2024 10:53
@sharbuz sharbuz enabled auto-merge (squash) February 6, 2024 12:19
Copy link
Contributor

mergify bot commented Feb 6, 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 migrate-libbeat-pipeline upstream/migrate-libbeat-pipeline
git merge upstream/main
git push upstream migrate-libbeat-pipeline

@sharbuz sharbuz requested review from jlind23 and a team February 6, 2024 16:24
@sharbuz
Copy link
Contributor Author

sharbuz commented Feb 7, 2024

/test

@elasticmachine
Copy link
Collaborator

💔 Build Failed

Failed CI Steps

History

cc @sharbuz

@sharbuz
Copy link
Contributor Author

sharbuz commented Feb 7, 2024

/test

1 similar comment
@sharbuz
Copy link
Contributor Author

sharbuz commented Feb 7, 2024

/test

@jlind23 jlind23 disabled auto-merge February 7, 2024 11:25
@jlind23 jlind23 merged commit 625ba40 into elastic:main Feb 7, 2024
29 of 33 checks passed
@jlind23
Copy link
Collaborator

jlind23 commented Feb 7, 2024

Force merging this as the failure is unrelated. @sharbuz will create a follow up issue for @axw and @bturquet to investigate.

mergify bot pushed a commit that referenced this pull request Feb 7, 2024
(cherry picked from commit 625ba40)
mergify bot pushed a commit that referenced this pull request Feb 7, 2024
(cherry picked from commit 625ba40)

# Conflicts:
#	catalog-info.yaml
sharbuz added a commit that referenced this pull request Feb 7, 2024
(cherry picked from commit 625ba40)

Co-authored-by: sharbuz <87968844+sharbuz@users.noreply.github.com>
sharbuz added a commit that referenced this pull request Feb 7, 2024
* migrate libbeat pipeline (#37725)

(cherry picked from commit 625ba40)

# Conflicts:
#	catalog-info.yaml

* Update catalog-info.yaml

* Update catalog-info.yaml

---------

Co-authored-by: sharbuz <87968844+sharbuz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arm Enable builds in the CI for ARM testing backport-7.17 Automated backport to the 7.17 branch with mergify backport-v8.12.0 Automated backport with mergify enhancement libbeat Metricbeat Metricbeat needs_team Indicates that the issue/PR needs a Team:* label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants