[DOCS] Add release notes for 7.8.0#56340
Conversation
|
Pinging @elastic/es-docs (>docs) |
When generating the ES releases notes, automatically skip any PRs that have a version label for a released version in the same major series.
|
I'd appreciate a review on this, as although there's still a couple outstanding PRs, the rest can be reviewed. |
|
Re: the CI failure, I confess I'm not sure what I'm doing here with references, so I'd appreciate any guidance here. |
jrodewig
left a comment
jrodewig
left a comment
There was a problem hiding this comment.
Looks good overall. However, there are a few PRs that need to be removed.
I also left some suggestions for replacing the internal xrefs with the equivalent external link. Changing those should fix the broken docs CI test.
The remaining edits are mostly non-blocking nits.
I also tagged @lcawl and @szabosteve on ML-related sections. I'll defer to them on those .
Co-authored-by: James Rodewig <james.rodewig@elastic.co>
|
FYI I merged the extra machine learning PRs from https://raw.githubusercontent.com/elastic/ml-cpp/7.8/docs/CHANGELOG.asciidoc |
| @@ -173,6 +173,37 @@ Machine Learning:: | |||
| * Skip daily maintenance activity if upgrade mode is enabled {pull}54565[#54565] (issue: {issue}54326[#54326]) | |||
| * Start gathering and storing inference stats {pull}53429[#53429] | |||
| * Unassign DFA tasks in SetUpgradeModeAction {pull}54523[#54523] (issue: {issue}54326[#54326]) | |||
There was a problem hiding this comment.
@lcawl what is the official docs position on using the DFA acronym in public documentation? If it isn't supposed to be used then it should be expanded to "data frame analytics" here.
szabosteve
left a comment
szabosteve
left a comment
There was a problem hiding this comment.
I reviewed the ML related parts and left some suggestions. Thank you!
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co> Co-authored-by: Tim Vernum <tim@adjective.org>
|
I've addressed the final outstanding questions. Please let me know if there is anything further to address. |
jrodewig
left a comment
jrodewig
left a comment
There was a problem hiding this comment.
LGTM. Thanks for iterating on this @pugnascotia
Co-authored-by: James Rodewig <james.rodewig@elastic.co> Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co> Co-authored-by: Tim Vernum <tim@adjective.org> Co-authored-by: lcawl <lcawley@elastic.co>
jrodewig
changed the title
7 participants
Add release notes for 7.8.0.
Preview
https://elasticsearch_56340.docs-preview.app.elstc.co/diff