Skip to content

Conversation

@myasonik
Copy link
Contributor

@myasonik myasonik commented Nov 2, 2020

Fixes #82311

Not sure why or how it got out of sync in 7.10 after being correct on 7.8, 7.9, and master (as of June 2) but it's correct now again.

This is still correct in master so planning on merging this directly into 7.10 unless someone tells me otherwise.

Screen Shot 2020-11-02 at 16 14 14

sorenlouv and others added 30 commits October 13, 2020 15:28
…lastic#80198) (elastic#80322)

* Added component to truncate text programmatically and add tooltips. Changed all field text values to use this component.

* Fixed build failures.

* Switched from FC component to one without children

Co-authored-by: Paul Tavares <[email protected]>

* Added some docs for the shared component.

* Updated snapshots.

Co-authored-by: Paul Tavares <[email protected]>

Co-authored-by: Paul Tavares <[email protected]>
…eld (elastic#80001) (elastic#80332)

* Correct the Name field label on the Trusted Apps Create form to say "Name your trusted application" (was "Name your trusted app application").
* fixes to make grid layout a bit more solid.

* Changed the spacing a bit more.

* Updated snapshot.
(cherry picked from commit 4f4abf2)
…#80318)

* [ILM] Cloud-specific changes for 7.10 (elastic#79650)

* added cloud cold tier specific call out, probably wip

* added jest test

* rephrase copy

* Added logic for showing/hiding data tier allocation option

- Added server-side test for detecting legacy config
- Added client-side test for asserting on cloud data tier option
  is hidden

* added test for not-on-cloud case

* Refactored logic for rendering data allocation notices

Also updated a comment.

* paraphrashing of Adams copy recommendation

* Fix comment

* address pr feedback

* clarify slight hack comment

* complete refactor of new flag for tests

Co-authored-by: Kibana Machine <[email protected]>
# Conflicts:
#	x-pack/plugins/index_lifecycle_management/server/routes/api/nodes/register_list_route.ts

* replace r with (r)
* [ML] add swim lane styles for dark theme

* [ML] fix global time range update on sell selection

* [ML] fix getSelectionTimeRange

* [ML] fix range selection for embeddable

* [ML] fix job selection

* [ML] fix swim lane limit
…lastic#80376)

* Add loading spinner to EQL Query Bar

This gives the user some indication that the validation is being
performed in the background, and that their query's validity is
currently undefined.

* Preserve whitespace on display of rule queries

This is useful for all rules, but particularly for EQL rules, where
sequences will be multiline.

* Fix tests around query description

This content is now wrapped in a whitespace-preserving element, so these
need to be updated ever so slightly.

* Update cypress assertions around rule queries

* Remove extra space from assertion
  * This extra space was removed in the presentation component. I didn't
    find an explicit reason why it was there in the first place.
* Use have.value when asserting on form inputs
…rades (elastic#80391) (elastic#80421)

* filter out non-upgradeable agents when user bulk upgrades

* check for unenrolled or unenrolling agents in isAgentUpgradeable
… bug with query preview + tests (elastic#80110) (elastic#80425)

## Summary

This PR is meant to address the remaining rule query preview bugs. It addresses the following:

- gives same loading experience for all rule query types (previously there were two different loading states)
- makes sure noise warnings are showing for all query types and disappear on timeframe or query change
- updates when to/from are set so it is set on preview click
When `add`ing an element with an already existing ID, it will use the already existing data instead of setting new data. This made it so when you change the time picker the element health statuses would not update.

When we get new elements, replace the existing data for existing elements.

Fixes elastic#80335.
…0349) (elastic#80457)

This PR stabilizes the functional calendar tests, which are flaky on cloud.
…elastic#80317) (elastic#80466)

* [ML] Fix Anomaly Explorer charts time range to obey time picker range

* [ML] Fix explorer_charts_container_service Jest tests
… tests (elastic#80224) (elastic#80401)

* Enabled Trusted Apps functional tests
* Additional Trusted App Functional Tests
* Updated Jest snapshots
…ta gird. (elastic#80467) (elastic#80494)

- Fix for color coding for outlier detection due to updated feature influence format.
- Fix for displaying cell values for nested fields.
…) (elastic#80502)

## Summary

fixes elastic#80136

Haven't added/updated tests yet but tested the various combinations in the UI and all worked as expected.

<details><summary>Screenshots</summary>
<h3>Policy listing</h3>
<img width="1302" alt="Screen Shot 2020-10-14 at 7 05 19 AM" src="https://user-images.githubusercontent.com/57655/95981056-1c1c6d00-0dec-11eb-9746-5e91043f3d19.png">

<h3>Update existing policy succeeds</h3>
<img width="1308" alt="Screen Shot 2020-10-14 at 7 05 49 AM" src="https://user-images.githubusercontent.com/57655/95981059-1cb50380-0dec-11eb-9e55-7a14b72085f9.png">
<h3>Creating new policy w/existing name fails</h3>
<img width="1308" alt="Screen Shot 2020-10-14 at 7 06 06 AM" src="https://user-images.githubusercontent.com/57655/95981062-1cb50380-0dec-11eb-833b-1ab6636a5c67.png">

<h3>Updating policy to existing name fails</h3>
<img width="1308" alt="Screen Shot 2020-10-14 at 7 08 05 AM" src="https://user-images.githubusercontent.com/57655/95981078-1f175d80-0dec-11eb-91d5-7ffd976da48e.png">

</details>
### Checklist

- [ ] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
jloleysens and others added 15 commits October 29, 2020 09:39
…lastic#82026)

* Enhance page load duration metrics and helper tooltips
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
…ic#81952) (elastic#82022)

Fixes elastic#81950

by not assuming the summary is present in a bucket with partial check
info
…lastic#82054)

* Mimics metrics permissions for alert mgmt in logs feature

* Updates logs security functional tests

Alert management permissions were added for read and all logs users in this PR, so both of these users now have "Stack Management" appear in the nav.

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
…astic#82075)

* Add experimental copy to upgrade agent(s)

* Adjust copy after review

* Fix string

* Adjust quotes
# Conflicts:
#	x-pack/plugins/ingest_manager/public/applications/ingest_manager/sections/fleet/components/agent_upgrade_modal/index.tsx
…sion path chart (elastic#82011) (elastic#82108)

Fixes a regression that caused data grid cells for feature importance to be empty and clicking on the button to show the decision path chart popover to render the whole page empty.
Fixed links to 404 page.

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Bo Andersen <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>
@myasonik myasonik added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// REASSIGN from Team:Core UI Deprecated label for old Core UI team v7.10.0 labels Nov 2, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core-ui (Team:Core UI)

@myasonik myasonik added the release_note:skip Skip the PR/issue when compiling release notes label Nov 2, 2020
@kobelb
Copy link
Contributor

kobelb commented Nov 2, 2020

#66320 was never backported to the 7.x branch, and a separate PR was used to make these changes directly in 7.8. Since these changes are already in master, I'd suggest targeting this PR against the 7.x branch, and then backporting it to 7.10 branch.

@myasonik myasonik changed the base branch from 7.10 to 7.x November 2, 2020 22:04
@myasonik
Copy link
Contributor Author

myasonik commented Nov 2, 2020

Things have gone wrong...

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
advancedSettings 50 52 +2
canvas 1131 1120 -11
console 172 170 -2
data 571 572 +1
enterpriseSearch 403 413 +10
expressions 107 105 -2
globalSearchBar 23 21 -2
graph 169 170 +1
home 208 207 -1
indexLifecycleManagement 206 208 +2
infra 1147 1144 -3
kibanaReact 307 306 -1
kibanaUtils 193 192 -1
logstash 190 189 -1
maps 618 619 +1
ml 1220 1228 +8
monitoring 638 633 -5
searchprofiler 87 85 -2
security 462 463 +1
securitySolution 2043 2040 -3
timelion 99 91 -8
transform 339 337 -2
triggersActionsUi 285 283 -2
uiActionsEnhanced 138 139 +1
uptime 657 656 -1
visDefaultEditor 241 240 -1
visTypeTimelion 45 37 -8
visualize 207 206 -1
watcher 238 237 -1
total -31

@kbn/ui-shared-deps asset size

id before after diff
[email protected] 2.4MB 2.4MB +134.0B
kbn-ui-shared-deps.js 4.7MB 4.7MB +68.9KB
total +69.0KB

async chunks size

id before after diff
advancedSettings 892.2KB 941.0KB +48.8KB
apm 4.1MB 4.2MB +8.4KB
canvas 1.5MB 1.4MB -76.9KB
console 1.1MB 1.0MB -10.0KB
data 273.8KB 289.1KB +15.3KB
discover 440.8KB 440.8KB +43.0B
enterpriseSearch 640.4KB 649.3KB +8.9KB
graph 1.3MB 1.3MB +546.0B
home 404.0KB 394.5KB -9.5KB
indexLifecycleManagement 233.3KB 236.1KB +2.8KB
indexPatternManagement 662.9KB 667.9KB +5.0KB
infra 3.8MB 3.8MB -44.0KB
ingestManager 1.3MB 1.2MB -8.4KB
kibanaReact 358.7KB 359.0KB +328.0B
lens 1.1MB 1.0MB -6.5KB
logstash 213.4KB 203.9KB -9.5KB
maps 3.2MB 3.2MB +4.9KB
ml 11.3MB 11.3MB +31.2KB
monitoring 1.2MB 1.1MB -60.8KB
observability 167.3KB 167.3KB +12.0B
savedObjectsManagement 207.0KB 207.1KB +41.0B
searchprofiler 705.1KB 695.1KB -10.0KB
security 1.0MB 1.0MB +482.0B
securitySolution 10.5MB 10.5MB -2.8KB
timelion 461.6KB 382.8KB -78.8KB
transform 1.2MB 1.2MB -19.2KB
triggersActionsUi 1.5MB 1.5MB +11.0KB
uptime 1.7MB 1.7MB -9.0KB
visDefaultEditor 458.6KB 449.1KB -9.5KB
visTypeTagcloud 295.4KB 295.4KB +15.0B
visTypeTimelion 154.3KB 75.5KB -78.8KB
visTypeTimeseries 1.8MB 1.8MB +48.0B
visualize 274.1KB 264.6KB -9.5KB
watcher 1.0MB 1.0MB -9.9KB
total -315.1KB

distributable file count

id before after diff
default 48118 48484 +366
oss 29400 28827 -573

page load bundle size

id before after diff
apm 48.2KB 48.3KB +65.0B
canvas 1.0MB 1013.8KB -12.2KB
cloud 5.9KB 6.0KB +88.0B
console 30.5KB 30.6KB +65.0B
core 661.8KB 655.9KB -5.8KB
dashboard 350.8KB 350.8KB -1.0B
data 1.1MB 1.1MB -1.9KB
discover 88.0KB 88.0KB -2.0B
embeddable 222.3KB 222.4KB +82.0B
expressions 204.2KB 194.7KB -9.5KB
globalSearchBar 46.8KB 37.2KB -9.6KB
graph 16.1KB 16.3KB +138.0B
home 26.0KB 26.1KB +65.0B
indexLifecycleManagement 89.9KB 90.0KB +26.0B
indexManagement 632.3KB 632.9KB +559.0B
indexPatternManagement 136.1KB 136.2KB +73.0B
infra 178.6KB 178.9KB +303.0B
ingestManager 391.4KB 392.0KB +612.0B
kibanaOverview 40.5KB 40.5KB +6.0B
kibanaReact 143.9KB 134.3KB -9.6KB
kibanaUtils 179.5KB 170.1KB -9.4KB
lens 79.7KB 79.8KB +65.0B
licensing 23.1KB 23.5KB +350.0B
lists 164.7KB 164.7KB +8.0B
logstash 37.6KB 37.7KB +65.0B
ml 65.6KB 65.7KB +65.0B
monitoring 247.8KB 248.5KB +709.0B
observability 73.0KB 76.4KB +3.4KB
reporting 164.5KB 164.5KB +15.0B
searchprofiler 51.0KB 51.1KB +65.0B
security 169.9KB 172.1KB +2.1KB
securityOss 15.4KB 15.4KB +12.0B
securitySolution 593.2KB 604.0KB +10.8KB
telemetryManagementSection 36.6KB 38.2KB +1.6KB
transform 25.5KB 25.6KB +65.0B
triggersActionsUi 151.5KB 141.6KB -9.9KB
uiActionsEnhanced 327.1KB 327.6KB +538.0B
upgradeAssistant 64.4KB 64.4KB +7.0B
uptime 25.2KB 25.3KB +65.0B
visDefaultEditor 34.4KB 34.4KB +65.0B
visualize 41.4KB 41.5KB +91.0B
watcher 28.1KB 28.1KB +65.0B
total -45.8KB

History

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

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

Labels

REASSIGN from Team:Core UI Deprecated label for old Core UI team release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.