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.

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

Move infra plugin server to NP folder #53945

Closed
wants to merge 37 commits into from

Conversation

jasonrhodes
Copy link
Member

This PR won't merge, but should demonstrate necessary changes for this migration.

jasonrhodes and others added 30 commits December 11, 2019 17:40
This is a real / proper migration for Metrics. And a partial / stopgap migration for Logs via ui/new_platform (for now).
Decode should be in the dependencies array but this causes re-render and memory maxxing problems. This problem exists on master, so I'm going to revert it here to allow this work to progress.
This isn't 100% ideal but this is quicker than battling with the mocks that aren't working
This stops our application from trying to mount and run just because other plugins are importing our static exports from index. Fixes APM test failures.
@kibanamachine
Copy link
Contributor

💔 Build Failed

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

@jasonrhodes
Copy link
Member Author

Closed in favor of #53955

@jasonrhodes jasonrhodes closed this Jan 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants