Skip to content

[7.x] [APM] Client new platform migration (#64046)#64958

Merged
ogupte merged 2 commits intoelastic:7.xfrom
ogupte:backport/7.x/pr-64046
May 1, 2020
Merged

[7.x] [APM] Client new platform migration (#64046)#64958
ogupte merged 2 commits intoelastic:7.xfrom
ogupte:backport/7.x/pr-64046

Conversation

@ogupte
Copy link
Contributor

@ogupte ogupte commented Apr 30, 2020

Backports the following commits to 7.x:

* migrate files from legacy path to new plugin path

* update file paths to reflect migrated files

* move minimal legacy client files back to legacy path in order to run kibana

* Completes the full cutover to the new kibana platform removing all shims and legacy adapters.

* Adds APM to ignored list for casing check.

* - moves public/utils/pickKeys.ts to common/utils/pick_keys.ts
- exposes getTraceUrl as a plugin static export of apm/public and updates import in infra
- fixes FeaturesPluginSetup import in apm/public app
- renames get_apm_index_pattern_titles -> get_apm_index_pattern_title
- getApmIndexPatternTitle is now a synchronous getter function
- removes unused comments and xpack.apm.apmForESDescription i18n translations

* Moves automatic index pattern saved object creation from
plugin start to when the Home screen first renders

* removed unnecessary legacy css imports

* fixed ci issues by:
- moving readOnly badge, and help extension setup to occure only
  when apm app is mounted
- registering saved object types
- also moved createStaticIndexPattern from a react useEffect on the
  APM home screen to when the app is mounted
@ogupte ogupte added the backport This PR is a backport of another PR label Apr 30, 2020
@ogupte
Copy link
Contributor Author

ogupte commented May 1, 2020

retest

1 similar comment
@ogupte
Copy link
Contributor Author

ogupte commented May 1, 2020

retest

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@ogupte ogupte merged commit f39a1e6 into elastic:7.x May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants