-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Conversation
jenkins test this please |
CHANGELOG.md
Outdated
|
||
* 7.4.1 as the default stack version | ||
* 6.8.4 as 6.x tested version | ||
* Helm 2.15.1 support in [#309](https://github.com/elastic/helm-charts/pull/338) [@jmlrt](https://github.com/jmlrt) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Helm 2.15.1 support in [#309](https://github.com/elastic/helm-charts/pull/338) [@jmlrt](https://github.com/jmlrt) | |
* Helm 2.15.1 support in [#338](https://github.com/elastic/helm-charts/pull/338) [@jmlrt](https://github.com/jmlrt) |
CHANGELOG.md
Outdated
@@ -1,5 +1,34 @@ | |||
# Changelog | |||
|
|||
## 7.4.1 - 2019/10/22 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## 7.4.1 - 2019/10/22 | |
## 7.4.1 - 2019/10/23 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, left comments on minor doc tweaks.
fixed changelog and removed logstash files commited by mistake |
CHANGELOG.md
Outdated
|
||
* 7.4.1 as the default stack version | ||
* 6.8.4 as 6.x tested version | ||
* Helm 2.15.1 support in [#309](https://github.com/elastic/helm-charts/pull/309) [@jmlrt](https://github.com/jmlrt) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be #338 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes sorry 🤦♂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No worries
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
does what the tin say