Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[logstash] update doc and values.yaml for http.host issues #591

Merged
merged 1 commit into from
Apr 27, 2020

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Apr 21, 2020

  • add warning that http.host: 0.0.0.0 is required with default probes
  • add syntax to disable or override default probes
  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

* add warning that `http.host: 0.0.0.0` is required with default probes
* add syntax to disable or override default probes
@jmlrt jmlrt added the enhancement New feature or request label Apr 21, 2020
@jmlrt jmlrt requested a review from a team April 21, 2020 15:04
Copy link

@mgreau mgreau left a comment

Choose a reason for hiding this comment

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

LGTM

@jmlrt jmlrt merged commit 8083a5e into elastic:master Apr 27, 2020
@jmlrt jmlrt deleted the logstash-http-host-var branch April 27, 2020 08:12
@jmlrt jmlrt added the v7.7.0 label Apr 27, 2020
jmlrt added a commit that referenced this pull request Apr 27, 2020
[logstash] update doc and values.yaml for http.host issues
@jmlrt
Copy link
Member Author

jmlrt commented Apr 27, 2020

backported to 7.7 branch in ab26d11

@jmlrt jmlrt mentioned this pull request Oct 28, 2020
This was referenced Nov 17, 2020
@jmlrt jmlrt mentioned this pull request Feb 8, 2021
This was referenced Mar 15, 2021
@jmlrt jmlrt mentioned this pull request May 25, 2021
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
enhancement New feature or request v7.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants