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

Remove jms and hipchat outputs (broken plugins) #408

Closed
wants to merge 1 commit into from

Conversation

jordansissel
Copy link
Contributor

Ideally we could have the docs sync tool do this task, but it doesn't today :)

This removes the jms and hipchat output plugins per elastic/logstash#7993 (comment)

% perl build_docs.pl --doc ../logstash/docs/index.asciidoc --chunk 1
Building HTML from ../logstash/docs/index.asciidoc
Done
See: /home/jls/projects/docs/html_docs/index.html

Docs build seems ok. I checked (with firefox) to see if the jms and hipchat outputs were removed.

Reasons:

* The jms output cannot be installed under Logstash 5.6 because of a
transitive dependency that requires Ruby >=2.1
* The hipchat output cannot be installed under Logstash 5.6 because of a
transitive dependency that requires Ruby >=2.0

History here: elastic/logstash#7993 (comment)
@dedemorton
Copy link
Contributor

@jordansissel You'll also need to remove the mention that shows up here:

https://www.elastic.co/guide/en/logstash/current/introduction.html

Incidentally you didn't catch this with the local build because Deb had to change the link format in order to accommodate translated versions of the getting started content that have only a sub-set of the topics. The full build will return an error for missing links.

Copy link
Contributor

@dedemorton dedemorton left a comment

Choose a reason for hiding this comment

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

See the comment I added about the intro having a link that needs to be removed.

@jordansissel
Copy link
Contributor Author

Good catch. I'll file a PR to remove that reference in the logstash docs.

@jordansissel
Copy link
Contributor Author

elastic/logstash#8115 removes the reference from the Logstash docs.

@dedemorton
Copy link
Contributor

Approved pending merge of elastic/logstash#8115

@elasticsearch-bot
Copy link

Jordan Sissel merged this into the following branches!

Branch Commits
5.6 0bfbe4c

elasticsearch-bot pushed a commit that referenced this pull request Aug 31, 2017
Reasons:

* The jms output cannot be installed under Logstash 5.6 because of a
transitive dependency that requires Ruby >=2.1
* The hipchat output cannot be installed under Logstash 5.6 because of a
transitive dependency that requires Ruby >=2.0

History here: elastic/logstash#7993 (comment)

Fixes #408
@jordansissel
Copy link
Contributor Author

Merged; closing.

@jsvd jsvd deleted the remove-nonfunctioning-plugins branch November 25, 2019 15:43
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.

3 participants