-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Fleet] Fix displaying agent status bar with no agents #156541
[Fleet] Fix displaying agent status bar with no agents #156541
Conversation
Pinging @elastic/fleet (Team:Fleet) |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a small comment, otherwise LGTM
@@ -35,13 +35,30 @@ export const AgentStatusBar: React.FC<{ | |||
return acc; | |||
}, [] as Array<{ stop: number; color: string }>); | |||
}, [agentStatus]); | |||
|
|||
// const totalAgents = useMemo(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forgotten comment? :D
|
||
import { AgentStatusBar } from './status_bar'; | ||
|
||
describe('AgentStatusBar', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the test!
…hub.com:nchaulet/kibana into fix-121483-status-bar-whith-no-agents
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @nchaulet |
(cherry picked from commit 4301d80)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
#156583) # Backport This will backport the following commits from `main` to `8.8`: - [[Fleet] Fix displaying agent status bar with no agents (#156541)](#156541) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Nicolas Chaulet","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-05-03T16:14:55Z","message":"[Fleet] Fix displaying agent status bar with no agents (#156541)","sha":"4301d80d12e9cd3b48573d6dd1719baff8335c64","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:Fleet","v8.8.0","v8.9.0"],"number":156541,"url":"https://github.com/elastic/kibana/pull/156541","mergeCommit":{"message":"[Fleet] Fix displaying agent status bar with no agents (#156541)","sha":"4301d80d12e9cd3b48573d6dd1719baff8335c64"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/156541","number":156541,"mergeCommit":{"message":"[Fleet] Fix displaying agent status bar with no agents (#156541)","sha":"4301d80d12e9cd3b48573d6dd1719baff8335c64"}}]}] BACKPORT--> Co-authored-by: Nicolas Chaulet <[email protected]>
Summary
Resolve #121483
Fix displaying the agent status bar when filtering results in no agents.
How to test