Skip to content
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

[meta] failing plugins specs under master #4371

Closed
14 of 15 tasks
colinsurprenant opened this issue Dec 18, 2015 · 11 comments
Closed
14 of 15 tasks

[meta] failing plugins specs under master #4371

colinsurprenant opened this issue Dec 18, 2015 · 11 comments
Labels

Comments

@colinsurprenant
Copy link
Contributor

Here's the survey of the currently failing specs under master at 066e778

ng-pipeline related failures

devutils pipeline mocking

bad in-specs pipeline mocking

out-of-order batches?

unknown origin

I suspect this are actually related to elastic/logstash-devutils#42

hanging specs

other

solutions

@colinsurprenant
Copy link
Contributor Author

I just added the devutils pipeline mocking problems issue.

@colinsurprenant
Copy link
Contributor Author

proposed short-term fix by @andrewvc #4373

@colinsurprenant
Copy link
Contributor Author

real "long-term" refactor required in #4382

@colinsurprenant
Copy link
Contributor Author

#4384 solves most issues. updated the description tasks.

@colinsurprenant
Copy link
Contributor Author

added logstash-plugins/logstash-input-log4j#27 can either @andrewvc or @jsvd take a look? not sure if related to changes introduced by your last pipeline modifications

@colinsurprenant
Copy link
Contributor Author

log4j fixed by logstash-plugins/logstash-input-log4j#28 thanks @andrewvc

@ph
Copy link
Contributor

ph commented Dec 22, 2015

@colin Where is this list coming from a local run of a Jenkins job?
This list is different from http://build-eu-00.elasticsearch.org/view/LS%20Master/job/Logstash_Master_Default_Plugins/899/console

@ph
Copy link
Contributor

ph commented Dec 22, 2015

I have fixed the other issues found from running this job

logstash-plugins/logstash-filter-date#53
logstash-plugins/logstash-output-s3#64

@jsvd
Copy link
Member

jsvd commented Dec 22, 2015

@jordansissel
Copy link
Contributor

Looks like most of the items in this check list are now checked. Is it OK to close any tickets that are marked w/ a check here?

@colinsurprenant
Copy link
Contributor Author

this is all done now for the exception of #4382 which will be dealt with eventually. closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants