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.

Sign up for GitHub

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

Jump to bottom

migrate winlogbeat pipeline #37926

Merged
merged 6 commits into from
Feb 9, 2024
Merged

migrate winlogbeat pipeline #37926

merged 6 commits into from
Feb 9, 2024

Conversation

sharbuz
Copy link
Contributor

@sharbuz sharbuz commented Feb 8, 2024 •
edited
Loading

What is the problem this PR solves?

Jenkins->Buildkite pipelines migration

How does this PR solve the problem?

Migrates the winlogbeat pipeline

Example of the pipeline:
https://buildkite.com/elastic/beats-winlogbeat/builds/356

Examples of dependencies:
https://buildkite.com/elastic/beats-libbeat/builds/1192
https://buildkite.com/elastic/beats-metricbeat/builds/2405
https://buildkite.com/elastic/beats-packetbeat/builds/1187

Related issues

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

@sharbuz sharbuz added Winlogbeat libbeat Metricbeat Metricbeat backport-7.17 Automated backport to the 7.17 branch with mergify backport-v8.12.0 Automated backport with mergify labels Feb 8, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 8, 2024
@botelastic
Copy link

botelastic bot commented Feb 8, 2024

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

@mergify mergify bot assigned sharbuz Feb 8, 2024
@elasticmachine
Copy link
Collaborator

elasticmachine commented Feb 8, 2024 •
edited
Loading

💚 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-02-09T14:12:27.886+0000

  • Duration: 33 min 57 sec

Test stats 🧪

Test Results
Failed 0
Passed 6344
Skipped 32
Total 6376

💚 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!)

@mergify Mergify
Copy link
Contributor

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

@sharbuz
Copy link
Contributor Author

sharbuz commented Feb 9, 2024

/test winlogbeat

@sharbuz sharbuz marked this pull request as ready for review February 9, 2024 11:44
@sharbuz sharbuz requested a review from a team as a code owner February 9, 2024 11:44
v1v
v1v requested changes Feb 9, 2024
View reviewed changes
.buildkite/scripts/common.sh Outdated Show resolved Hide resolved
.buildkite/winlogbeat/pipeline.winlogbeat.yml Outdated Show resolved Hide resolved
@sharbuz sharbuz requested a review from v1v February 9, 2024 12:05
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @sharbuz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @sharbuz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @sharbuz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @sharbuz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @sharbuz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @sharbuz

dliappis
dliappis approved these changes Feb 9, 2024
View reviewed changes
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@sharbuz sharbuz merged commit 4c75049 into elastic:main Feb 9, 2024
38 checks passed
mergify bot pushed a commit that referenced this pull request Feb 9, 2024
@sharbuz @mergify
migrate winlogbeat pipeline (#37926)
10ce4b2 
* migrate winlogbeat pipeline

* fix VAR name

* fix remarks

* update win_unit_tests.ps1

* update win_unit_tests.ps1

(cherry picked from commit 4c75049)
mergify bot pushed a commit that referenced this pull request Feb 9, 2024
@sharbuz @mergify
migrate winlogbeat pipeline (#37926)
819a6f1 
* migrate winlogbeat pipeline

* fix VAR name

* fix remarks

* update win_unit_tests.ps1

* update win_unit_tests.ps1

(cherry picked from commit 4c75049)
This was referenced Feb 9, 2024
Merged
Merged
sharbuz added a commit that referenced this pull request Feb 9, 2024
@mergify @sharbuz
migrate winlogbeat pipeline (#37926) (#37956)
97682f1 
* migrate winlogbeat pipeline

* fix VAR name

* fix remarks

* update win_unit_tests.ps1

* update win_unit_tests.ps1

(cherry picked from commit 4c75049)

Co-authored-by: sharbuz <[email protected]>
sharbuz added a commit that referenced this pull request Feb 9, 2024
@mergify @sharbuz
migrate winlogbeat pipeline (#37926) (#37957)
6dabb90 
* migrate winlogbeat pipeline

* fix VAR name

* fix remarks

* update win_unit_tests.ps1

* update win_unit_tests.ps1

(cherry picked from commit 4c75049)

Co-authored-by: sharbuz <[email protected]>
@sharbuz sharbuz deleted the migrate-winlogbeat branch February 9, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dliappis dliappis dliappis approved these changes

@v1v v1v v1v approved these changes

@pazone pazone Awaiting requested review from pazone

Assignees

@sharbuz sharbuz

Labels
backport-7.17 Automated backport to the 7.17 branch with mergify backport-v8.12.0 Automated backport with mergify libbeat Metricbeat Metricbeat needs_team Indicates that the issue/PR needs a Team:* label Winlogbeat
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sharbuz @elasticmachine @dliappis @v1v