[Automatic Import] Telemetry and UI Regression Bug#265161
Merged
robester0403 merged 3 commits intoelastic:mainfrom Apr 23, 2026
Merged
[Automatic Import] Telemetry and UI Regression Bug#265161robester0403 merged 3 commits intoelastic:mainfrom
robester0403 merged 3 commits intoelastic:mainfrom
Conversation
robester0403
added
release_note:skip
botelastic
Bot
added
the
Team:Fleet
Contributor
|
Pinging @elastic/integration-experience (Team:Integration-Experience) |
Contributor
|
Pinging @elastic/fleet (Team:Fleet) |
Contributor
ApprovabilityVerdict: Needs human review While the changes themselves are straightforward bug fixes (preventing duplicate telemetry events and adding UI spacing), the author does not own any of the modified files - both are owned by @elastic/fleet. Code owners should review these changes to their files. You can customize Macroscope's approvability policy. Learn more. |
jen-huang
reviewed
Apr 22, 2026
Contributor
jen-huang
left a comment
jen-huang
left a comment
There was a problem hiding this comment.
I removed the top padding in #264212, there was too much whitespace for the browse integrations tab.
I recommend adding a spacer just before <ManageIntegrationsTable /> is rendered, or padding to that component itself. Otherwise we'll re-introduce double whitespace.
… top of manage integrations table
Contributor
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
|
Contributor
Author
That makes sense, I will use a spacer in place of padding. |
jen-huang
approved these changes
Apr 23, 2026
Contributor
jen-huang
left a comment
jen-huang
left a comment
There was a problem hiding this comment.
Ty for the changes! LGTM
Contributor
|
Starting backport for target branches: 9.4 https://github.com/elastic/kibana/actions/runs/24831585217 |
Contributor
💚 All backports created successfully
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
Apr 23, 2026
…265257) # Backport This will backport the following commits from `main` to `9.4`: - [[Automatic Import] Telemetry and UI Regression Bug (#265161)](#265161) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Robert So (robester0403)","email":"85914248+robester0403@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-04-23T11:01:53Z","message":"[Automatic Import] Telemetry and UI Regression Bug (#265161)\n\nAdded dependencies to the manage integrations table viewed telemetry\nevent that was attempting to fire too early and therefore was not\nsuccessful.\n\nAdded top spacing to the ui of manage integrations table.","sha":"19c3e8a77b5be8e02bf32263417835f3b2b79211","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","backport:version","Feature:AutomaticImport","Team:Integration-Experience","v9.4.0","v9.5.0"],"title":"[Automatic Import] Telemetry and UI Regression Bug","number":265161,"url":"https://github.com/elastic/kibana/pull/265161","mergeCommit":{"message":"[Automatic Import] Telemetry and UI Regression Bug (#265161)\n\nAdded dependencies to the manage integrations table viewed telemetry\nevent that was attempting to fire too early and therefore was not\nsuccessful.\n\nAdded top spacing to the ui of manage integrations table.","sha":"19c3e8a77b5be8e02bf32263417835f3b2b79211"}},"sourceBranch":"main","suggestedTargetBranches":["9.4"],"targetPullRequestStates":[{"branch":"9.4","label":"v9.4.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/265161","number":265161,"mergeCommit":{"message":"[Automatic Import] Telemetry and UI Regression Bug (#265161)\n\nAdded dependencies to the manage integrations table viewed telemetry\nevent that was attempting to fire too early and therefore was not\nsuccessful.\n\nAdded top spacing to the ui of manage integrations table.","sha":"19c3e8a77b5be8e02bf32263417835f3b2b79211"}}]}] BACKPORT--> Co-authored-by: Robert So (robester0403) <85914248+robester0403@users.noreply.github.com>
smith
pushed a commit
to smith/kibana
that referenced
this pull request
Apr 23, 2026
Added dependencies to the manage integrations table viewed telemetry event that was attempting to fire too early and therefore was not successful. Added top spacing to the ui of manage integrations table.
rbrtj
pushed a commit
to walterra/kibana
that referenced
this pull request
Apr 27, 2026
Added dependencies to the manage integrations table viewed telemetry event that was attempting to fire too early and therefore was not successful. Added top spacing to the ui of manage integrations table.
SoniaSanzV
pushed a commit
to SoniaSanzV/kibana
that referenced
this pull request
Apr 27, 2026
Added dependencies to the manage integrations table viewed telemetry event that was attempting to fire too early and therefore was not successful. Added top spacing to the ui of manage integrations table.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This pr has the following minor fixes:
Added dependencies to the manage integrations table viewed telemetry event that was attempting to fire too early and therefore was not successful.
Adds top padding to the manage integrations table.
Checklist
Check the PR satisfies following conditions.
Reviewers should verify this PR satisfies this list as well.
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support- [ ] Documentation was added for features that require explanation or tutorials- [ ] Unit or functional tests were updated or added to match the most common scenarios- [ ] If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list- [ ] This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. Therelease_note:breakinglabel should be applied in these situations.- [ ] Flaky Test Runner was used on any tests changed- [ ] The PR description includes the appropriate Release Notes section, and the correctrelease_note:*label is applied per the guidelinesbackport:*labels.