Skip to content

[7.6] Fix initialization of Logstash service URL (#17497)#17593

Merged
ycombinator merged 3 commits intoelastic:7.6from
ycombinator:backport/7.6/pr-17497
Apr 9, 2020
Merged

[7.6] Fix initialization of Logstash service URL (#17497)#17593
ycombinator merged 3 commits intoelastic:7.6from
ycombinator:backport/7.6/pr-17497

Conversation

@ycombinator
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.6:

@ycombinator ycombinator added [zube]: In Review backport Team:Services (Deprecated) Label for the former Integrations-Services team labels Apr 7, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/integrations-services (Team:Services)

Copy link
Copy Markdown
Contributor

@kaiyan-sheng kaiyan-sheng left a comment

Choose a reason for hiding this comment

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

lgtm

@ycombinator
Copy link
Copy Markdown
Contributor Author

CI failures are legitimate. To fix them we need to merge #17598 first, then rebase this PR here on 7.6.

* Fix initialization of Logstash service URL

* Adding CHANGELOG entry

* Expose Logstash HTTP API port in local environment

* Remove once field as it is no longer used

* Encapsulating more logic into method

* Debugging

* Compare cluster UUIDs

* Adding ES service

* Adding default LS pipeline that outputs to ES cluster

* Use beats input so LS pipeline (and node) keeps running

* Fixing up ES config in docker compose file

* Handle errors

* Fixing typo

* COPY pipeline file as part of image

* Refactoring code to make testable + Adding unit tests

* Adding comment for test

* Fixing URIs
@ycombinator
Copy link
Copy Markdown
Contributor Author

CI failures are unrelated. Merging.

@ycombinator ycombinator merged commit 0f0acb1 into elastic:7.6 Apr 9, 2020
@ycombinator ycombinator deleted the backport/7.6/pr-17497 branch April 9, 2020 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Services (Deprecated) Label for the former Integrations-Services team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants