Skip to content
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.

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

Ensure tar is present in Docker image #76696

Merged
merged 1 commit into from
Aug 19, 2021

Conversation

pugnascotia
Copy link
Contributor

Closes #76695.

Due to a Kubernetes quirk, we need to ensure that the tar CLI tool is
present in the Docker image. It was removed in #75079 because we
reworked how the image was built, and anything not explicitly installed
was left out.

Closes elastic#76695.

Due to a Kubernetes quirk, we need to ensure that the `tar` CLI tool is
present in the Docker image. It was removed in elastic#75079 because we
reworked how the image was built, and anything not explicitly installed
was left out.
@pugnascotia pugnascotia added >non-issue :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts v7.15.0 v7.16.0 labels Aug 19, 2021
@pugnascotia pugnascotia requested a review from breskeby August 19, 2021 08:06
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Aug 19, 2021
@elasticmachine
Copy link
Collaborator

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

@pugnascotia pugnascotia added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Aug 19, 2021
Copy link
Contributor

@breskeby breskeby left a comment

Choose a reason for hiding this comment

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

lgtm

@pugnascotia pugnascotia merged commit 0b6704b into elastic:7.x Aug 19, 2021
@pugnascotia pugnascotia deleted the 76695-add-tar-to-docker branch August 19, 2021 10:12
pugnascotia added a commit that referenced this pull request Aug 19, 2021
Closes #76695.

Due to a Kubernetes quirk, we need to ensure that the `tar` CLI tool is
present in the Docker image. It was removed in #75079 because we
reworked how the image was built, and anything not explicitly installed
was left out.
@pugnascotia
Copy link
Contributor Author

Backported to 7.15 in 161a134.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >non-issue Team:Delivery Meta label for Delivery team v7.15.0 v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants