Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Get artifacts from beats/build/distributions #40

Merged
merged 1 commit into from
Jul 2, 2018

Conversation

andrewkroh
Copy link
Member

@andrewkroh andrewkroh commented Jun 29, 2018

Prior to elastic/beats#7388 artifacts were written to build/upload by the Beats build. This directory was renamed to better reflect its purpose and contents.

Unfortunately I didn't realize the rename would affect this project. I tried to add a symlink in the release manager, but the artifacts are copied to ARTIFACTS_DIR so that didn't work.

This should fix the issue seen in RM builds.

Does this PR include tests?

No, but I did test it with make from-snapshots. Output: https://travis-ci.com/andrewkroh/beats-docker/builds/77746885

Prior to elastic/beats#7388 artifacts were written to build/upload
by the Beats build. This directory was renamed to better reflect its
purpose and contents.
@andrewkroh andrewkroh requested a review from ninaspitfire June 29, 2018 19:31
Copy link
Member

@mgreau mgreau left a comment

Choose a reason for hiding this comment

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

LGTM

@mgreau mgreau merged commit e07b18f into elastic:master Jul 2, 2018
andrewkroh added a commit to andrewkroh/beats-docker that referenced this pull request Jul 20, 2018
Prior to elastic/beats#7388 artifacts were written to build/upload
by the Beats build. This directory was renamed to better reflect its
purpose and contents.
mgreau pushed a commit that referenced this pull request Jul 23, 2018
Prior to elastic/beats#7388 artifacts were written to build/upload
by the Beats build. This directory was renamed to better reflect its
purpose and contents.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants