Skip to content

Conversation

@pchila
Copy link
Member

@pchila pchila commented Nov 21, 2025

What does this PR do?

Removes upload of build/** from integration tests steps since that would cause uploading everything (including subdirectories) under build/ including reuploading large elastic-agent packages (already uploaded at the packaging steps)

It also introduces more "precise" artifacts download from the packaging steps leveraging platform and package formats.

Why is it important?

To reduce wasted space and time in CI builds

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?
  • ...

@mergify mergify bot assigned pchila Nov 21, 2025
@mergify
Copy link
Contributor

mergify bot commented Nov 21, 2025

This pull request does not have a backport label. Could you fix it @pchila? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-./d./d is the label that automatically backports to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 21, 2025

⏳ Build in-progress, with failures

Failed CI Steps

History

cc @pchila

@pchila pchila changed the title Avoid uploading unnecessary artifacts Avoid downloading/uploading unnecessary artifacts Nov 21, 2025
@pchila pchila changed the title Avoid downloading/uploading unnecessary artifacts Avoid downloading/uploading unnecessary artifacts in CI runs Nov 21, 2025
@pchila pchila added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team ci backport-active-all Automated backport with mergify to all the active branches labels Nov 21, 2025
@pchila pchila marked this pull request as ready for review November 21, 2025 14:58
@pchila pchila requested review from a team as code owners November 21, 2025 14:58
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

Copy link
Contributor

@swiatekm swiatekm left a comment

Choose a reason for hiding this comment

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

LGTM, spot checked some BK steps and the artifacts looked correct to me.

@pchila pchila merged commit 3cedabf into elastic:main Nov 21, 2025
27 checks passed
@github-actions
Copy link
Contributor

@Mergifyio backport 8.19 9.1 9.2

@mergify
Copy link
Contributor

mergify bot commented Nov 21, 2025

backport 8.19 9.1 9.2

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Nov 21, 2025
* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests

(cherry picked from commit 3cedabf)

# Conflicts:
#	.buildkite/bk.integration.pipeline.yml
mergify bot pushed a commit that referenced this pull request Nov 21, 2025
* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests

(cherry picked from commit 3cedabf)
mergify bot pushed a commit that referenced this pull request Nov 21, 2025
* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests

(cherry picked from commit 3cedabf)
pchila added a commit that referenced this pull request Nov 21, 2025
…#11347)

* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests

(cherry picked from commit 3cedabf)

Co-authored-by: Paolo Chilà <[email protected]>
pchila added a commit that referenced this pull request Nov 21, 2025
…#11346)

* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests

(cherry picked from commit 3cedabf)

Co-authored-by: Paolo Chilà <[email protected]>
pchila added a commit that referenced this pull request Nov 21, 2025
…acts in CI runs (#11345)

* Avoid downloading/uploading unnecessary artifacts in CI runs (#11327)

* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests

(cherry picked from commit 3cedabf)

# Conflicts:
#	.buildkite/bk.integration.pipeline.yml

* fix conflicts

---------

Co-authored-by: Paolo Chilà <[email protected]>
hayotbisonai pushed a commit to hayotbisonai/elastic-agent that referenced this pull request Nov 23, 2025
…#11327)

* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests
swiatekm pushed a commit that referenced this pull request Nov 24, 2025
* avoid uploading the whole build/ directory tree

* Download only required artifacts for integration tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-active-all Automated backport with mergify to all the active branches ci skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants