Skip to content

Conversation

kaisecheng
Copy link
Contributor

Fixed: elastic/logstash#11236

This PR replace remove ruby pipeline

@kaisecheng kaisecheng requested a review from andsel December 22, 2020 13:18
Copy link

@andsel andsel left a comment

Choose a reason for hiding this comment

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

I've left a note and a question

COPY --chown=logstash:logstash *.gemspec VERSION* version* /usr/share/plugins/plugin/
RUN cp /usr/share/logstash/logstash-core/versions-gem-copy.yml /usr/share/logstash/versions.yml
ENV PATH="${PATH}:/usr/share/logstash/vendor/jruby/bin"
# NOTE: since 8.0 JDK is bundled as part of the LS distribution under $LS_HOME/jdk
Copy link

Choose a reason for hiding this comment

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

It's from 7.10 that's bundled in the distribution

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it is copied from .ci/Dockerfile I can change it to 7.10


echo "Testing against version: $ELASTIC_STACK_VERSION"

if [[ "$ELASTIC_STACK_VERSION" = *"-SNAPSHOT" ]]; then
Copy link

Choose a reason for hiding this comment

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

I've not a such a deep knowledge of docker, but docker pull is the same as this curl + docker load ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

docker pull pull the image from the registry while docker load import image from STDIN .tar.gz

@kaisecheng kaisecheng requested a review from andsel January 6, 2021 08:51
Copy link

@andsel andsel left a comment

Choose a reason for hiding this comment

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

LGTM

@kaisecheng kaisecheng merged commit 1d8117c into logstash-plugins:master Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants