search

elastic / beats-docker

Official Beats Docker images
166 stars 55 forks source link

Don't build images by default when using the `test` target #30

Closed dliappis closed 6 years ago

dliappis commented 6 years ago

To support tests based on snapshot builds the test target shouldn't try to build images by default.

Also, use the more generic venv code we already have in elasticsearch-docker so that it doesn't fail when there's a newer version than Python 3.5.

dliappis commented 6 years ago

This is needed to support testing in https://devops-ci.elastic.co

Will need to be backported to 6.x, 6.1 and 6.0.

jarpy commented 6 years ago

LGTM, naturally. Thanks.

jarpy commented 6 years ago

Merged in 96be74bc5aa5de0834519c0e1be1179d83b0674e and backported to 6.0, 6.1, 6.x.