-
Notifications
You must be signed in to change notification settings - Fork 1
Feature/build fixes #3
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
Conversation
@mjnorman - can you pull down the latest code from this branch (have rebased and squashed a couple of commits, so watch out for that) and see if it works for you? |
This is the problem I was having this weekend. When I do a However I am able to build the gem successfully. And it also installs and runs and actually exports data to seq. What is wrong with the build that it won't find the tests and run them? |
What version of the JRE / JDK do you have? I'm using JDK8 and JRuby v9.1.17.0 if that helps. I spend 25 mins on a similar problem until I ran On the off chance that it is, you'll need to ensure that JRuby is first in your
|
I'm sure it has to be something stupid I'm doing, but here is the output. I reran bundle install here also.
Also ran
Any other ideas? |
It's just |
:( |
Weird - maybe my editor added UTF8 preamble bytes to the file or something? Wonder why it works on my system though... Do you have a hex editor? :) |
I can get one. How about this. Can you clone into a new directory, pull down your branch, and see if you get the same thing? Can you paste all that output here and let me compare? |
Sure - will do that first thing tomorrow. |
So this has to do with something in logstash's configurations. I was able to create a new standard ruby gem project--
Then I was able to run the specs normally--
|
I'm about to try fetching the cloning and building the repository from scratch. Will let you know how I go :) |
Yep, I get that error too when I check out from scratch! Let me see if I can figure out why... |
Ok, this worked for me now:
Can you do a pull and give it a try? |
UGH--
|
Bizarre! Ok, perhaps stronger measures are called for: Can you unzip spec.zip in the root of your repository and see if the unzipped files work? I'm thinking this might be a git configuration issue (or something along those lines)... |
Yes I’ll do that. What is your dev environment? |
Output of
|
I’m not in the office, will do this tonight. Wanted to get this down while I’m thinking about it though. I can create a new gem |
Extracted that to the directory, same error. Even started with a fresh clone, git checkout feature branch, then extract to overwrite spec files. Same outcome. |
Interesting - are you using Logstash from source by any chance? I'm not; I'm just installing the gems... |
(I don't actually have Logstash installed locally) |
I just downloaded the zip from logstash website and extracted it. |
Ok so I have figured out it is something with my machine. I took a Windows 2012 server, installed jruby and git, cloned, and ran the rspec and it worked fine. I don't know what's up with my machine, but you can probably complete this pull request if you don't have any other changes. |
Ok, will get it merged and published :) |
I finally managed to fix my build, Thanks! |
Believe we may have some issues here now that I'm actually able to use the plugin
|
Ugh, that looks ugly. What version of logstash are you using? I'm going to try to set up a repro environment this morning. |
So far I have not been able to reproduce it again, but it happened with 6.4.1. Right now I have had logstash streaming to seq using this branch's updates for a couple of hours on 6.4.1 and have not seen it since. Would like to see if you experience any issues with it. |
Ok, cheers! I'm wondering if there are any acceptance-test / integration-test suites as part of Logstash than can be used to test plugins for compliance and correctness of behaviour...? Will see what I can come up with :) |
Update plugin to work against latest version(s) of Logstash.
Relates to #1 and #2.
Replaces #2.
CC: @mjnorman