Skip to content

Conversation

@eedugon
Copy link
Contributor

@eedugon eedugon commented Dec 17, 2025

When we added that document we didn't realized we already had documented some specific steps and checks for Elastic Security users regarding detection rules.

This PR adds a note for the 7.17 --> 8.19 upgrade step to remind Elastic Security users to read that doc and highlights some of the items.

Closes #3619

@eedugon eedugon requested a review from a team as a code owner December 17, 2025 13:46
@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025

Vale Linting Results

Summary: 1 suggestion found

💡 Suggestions (1)
File Line Rule Message
deploy-manage/upgrade/deployment-or-cluster/upgrade-717.md 102 Elastic.Capitalization '8.19 upgrade preparations' should use sentence-style capitalization.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025

🔍 Preview links for changed docs

@shainaraskas shainaraskas requested a review from a team December 17, 2025 18:44
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

I think the experience team should validate this as well

Copy link
Contributor

@nastasha-solomon nastasha-solomon left a comment

Choose a reason for hiding this comment

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

Just left one suggestion that's totally optional. Otherwise, lgtm - thank you!


It's highly recommended to start this upgrade from the latest 7.17.x patch release to ensure that you’re using the most recent version of the Elastic Upgrade Assistant. You should also upgrade to the latest available 8.19.x patch release so that the same benefits apply when you later upgrade to 9.x.

:::::{note}
Copy link
Member

Choose a reason for hiding this comment

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

@natasha-moore-elastic or @jmikell821 can you please take a look at this change.

Copy link
Member

Choose a reason for hiding this comment

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

hah. never mind, @nastasha-solomon beat me to it (again!)

Copy link
Contributor

Choose a reason for hiding this comment

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

me jumping on PR reviews:
pr-reviews

eedugon and others added 3 commits December 29, 2025 10:14
Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
Co-authored-by: Nastasha Solomon <79124755+nastasha-solomon@users.noreply.github.com>
@eedugon eedugon merged commit c55fde9 into main Dec 29, 2025
7 of 8 checks passed
@eedugon eedugon deleted the upgrade_717_9x_siem branch December 29, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add to the 7.17 to 9.x upgrade doc info about Security solution

5 participants