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

[7.x] Add deprecation message for xpack.spaces.enabled #112234

Closed
wants to merge 1 commit into from

Conversation

watson
Copy link
Contributor

@watson watson commented Sep 15, 2021

Fixes part of #82467

This PR adds a deprecation message if xpack.spaces.enabled is set to false in kibana.yml.

We could have expanded the scope and also shown the deprecation message if xpack.spaces.enabled was set to true, but to lessen the burden of upgrading to 8.0, we have decided that just checking for false is enough.

Here's how it will look in the Upgrade Assistant UI:

image

Release note

Deprecates disabling the Spaces plugin.

@watson watson added release_note:deprecation backport:skip This commit does not require backporting v7.16.0 labels Sep 15, 2021
@watson watson requested review from a team September 15, 2021 10:02
@watson watson self-assigned this Sep 15, 2021
@kibanamachine
Copy link
Contributor

kibanamachine commented Sep 15, 2021

💔 Build Failed

Failed CI Steps

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @watson

@legrego
Copy link
Member

legrego commented Sep 15, 2021

We added this deprecation back in the 7.11 release (#83984). Is that not working the way we thought it would?

@watson
Copy link
Contributor Author

watson commented Sep 15, 2021

Ah you're right. I just realise that I've been working in the wrong file 🤦 - It should obvious go in the spaces plugin and not the security plugin. However, I can see that the deprecation warning added for spaces back then needs to be updated to align with our new guidelines. I'll close this PR and create a new one that changes that.

@watson watson closed this Sep 15, 2021
@watson watson deleted the deprecation-spaces-enabled branch September 15, 2021 11:27
@legrego
Copy link
Member

legrego commented Sep 15, 2021

@watson thanks. For consistency, we may want to target your new PR against master, since this deprecation message lives there as well. Either that, or we open a different PR to remove the message from master altogether (whichever y'all think is best)

@watson
Copy link
Contributor Author

watson commented Sep 15, 2021

Here's the new PR: #112242

I'll make a separate PR to remove the deprecation message in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:deprecation v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants