Skip to content

[Fleet] Do not enable uninstall command link without Agents:All#195185

Merged
nchaulet merged 1 commit intoelastic:mainfrom
nchaulet:fix-uninstall-command
Oct 7, 2024
Merged

[Fleet] Do not enable uninstall command link without Agents:All#195185
nchaulet merged 1 commit intoelastic:mainfrom
nchaulet:fix-uninstall-command

Conversation

@nchaulet
Copy link
Copy Markdown
Member

@nchaulet nchaulet commented Oct 7, 2024

Summary

Resolve #192055

Do not enable uninstall command link when user do not have Agents:All, we also show a tooltip explaining why it's disabled

UI Change

Screenshot 2024-10-07 at 10 17 17 AM

@nchaulet nchaulet added release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team backport:prev-minor v8.16.0 labels Oct 7, 2024
@nchaulet nchaulet self-assigned this Oct 7, 2024
@nchaulet nchaulet requested a review from a team as a code owner October 7, 2024 08:20
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

Copy link
Copy Markdown
Contributor

@juliaElastic juliaElastic left a comment

Choose a reason for hiding this comment

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

LGTM

@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #70 / discover/esql discover esql view switch modal should not show switch modal when switching to a data view while a saved search is open

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
fleet 1209 1210 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fleet 1.7MB 1.7MB +509.0B

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

cc @nchaulet

@nchaulet nchaulet merged commit 3c1c646 into elastic:main Oct 7, 2024
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

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

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 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

kibanamachine added a commit that referenced this pull request Oct 7, 2024
…#195185) (#195222)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Fleet] Do not enable uninstall command link without Agents:All
(#195185)](#195185)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Nicolas
Chaulet","email":"nicolas.chaulet@elastic.co"},"sourceCommit":{"committedDate":"2024-10-07T10:54:38Z","message":"[Fleet]
Do not enable uninstall command link without Agents:All
(#195185)","sha":"3c1c64678b142c36535e6db0810d8d02467489cc","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v9.0.0","backport:prev-minor","v8.16.0"],"title":"[Fleet]
Do not enable uninstall command link without
Agents:All","number":195185,"url":"https://github.com/elastic/kibana/pull/195185","mergeCommit":{"message":"[Fleet]
Do not enable uninstall command link without Agents:All
(#195185)","sha":"3c1c64678b142c36535e6db0810d8d02467489cc"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195185","number":195185,"mergeCommit":{"message":"[Fleet]
Do not enable uninstall command link without Agents:All
(#195185)","sha":"3c1c64678b142c36535e6db0810d8d02467489cc"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Nicolas Chaulet <nicolas.chaulet@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fleet]: Forbidden error is displayed on clicking Get Uninstall command for user role Agents: None.

5 participants