Skip to content

Conversation

@alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Apr 15, 2019

This PR fixes the artifact names to include the architecture as introduced in the 7.0.0 release.
Up until recently this worked because 7.0.0 was being built and picked up locally.

With the 7.0.0 release, we switched to download the packages instead of
using locally built ones.
This PR fixes the artifact names to include the architecture as
introduced in the 7.0.0 release.
@alpar-t alpar-t added >non-issue :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts v8.0.0 v7.2.0 labels Apr 15, 2019
@alpar-t alpar-t requested review from jasontedor and rjernst April 15, 2019 13:15
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@alpar-t
Copy link
Contributor Author

alpar-t commented Apr 15, 2019

@elasticmachine run elasticsearch-ci/packaging

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@alpar-t
Copy link
Contributor Author

alpar-t commented Apr 15, 2019

The sample packaging tests passed and the full run looks like unrelated memory allocation failure:

# curl: error while loading shared libraries: libcrypt.so.1: failed to map segment from shared object: Cannot allocate memory``` 
will merge this to get the initial issue out of the way. 

@alpar-t alpar-t merged commit 6aa52b1 into elastic:master Apr 15, 2019
alpar-t added a commit that referenced this pull request Apr 15, 2019
With the 7.0.0 release, we switched to download the packages instead of
using locally built ones.
This PR fixes the artifact names to include the architecture as
introduced in the 7.0.0 release.
gurkankaymak pushed a commit to gurkankaymak/elasticsearch that referenced this pull request May 27, 2019
)

With the 7.0.0 release, we switched to download the packages instead of
using locally built ones.
This PR fixes the artifact names to include the architecture as
introduced in the 7.0.0 release.
alpar-t added a commit that referenced this pull request May 29, 2019
With the 7.0.0 release, we switched to download the packages instead of
using locally built ones.
This PR fixes the artifact names to include the architecture as
introduced in the 7.0.0 release.

Closes #42003
@alpar-t alpar-t deleted the fix-download-location-pacakging-tests branch November 11, 2019 09:41
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >non-issue Team:Delivery Meta label for Delivery team v7.2.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants