Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Logstash >= 2.0 #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ahmedammar
Copy link

Basic support for 2.0

Reference #8

@raymolin
Copy link
Contributor

hey @ahmedammar , I was trying to install this version of the logstash plugin. I ran into issues both with trying to install it with the gem installed locally as well as with the full path to the plugin (it seems to always read the gem from rubygems.org). Can I ask how you set it up?

@ahmedammar
Copy link
Author

@raymolin:
These are the commands I use:
to build:

jruby -S gem install bundler
bundle install
jruby -S gem build logstash-input-dynamodb.gemspec 

install in log stash:

bin/plugin install ../logstash-input-dynamodb/logstash-input-dynamodb-1.0.0-java.gem 

@raymolin
Copy link
Contributor

Hmm, I've tried that, but I usually hit an issue with this line 's.files = git ls-files.split($)'
So I replaced it with 'Dir["Gemfile", "Rakefile", "LICENSE.txt", "README.md", "lib//", "spec/"'

The plugin successfully installs, but doesn't show up in bin/plugin list.

@ahmedammar
Copy link
Author

@raymolin
Can you even install the original plugin? It seems it was always written to install from a git clone (not a downloaded zip from github) ...

Can you try building and install from a cloned git tree?

@raymolin
Copy link
Contributor

I'm going to test that out right now. To confirm, I am building from the cloned git tree, so I can build the plugin. Where I am seeing issues is, bin/plugin install. I either get 'not a git repository' or the plugin doesn't show up on my list.

@ahmedammar
Copy link
Author

Just pushed a fix in the scan path as I had broken it.

@marcosnils
Copy link

Hey there, just wanted to let you know that the docker container is available now:

https://hub.docker.com/r/mantika/logstash-dynamodb-streams/

@ahmedammar ahmedammar force-pushed the master branch 5 times, most recently from 054bf24 to dcda11a Compare April 7, 2016 22:55
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.

3 participants