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

Add warnings to upgrade assistant for search sessions #206998

Merged
merged 14 commits into from
Jan 28, 2025

Conversation

lukasolson
Copy link
Member

@lukasolson lukasolson commented Jan 16, 2025

Summary

Part of #203925.
Resolves #205813.

Adds warning messages to the upgrade assistant if a cluster has unexpired search sessions, since the feature is being disabled by default and will have to be explicitly re-enabled to manage the sessions.

@lukasolson lukasolson self-assigned this Jan 16, 2025
Copy link
Contributor

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.

@lukasolson lukasolson added v9.0.0 Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. Feature:Search Sessions backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes labels Jan 16, 2025
@lukasolson lukasolson marked this pull request as ready for review January 17, 2025 22:05
@lukasolson lukasolson requested review from a team as code owners January 17, 2025 22:05
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

Copy link
Contributor

@jughosta jughosta left a comment

Choose a reason for hiding this comment

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

Hi Lukas,

I left some suggestions for the messages.

@lukasolson lukasolson requested a review from jughosta January 27, 2025 18:56
Copy link
Contributor

@jughosta jughosta left a comment

Choose a reason for hiding this comment

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

LGTM 👍

We are targeting 9.0 with this change, right? Then backport:skip label would fit better than backport:prev-minor.

@lukasolson
Copy link
Member Author

We are targeting 9.0 with this change, right?

Wouldn't this be backported to 8.x so it shows up before you upgrade to 9.0?

@jughosta
Copy link
Contributor

If we are planning to have this warning in 8.x then I think the wording needs to be slightly updated for 8.x branch: instead of has been disabled to will be disabled. Wdyt?

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #83 / InfraOps App Metrics UI Home page with metrics present Saved Views should update the current saved view and load it

Metrics [docs]

✅ unchanged

History

cc @lukasolson

@lukasolson lukasolson merged commit b998946 into elastic:main Jan 28, 2025
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/13020528425

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 206998

Questions ?

Please refer to the Backport tool documentation

@lukasolson
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

lukasolson added a commit to lukasolson/kibana that referenced this pull request Jan 29, 2025
## Summary

Part of elastic#203925.
Resolves elastic#205813.

Adds warning messages to the upgrade assistant if a cluster has
unexpired search sessions, since the feature is being disabled by
default and will have to be explicitly re-enabled to manage the
sessions.

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Julia Rechkunova <[email protected]>
(cherry picked from commit b998946)

# Conflicts:
#	docs/upgrade-notes.asciidoc
lukasolson added a commit that referenced this pull request Jan 30, 2025
…#208833)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Add warnings to upgrade assistant for search sessions
(#206998)](#206998)

<!--- Backport version: 9.6.4 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Lukas
Olson","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-28T22:13:53Z","message":"Add
warnings to upgrade assistant for search sessions (#206998)\n\n##
Summary\r\n\r\nPart of
https://github.com/elastic/kibana/issues/203925.\r\nResolves
https://github.com/elastic/kibana/issues/205813.\r\n\r\nAdds warning
messages to the upgrade assistant if a cluster has\r\nunexpired search
sessions, since the feature is being disabled by\r\ndefault and will
have to be explicitly re-enabled to manage
the\r\nsessions.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<[email protected]>\r\nCo-authored-by:
Julia Rechkunova
<[email protected]>","sha":"b998946003273245f9ea851ad60347c9f83c593a","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:DataDiscovery","Feature:Search
Sessions","backport:prev-minor"],"title":"Add warnings to upgrade
assistant for search
sessions","number":206998,"url":"https://github.com/elastic/kibana/pull/206998","mergeCommit":{"message":"Add
warnings to upgrade assistant for search sessions (#206998)\n\n##
Summary\r\n\r\nPart of
https://github.com/elastic/kibana/issues/203925.\r\nResolves
https://github.com/elastic/kibana/issues/205813.\r\n\r\nAdds warning
messages to the upgrade assistant if a cluster has\r\nunexpired search
sessions, since the feature is being disabled by\r\ndefault and will
have to be explicitly re-enabled to manage
the\r\nsessions.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<[email protected]>\r\nCo-authored-by:
Julia Rechkunova
<[email protected]>","sha":"b998946003273245f9ea851ad60347c9f83c593a"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/206998","number":206998,"mergeCommit":{"message":"Add
warnings to upgrade assistant for search sessions (#206998)\n\n##
Summary\r\n\r\nPart of
https://github.com/elastic/kibana/issues/203925.\r\nResolves
https://github.com/elastic/kibana/issues/205813.\r\n\r\nAdds warning
messages to the upgrade assistant if a cluster has\r\nunexpired search
sessions, since the feature is being disabled by\r\ndefault and will
have to be explicitly re-enabled to manage
the\r\nsessions.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<[email protected]>\r\nCo-authored-by:
Julia Rechkunova
<[email protected]>","sha":"b998946003273245f9ea851ad60347c9f83c593a"}}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) Feature:Search Sessions release_note:skip Skip the PR/issue when compiling release notes Team:DataDiscovery Discover, search (e.g. data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Search Sessions] Add warning messaging to upgrade assistant
4 participants