[CI] Fix Core DRA crossbuild by pazone · Pull Request #7203 · elastic/elastic-agent

pazone · 2025-03-05T12:48:04Z

What does this PR do?

Fixes Core DRA snapshot build by splittion the build step into separate steps

x86_64: Elastic-Agent Core Snapshot produces:

elastic-agent-core-9.1.0-SNAPSHOT-linux-x86_64.tar.gz
elastic-agent-core-9.1.0-SNAPSHOT-darwin-aarch64.tar.gz
elastic-agent-core-9.1.0-SNAPSHOT-darwin-x86_64.tar.gz
elastic-agent-core-9.1.0-SNAPSHOT-windows-x86_64.zip

ARM: Elastic-Agent Core Snapshot produces:

elastic-agent-core-9.1.0-SNAPSHOT-linux-arm64.tar.gz + sha + csv

Build example

Why is it important?

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 · 2025-03-05T12:48:44Z

This pull request does not have a backport label. Could you fix it @pazone? 🙏
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.

.buildkite/pipeline.elastic-agent-binary-dra.yml

Co-authored-by: Paolo Chilà <paolo.chila@elastic.co>

dliappis · 2025-03-05T17:38:39Z

Last build when setting the right env vars:

DRA_BRANCH="main"
DRA_WORKFLOW="snapshot"
DRY_RUN="true"
RUN_SNAPSHOT="true"

seems to have passed: https://buildkite.com/elastic/elastic-agent-binary-dra/builds/2528#01956753-433a-4102-a15b-80508f139113

dliappis

LGTM based on #7203 (comment)

but I'd like to see @pchila 's or @pkoutsovasilis 's review too, in terms of whether the PLATFORMS vars are correct and whether we also need PACKAGE env vars.

.buildkite/pipeline.elastic-agent-binary-dra.yml

dliappis

LGTM thanks for iterating.

I compared the name and number of artifacts published using this PR vs the last successful build from main (https://buildkite.com/elastic/elastic-agent-binary-dra/builds/2506#0195652a-9f95-43b3-98b2-31752e9cf837) and they look identical.

elastic-sonarqube · 2025-03-06T12:42:05Z

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

elasticmachine · 2025-03-06T12:42:30Z

💛 Build succeeded, but was flaky

Buildkite Build
Commit: 2478c0e

Failed CI Steps

Unit tests - Windows 2016

History

💚 Build #18126 succeeded 7523037

cc @pazone

* [CI] Fix DRA crossbuild * Added staging fix * Fixed mistype * Fixed darwin build * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * replaced commas with spaces --------- Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> (cherry picked from commit d0548da)

* [CI] Fix DRA crossbuild * Added staging fix * Fixed mistype * Fixed darwin build * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * replaced commas with spaces --------- Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> (cherry picked from commit d0548da) # Conflicts: # .buildkite/pipeline.elastic-agent-binary-dra.yml

* [CI] Fix DRA crossbuild * Added staging fix * Fixed mistype * Fixed darwin build * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * replaced commas with spaces --------- Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> (cherry picked from commit d0548da)

* upstream/main: feat: bump go to 1.24.0 (elastic#6932) [CI] Fix Core DRA crossbuild (elastic#7203) fix(ci): Use correct `READ_ONLY` access value instead of `READ` (elastic#7239) ci(buildkite): add initial elastic-agent-helm-charts pipeline (elastic#7176) chore: update elastic-agent-libs to v0.18.8 (elastic#6938) [flakiness] Correct group for TestEndpointAgentServiceMonitoring (elastic#7165)

* [CI] Fix DRA crossbuild * Added staging fix * Fixed mistype * Fixed darwin build * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * replaced commas with spaces --------- Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> (cherry picked from commit d0548da) Co-authored-by: Pavel Zorin <pavel.zorin@elastic.co>

* [CI] Fix Core DRA crossbuild (#7203) * [CI] Fix DRA crossbuild * Added staging fix * Fixed mistype * Fixed darwin build * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * replaced commas with spaces --------- Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> (cherry picked from commit d0548da) # Conflicts: # .buildkite/pipeline.elastic-agent-binary-dra.yml * Resolved conflicts --------- Co-authored-by: Pavel Zorin <pavel.zorin@elastic.co>

* [CI] Fix DRA crossbuild * Added staging fix * Fixed mistype * Fixed darwin build * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * Update .buildkite/pipeline.elastic-agent-binary-dra.yml Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> * replaced commas with spaces --------- Co-authored-by: Paolo Chilà <paolo.chila@elastic.co> (cherry picked from commit d0548da) Co-authored-by: Pavel Zorin <pavel.zorin@elastic.co>

[CI] Fix DRA crossbuild

af34df1

mergify bot assigned pazone Mar 5, 2025

pazone added skip-changelog backport-active-all Automated backport with mergify to all the active branches Team:Ingest-EngProd labels Mar 5, 2025

pazone marked this pull request as ready for review March 5, 2025 13:03

pazone requested a review from a team as a code owner March 5, 2025 13:03

pazone requested review from dliappis and pchila March 5, 2025 13:03

Pavel Zorin added 2 commits March 5, 2025 14:34

Added staging fix

f8ce58d

Fixed mistype

be507ea

pazone marked this pull request as draft March 5, 2025 14:04

Fixed darwin build

49fb414

pazone marked this pull request as ready for review March 5, 2025 14:41

pchila reviewed Mar 5, 2025

View reviewed changes

.buildkite/pipeline.elastic-agent-binary-dra.yml Outdated Show resolved Hide resolved

.buildkite/pipeline.elastic-agent-binary-dra.yml Outdated Show resolved Hide resolved

Pavel Zorin and others added 2 commits March 5, 2025 15:50

Update .buildkite/pipeline.elastic-agent-binary-dra.yml

b6998c2

Co-authored-by: Paolo Chilà <paolo.chila@elastic.co>

Update .buildkite/pipeline.elastic-agent-binary-dra.yml

7523037

Co-authored-by: Paolo Chilà <paolo.chila@elastic.co>

pazone mentioned this pull request Mar 5, 2025

[CI] Fix unified release ARM packaging #7205

Merged

8 tasks

dliappis previously approved these changes Mar 5, 2025

View reviewed changes

v1v reviewed Mar 6, 2025

View reviewed changes

.buildkite/pipeline.elastic-agent-binary-dra.yml Outdated Show resolved Hide resolved

.buildkite/pipeline.elastic-agent-binary-dra.yml Outdated Show resolved Hide resolved

pazone requested a review from pkoutsovasilis March 6, 2025 09:58

Pavel Zorin added 2 commits March 6, 2025 12:03

replaced commas with spaces

24b88c9

Merged

2478c0e

pazone dismissed dliappis’s stale review via 2478c0e March 6, 2025 11:04

v1v approved these changes Mar 6, 2025

View reviewed changes

dliappis self-requested a review March 6, 2025 12:08

dliappis approved these changes Mar 6, 2025

View reviewed changes

pazone changed the title ~~[CI] Fix DRA crossbuild~~ [CI] Fix Core DRA crossbuild Mar 6, 2025

pazone requested a review from pchila March 6, 2025 12:16

pazone merged commit d0548da into main Mar 6, 2025
10 checks passed

pazone deleted the ci_fix_dra_crossbuild branch March 6, 2025 14:36

mergify bot mentioned this pull request Mar 6, 2025

[9.0](backport #7203) [CI] Fix Core DRA crossbuild #7242

Merged

8 tasks

mergify bot mentioned this pull request Mar 6, 2025

[8.18](backport #7203) [CI] Fix Core DRA crossbuild #7243

Merged

8 tasks

mergify bot mentioned this pull request Mar 6, 2025

[8.17](backport #7203) [CI] Fix Core DRA crossbuild #7244

Merged

8 tasks

mergify bot mentioned this pull request Mar 6, 2025

[8.16](backport #7203) [CI] Fix Core DRA crossbuild #7245

Merged

8 tasks

mergify bot mentioned this pull request Mar 6, 2025

[8.x](backport #7203) [CI] Fix Core DRA crossbuild #7246

Merged

8 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[CI] Fix Core DRA crossbuild#7203

[CI] Fix Core DRA crossbuild#7203
pazone merged 8 commits intomainfrom
ci_fix_dra_crossbuild

pazone commented Mar 5, 2025 •

edited

Loading

Uh oh!

mergify bot commented Mar 5, 2025

Uh oh!

Uh oh!

Uh oh!

dliappis commented Mar 5, 2025

Uh oh!

dliappis left a comment

Uh oh!

Uh oh!

Uh oh!

dliappis left a comment

Uh oh!

elastic-sonarqube bot commented Mar 6, 2025

Uh oh!

elasticmachine commented Mar 6, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

pazone commented Mar 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

Why is it important?

Checklist

Disruptive User Impact

How to test this PR locally

Related issues

Questions to ask yourself

Uh oh!

mergify bot commented Mar 5, 2025

Uh oh!

Uh oh!

Uh oh!

dliappis commented Mar 5, 2025

Uh oh!

dliappis left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

dliappis left a comment

Choose a reason for hiding this comment

Uh oh!

elastic-sonarqube bot commented Mar 6, 2025

Quality Gate passed

Uh oh!

elasticmachine commented Mar 6, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

History

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

pazone commented Mar 5, 2025 •

edited

Loading