Skip to content

Remove building ppc#504

Merged
dliappis merged 1 commit into
elastic:mainfrom
simitt:remove-ppc
Feb 6, 2025
Merged

Remove building ppc#504
dliappis merged 1 commit into
elastic:mainfrom
simitt:remove-ppc

Conversation

@simitt
Copy link
Copy Markdown
Contributor

@simitt simitt commented Feb 6, 2025

Building ppc images is not actually used and adds complexity and transient failures in the CI. The solution is to remove the ppc builds.

@simitt simitt requested a review from a team as a code owner February 6, 2025 07:50
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Feb 6, 2025

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

  • backport-v./d./d is the label to automatically backport to the 1./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify Bot added the backport-skip label Feb 6, 2025
dliappis
dliappis approved these changes Feb 6, 2025
View reviewed changes
Copy link
Copy Markdown
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

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Feb 6, 2025

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Feb 6, 2025

💚 Build Succeeded

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Feb 6, 2025 •
edited
Loading

💛 Build succeeded, but was flaky

Failed CI Steps

History

@dliappis dliappis merged commit 030bb6c into elastic:main Feb 6, 2025
@michel-laterman michel-laterman mentioned this pull request Feb 6, 2025
Merged
@v1v v1v mentioned this pull request Feb 11, 2025
Merged
@nimarezainia nimarezainia mentioned this pull request Apr 6, 2025
Closed
v1v added a commit to v1v/golang-crossbuild that referenced this pull request Apr 7, 2025
@v1v
Revert "Remove building ppc (elastic#504)"
3603c09 
This reverts commit 030bb6c.
@v1v v1v mentioned this pull request Apr 7, 2025
Merged
v1v added a commit that referenced this pull request Apr 7, 2025
@v1v
Revert "Remove building ppc (#504) (#513)" (#582)
d40b4f7
mergify Bot pushed a commit that referenced this pull request Apr 7, 2025
@v1v @mergify
Revert "Remove building ppc (#504) (#513)" (#582)
2f65f23 
(cherry picked from commit d40b4f7)

# Conflicts:
#	go/Makefile.debian12
@mergify mergify Bot mentioned this pull request Apr 7, 2025
Merged
v1v added a commit that referenced this pull request Apr 7, 2025
@mergify @v1v
[1.23](backport #582) Revert "Remove building ppc (#504) (#513)" (#584)
34f40ab 
Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v added a commit that referenced this pull request Apr 7, 2025
@v1v
Revert "Remove building ppc (#504) (#513)" (#582)
e85e59b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jlind23 jlind23 jlind23 approved these changes

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

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

+1 more reviewer

@dliappis dliappis dliappis approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

backport-skip

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@simitt @elasticmachine @dliappis @jlind23