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

Upgrade EUI to v62.0.3 #136865

Closed
wants to merge 22 commits into from
Closed

Upgrade EUI to v62.0.3 #136865

wants to merge 22 commits into from

Conversation

cee-chen
Copy link
Contributor

@cee-chen cee-chen commented Jul 21, 2022 •
edited
Loading

Closing this in favor of #138351, since the linked PR has had better success squashing flaky FTR failures

@cee-chen cee-chen added release_note:skip Skip the PR/issue when compiling release notes EUI backport:skip This commit does not require backporting v8.5.0 ci:deploy-cloud labels Jul 21, 2022
@cee-chen

This comment was marked as resolved.

Sign in to view
@elizabetdev

This comment was marked as resolved.

Sign in to view
@cee-chen cee-chen changed the title Upgrade EUI to v61.0.0 Upgrade EUI to v62.0.0 Jul 27, 2022
@cee-chen cee-chen force-pushed the eui/61.0.0 branch 2 times, most recently from 5034981 to 87ce793 Compare July 27, 2022 20:06
@cee-chen cee-chen changed the title Upgrade EUI to v62.0.0 Upgrade EUI to v62.0.1 Jul 27, 2022
@cee-chen cee-chen force-pushed the eui/61.0.0 branch 2 times, most recently from f0cf117 to 24be441 Compare July 28, 2022 15:33
@cee-chen cee-chen changed the title Upgrade EUI to v62.0.1 Upgrade EUI to v62.0.2 Aug 2, 2022
@cee-chen cee-chen force-pushed the eui/61.0.0 branch 3 times, most recently from f3d730e to 6b35170 Compare August 3, 2022 17:56
@cee-chen cee-chen mentioned this pull request Aug 3, 2022
Merged
2 tasks
@cee-chen cee-chen changed the title Upgrade EUI to v62.0.2 Upgrade EUI to v62.0.3 Aug 4, 2022
@cee-chen cee-chen force-pushed the eui/61.0.0 branch 5 times, most recently from 9ab0114 to ed4dcaf Compare August 8, 2022 17:37
cee-chen added 11 commits August 8, 2022 13:21
@cee-chen
[EuiErrorBoundary] Update snapshots from Emotion conversion
8c9639f
@cee-chen
[EuiImage] Update snapshots, tests, and CSS to account for Emotion co…
bfc763e 
…nversion
@cee-chen
[EuiImage][RTL] Fix test failures caused by EuiImage changes
7bb1ea8
@cee-chen
[EuiCommentList] Deprecate EuiCommentProps.type
00a7904
@cee-chen
[EuiCommentList] Rename timelineIcon prop to timelineAvatar
0f5fe13 
- see elastic/eui#6071
@cee-chen
[EuiCommentList] Fix selectors deprecated by Emotion conversion
f5d923c
@cee-chen
[EuiPopover][EuiCommentEvent][Enzyme] Fix mounted test failures cause…
169f8c9 
…d by Emotion conversions

- Mounting displays the Emotion wrapper with the data-test-subj on them - we need to specify the output div renders in order for text assertions to be correct
@cee-chen
[EuiPopover] Deprecate initialFocus={false} as an option
3f7d556 
see elastic/eui#6044
@cee-chen
[EuiPopover] Rename display=inlineBlock to inline-block
bf35bbc 
- see elastic/eui#5977
@cee-chen
[EuiPopover] Update snapshots from Emotion conversion
7a27c58
@cee-chen
[EuiPopover] Replace deprecated .euiPopover__panel-isOpen class wit…
717233f 
…h new `[data-popover-open]` attribute
@cee-chen cee-chen force-pushed the eui/61.0.0 branch 2 times, most recently from cd4de46 to 718efe0 Compare August 8, 2022 22:35
cee-chen added 4 commits August 8, 2022 18:15
@cee-chen
[EuiPopover][RTL] Fix test failures caused by not waiting for EuiPopo…
732c6a4 
…ver animation/transition
@cee-chen
Skip failing a11y tests
5a0b293 
- test w/ similar error already skipped in another test above
- requires closing the popover for next test to pass
- not sure why delete action is no longer available
@cee-chen
Fix failing Security Cypress tests
0c42a27
@cee-chen
Fix FTR Add/Save Filter failures
d279fd6
@cee-chen
Copy link
Contributor Author

cee-chen commented Aug 9, 2022

@elasticmachine merge upstream

@cee-chen
Copy link
Contributor Author

cee-chen commented Aug 9, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented Aug 9, 2022 •
edited
Loading

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #29 / lens app - group 2 lens dashboard tests should not carry over filters if creating a new lens visualization from within dashboard
  • [job] [logs] FTR Configs #29 / lens app - group 2 lens dashboard tests should not carry over filters if creating a new lens visualization from within dashboard

Metrics [docs]

Async chunks

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

id before after diff
apm 3.0MB 3.0MB -16.0B
canvas 1.0MB 1.0MB +6.0B
cases 358.5KB 358.4KB -40.0B
osquery 1.0MB 1.0MB -16.0B
securitySolution 5.6MB 5.6MB +2.0B
synthetics 912.4KB 912.4KB -12.0B
triggersActionsUi 1.0MB 1.0MB +6.0B
unifiedSearch 232.6KB 232.6KB +54.0B
visDefaultEditor 142.1KB 142.1KB -8.0B
visTypeTimeseries 451.4KB 451.4KB -16.0B
total -40.0B

Canvas Sharable Runtime

The Canvas "shareable runtime" is an bundle produced to enable running Canvas workpads outside of Kibana. This bundle is included in third-party webpages that embed canvas and therefor should be as slim as possible.

id before after diff
module count 4876 4904 +28
total size 7.6MB 7.6MB -13.2KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 349.2KB 350.4KB +1.2KB
kbnUiSharedDeps-css 487.5KB 470.6KB -16.9KB
kbnUiSharedDeps-npmDll 5.1MB 5.1MB +19.7KB
securitySolution 254.7KB 254.7KB -30.0B
unifiedSearch 46.8KB 46.8KB +5.0B
total +4.0KB

History

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

@cee-chen cee-chen removed release_note:skip Skip the PR/issue when compiling release notes EUI backport:skip This commit does not require backporting v8.5.0 labels Aug 9, 2022
@cee-chen cee-chen closed this Aug 9, 2022
@cee-chen cee-chen deleted the eui/61.0.0 branch October 4, 2022 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cee-chen @elizabetdev @kibana-ci @kibanamachine