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

Made two changes to get around known logstash issues #72

Closed
wants to merge 3 commits into from

Conversation

bhirsch70
Copy link

@sadsfae First, thanks for this work, it's exactly what I needed and really helped me out!

I ran into two issues during the playbook run.

  1. An incompatibility with the java version that automatically got installed (11). After some research, I found the issue #9316 that states you must use java 1.8
    with java10 I get these error :Unrecognized VM option 'UseParNewGC' elastic/logstash#9316

  2. A bug in which the logstash systemd unit does not get installed. issue #9403
    Missing systemd scripts in 6.2.4 elastic/logstash#9403

I've linked to both issues in the commit comments.

'with java10 I get these error :Unrecognized VM option 'UseParNewGC' #9316'
elastic/logstash#9316

- Added a workaround to problem in which the logstash systemd unit does not get
installed automatically.  See known issue:
'Missing systemd scripts in 6.2.4 #9403'
elastic/logstash#9403
@sadsfae
Copy link
Owner

sadsfae commented Jun 10, 2019

@bhirsch70 Did you still want to work on this PR? Thanks for working on it earlier, some things have changed since so give the current master branch a shot and see if it's still needed. You'll need to rebase against tip of master if so.

sadsfae added a commit that referenced this pull request Aug 4, 2019
* I cannot merge PR #72 but including two changes from it in applying
setcap for jvm options / workaround
* Specifically installing java-1.8.0-openjdk-headless, though EL7 will
always default to this.
* Removing an inference that Fedora is supported, it just changes too
fast to provide ELK coverage/testing for it.  Use CentOS7 instead.
sadsfae added a commit that referenced this pull request Aug 4, 2019
@sadsfae
Copy link
Owner

sadsfae commented Aug 4, 2019

Hey @bhirsch70 since the time from your PR and now the repo and upstream code has changed so I pulled in two of your fixes from this PR.

426acda

@sadsfae sadsfae closed this Aug 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants