Skip to content

ci: use if_changed #11414

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
Merged
merged 4 commits into from
Nov 28, 2025
Merged

ci: use if_changed #11414

merged 4 commits into from
Nov 28, 2025

Conversation

@v1v
Copy link
Member

@v1v v1v commented Nov 25, 2025 •
edited
Loading

What does this PR do?

remove dependency on the BK plugin monorepo-diff

Why is it important?

Faster builds
Remove the intermediate step to load the monorepo diff step

Checklist

  • I have read and understood the pull request guidelines of this project.
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

How to test this PR locally

Related issues

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

@v1v
ci: use if_changed
b766096 
remove dependency with the BK plugin monorepo-diff
@v1v v1v self-assigned this Nov 25, 2025
@v1v v1v requested a review from a team as a code owner November 25, 2025 14:46
@v1v v1v added the backport-8.19 Automated backport to the 8.19 branch label Nov 25, 2025
@v1v v1v requested a review from a team as a code owner November 25, 2025 14:46
@v1v v1v added the backport-9.1 Automated backport to the 9.1 branch label Nov 25, 2025
@v1v v1v added backport-9.2 Automated backport to the 9.2 branch skip-changelog labels Nov 25, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 25, 2025 •
edited
Loading

💛 Build succeeded, but was flaky

Failed CI Steps

History

cc @v1v

fr4nc1sc0-r4m0n
fr4nc1sc0-r4m0n approved these changes Nov 26, 2025
View reviewed changes
Copy link
Contributor

@fr4nc1sc0-r4m0n fr4nc1sc0-r4m0n left a comment

Choose a reason for hiding this comment

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

LGTM

@v1v v1v merged commit ed59055 into elastic:main Nov 28, 2025
20 of 21 checks passed
mergify bot pushed a commit that referenced this pull request Nov 28, 2025
@v1v @mergify
ci: use if_changed (#11414)
b14e0de 
(cherry picked from commit ed59055)

# Conflicts:
#	.buildkite/pipeline.yml
@mergify mergify bot mentioned this pull request Nov 28, 2025
Merged
8 tasks
mergify bot pushed a commit that referenced this pull request Nov 28, 2025
@v1v @mergify
ci: use if_changed (#11414)
291cc46 
(cherry picked from commit ed59055)
mergify bot pushed a commit that referenced this pull request Nov 28, 2025
@v1v @mergify
ci: use if_changed (#11414)
0d71203 
(cherry picked from commit ed59055)

# Conflicts:
#	.buildkite/pipeline.yml
This was referenced Nov 28, 2025
Closed
Merged
@v1v v1v removed the backport-9.1 Automated backport to the 9.1 branch label Nov 28, 2025
v1v pushed a commit that referenced this pull request Nov 28, 2025
@mergify
ci: use if_changed (#11414) (#11480)
8ec8819
v1v pushed a commit that referenced this pull request Nov 28, 2025
@mergify
[8.19] (backport #11414) ci: use if_changed (#11478)
20c44ac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fr4nc1sc0-r4m0n fr4nc1sc0-r4m0n fr4nc1sc0-r4m0n approved these changes

@blakerouse blakerouse Awaiting requested review from blakerouse blakerouse is a code owner automatically assigned from elastic/elastic-agent-control-plane

@michel-laterman michel-laterman Awaiting requested review from michel-laterman michel-laterman is a code owner automatically assigned from elastic/elastic-agent-control-plane

Assignees

@v1v v1v

Labels

backport-8.19 Automated backport to the 8.19 branch backport-9.2 Automated backport to the 9.2 branch skip-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@v1v @elasticmachine @fr4nc1sc0-r4m0n