Skip to content

Leverage artifact transformations to unpack old test es distributions #79827

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 1 commit into from
Oct 27, 2021
Merged

Leverage artifact transformations to unpack old test es distributions #79827

merged 1 commit into from
Oct 27, 2021

Conversation

breskeby
Copy link
Contributor

This allows better caching of unpacked distributions and is less code to maintain

@breskeby
Leverage artifact transformations to unpack old test es distributions
d21d3d2 
This allows better caching of unpacked distributions and is less code to maintain
@breskeby breskeby self-assigned this Oct 26, 2021
@breskeby breskeby added :Delivery/Build Build or test infrastructure >enhancement Team:Delivery Meta label for Delivery team v7.16.0 v8.0.0 labels Oct 26, 2021
@breskeby breskeby requested a review from mark-vieira October 26, 2021 14:52
@breskeby breskeby marked this pull request as ready for review October 26, 2021 14:52
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@breskeby breskeby requested a review from pugnascotia October 26, 2021 14:52
mark-vieira
mark-vieira approved these changes Oct 26, 2021
View reviewed changes
Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

Nice 👍

@breskeby breskeby merged commit 664b821 into elastic:master Oct 27, 2021
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
❌ 8.0 The branch "8.0" is invalid or doesn't exist
❌ 7.16 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 79827

breskeby added a commit to breskeby/elasticsearch that referenced this pull request Oct 27, 2021
@breskeby
Leverage artifact transformations to unpack old test es distributions (…
52dea4b 
…elastic#79827)

This allows better caching of unpacked distributions and is less code to maintain
@breskeby breskeby mentioned this pull request Oct 27, 2021
Merged
breskeby added a commit that referenced this pull request Oct 27, 2021
@breskeby
Leverage artifact transformations to unpack old test es distributions (…
be29fe1 
…#79827) (#79892)

This allows better caching of unpacked distributions and is less code to maintain
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Oct 27, 2021
@breskeby
Revert "Leverage artifact transformations to unpack old test es distr…
fa8bfca 
…ibutions (elastic#79827)"

This reverts commit 664b821.
This might have caused oldEs090Fixture failures (see elastic#79898)

We have not been able to reproduce the problem nor locally or on an centos8
We will revisit this advancement once we narrowed down the reasoning
of those failures
breskeby added a commit that referenced this pull request Oct 27, 2021
@breskeby
Revert "Leverage artifact transformations to unpack old test es distr…
46b91f5 
…ibutions (#79827)" (#79909)

This reverts commit 664b821.
This might have caused oldEs090Fixture failures (see #79898)

We have not been able to reproduce the problem nor locally or on an centos8
We will revisit this advancement once we narrowed down the reasoning
of those failures
@mark-vieira mark-vieira mentioned this pull request Oct 27, 2021
Merged
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 13, 2021
@breskeby
Leverage artifact transformations to unpack old test es distributions (…
d68aa02 
…elastic#79827) (elastic#79909)
breskeby added a commit that referenced this pull request Dec 14, 2021
@breskeby
Leverage artifact transformations to unpack old test es distributions (…
86bebcb 
…#79827) (#79909) (#81636)
@breskeby breskeby deleted the use-artifacttransform-for-old-es branch December 10, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mark-vieira mark-vieira mark-vieira approved these changes

@pugnascotia pugnascotia Awaiting requested review from pugnascotia

Assignees

@breskeby breskeby

Labels
:Delivery/Build Build or test infrastructure >enhancement Team:Delivery Meta label for Delivery team v7.16.0 v8.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@breskeby @elasticmachine @elasticsearchmachine @mark-vieira @jakelandis