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.

Sign up for GitHub

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

Jump to bottom

[Connectors] Update connector stats panels #183829

Merged
merged 4 commits into from
May 21, 2024
Merged

[Connectors] Update connector stats panels #183829

merged 4 commits into from
May 21, 2024

Conversation

jedrazb
Copy link
Member

@jedrazb jedrazb commented May 20, 2024 •
edited by kibanamachine

Summary

Design spec

image

Updates connector stats panels:

  • connector info grouped together
  • added connector ID with option to copy it to clipboard

Index stats panel:

  • doc count moved there

Pipelines panel:

  • unchanged, given that we have @custom and @inference pipelines we need to agree how to count preprocessors, out of scope of the original issue

Screenshots

Happy path case
Screenshot 2024-05-20 at 15 07 20

Index missing case e.g. connector created via API
Screenshot 2024-05-20 at 15 08 21

Checklist

Delete any items that are not applicable to this PR.

@jedrazb jedrazb added release_note:skip Skip the PR/issue when compiling release notes v8.14.0 v8.15.0 labels May 20, 2024
@jedrazb jedrazb marked this pull request as ready for review May 20, 2024 13:11
@jedrazb jedrazb requested a review from a team as a code owner May 20, 2024 13:11
efegurkan
efegurkan approved these changes May 21, 2024
View reviewed changes
Copy link
Member

@efegurkan efegurkan left a comment

Choose a reason for hiding this comment

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

LGTM

@jedrazb jedrazb merged commit 68becd2 into elastic:main May 21, 2024
5 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 21, 2024
@jedrazb
[Connectors] Update connector stats panels (elastic#183829)
dea0962 
(cherry picked from commit 68becd2)
@kibanamachine kibanamachine mentioned this pull request May 21, 2024
Open
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
✅ 8.14

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

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 23, 2024
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

8 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@efegurkan efegurkan efegurkan approved these changes

Assignees
No one assigned
Labels
backport missing Added to PRs automatically when the are determined to be missing a backport. release_note:skip Skip the PR/issue when compiling release notes v8.14.0 v8.15.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jedrazb @kibanamachine @efegurkan