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

[Known issue][Detection Engine][8.16.0 & 8.16.1] Exception item comment validation on newline chars #6280

Merged
merged 21 commits into from
Dec 12, 2024

Conversation

nastasha-solomon
Copy link
Contributor

@nastasha-solomon nastasha-solomon commented Dec 5, 2024

Partially addresses #6275 by updating the 8.16.0 and 8.16.1 release notes. This will be doc'd as a known issue for 8.17 in #6224.

Preview: 8.16.x release notes

@nastasha-solomon nastasha-solomon added release-notes known-issue Priority: High Issues that are time-sensitive and/or are of high customer importance Effort: Small Issues that can be resolved quickly v8.16.0 v8.16.1 labels Dec 5, 2024
@nastasha-solomon nastasha-solomon self-assigned this Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@nastasha-solomon nastasha-solomon marked this pull request as ready for review December 6, 2024 19:05
@nastasha-solomon nastasha-solomon requested a review from a team as a code owner December 6, 2024 19:05
@nastasha-solomon nastasha-solomon changed the title [Known issue][Detection Engine] Exception item comment validation on newline chars [Known issue][Detection Engine][8.16.0 & 8.16.1] Exception item comment validation on newline chars Dec 6, 2024
Copy link
Contributor

@rylnd rylnd left a comment

Choose a reason for hiding this comment

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

@nastasha-solomon I had a few suggestions; they apply to both sections but I didn't duplicate the suggested changes. Let me know if you have any questions.

docs/release-notes/8.16.asciidoc Outdated Show resolved Hide resolved
docs/release-notes/8.16.asciidoc Outdated Show resolved Hide resolved
docs/release-notes/8.16.asciidoc Outdated Show resolved Hide resolved
Copy link
Contributor

@rylnd rylnd left a comment

Choose a reason for hiding this comment

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

Thank you for adding markup, links, and much-needed grammar here! LGTM.

docs/release-notes/8.16.asciidoc Outdated Show resolved Hide resolved
Copy link
Contributor

@yctercero yctercero left a comment

Choose a reason for hiding this comment

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

Thank you, thank you! Left one minor comment.

@nastasha-solomon nastasha-solomon merged commit f0d60d6 into 8.x Dec 12, 2024
4 checks passed
mergify bot pushed a commit that referenced this pull request Dec 12, 2024
…nt validation on newline chars (#6280)

* First draft

* Minor tweaks

* missing word

* Grammar

* More small tweaks

* Ryland's input

* Small fixes

* Cleanup

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

* Formatting fixes

* fixes sample commands

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Updates last step

* Adds fix info

* Removes fix details

---------

Co-authored-by: Ryland Herrick <[email protected]>
(cherry picked from commit f0d60d6)
mergify bot pushed a commit that referenced this pull request Dec 12, 2024
…nt validation on newline chars (#6280)

* First draft

* Minor tweaks

* missing word

* Grammar

* More small tweaks

* Ryland's input

* Small fixes

* Cleanup

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

* Formatting fixes

* fixes sample commands

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Updates last step

* Adds fix info

* Removes fix details

---------

Co-authored-by: Ryland Herrick <[email protected]>
(cherry picked from commit f0d60d6)
nastasha-solomon added a commit that referenced this pull request Dec 12, 2024
…m comment validation on newline chars (backport #6280) (#6312)

* First draft

* Minor tweaks

* missing word

* Grammar

* More small tweaks

* Ryland's input

* Small fixes

* Cleanup

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

* Formatting fixes

* fixes sample commands

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Updates last step

* Adds fix info

* Removes fix details

---------

Co-authored-by: Ryland Herrick <[email protected]>
(cherry picked from commit f0d60d6)

Co-authored-by: Nastasha Solomon <[email protected]>
nastasha-solomon added a commit that referenced this pull request Dec 12, 2024
…m comment validation on newline chars (backport #6280) (#6311)

* First draft

* Minor tweaks

* missing word

* Grammar

* More small tweaks

* Ryland's input

* Small fixes

* Cleanup

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

Co-authored-by: Ryland Herrick <[email protected]>

* Update docs/release-notes/8.16.asciidoc

* Formatting fixes

* fixes sample commands

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Update docs/release-notes/8.16.asciidoc

* Updates last step

* Adds fix info

* Removes fix details

---------

Co-authored-by: Ryland Herrick <[email protected]>
(cherry picked from commit f0d60d6)

Co-authored-by: Nastasha Solomon <[email protected]>
@nastasha-solomon nastasha-solomon deleted the issue-6275-exception-item-ki branch December 16, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Effort: Small Issues that can be resolved quickly known-issue Priority: High Issues that are time-sensitive and/or are of high customer importance release-notes v8.16.0 v8.16.1 v8.17.0 v8.18.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants