-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
vis_type_timeseries server side new platform migration #52501
vis_type_timeseries server side new platform migration #52501
Conversation
…refactor some Framework level deps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, legacy server registration is still a little hard to follow but moving init directly into the register legacy API method makes it much much easier than the observable flow before, and I'm not sure we can do better until things are migrated. Thanks so much for taking this on!!!
@@ -25,7 +25,7 @@ describe('getEsShardTimeout', () => { | |||
const req = { | |||
getEsShardTimeout: async () => { | |||
return 12345; | |||
} | |||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ugh lol
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know right...dunno why it didn't tell me in the first place 🤔
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
Summary
WIP of the migration of thevis_type_timeseries
plugin to the new platform, on the server side.This is a server side migration of the
vis_type_timeseries
plugin to the new platform. It migrates what was needed for the Logs and Metrics dependencies.What's covered?
vis_type_timeseries
plugin now has a New Platform plugin, within thesrc/plugins
directory.getVisData
as part of it's public contract to other plugins within thesetup()
method.vis_type_timeseries
("metrics") plugin, with all hacks / workarounds removed. Infra was the only plugin I could find using thegetVisData
function in a plugin <-> plugin capacity.getVisData
function has a facade added at the top level to augment an object to look like the oldreq
it required, but using the newrequestContext
andrequest
from the New Platform router. This was to make refactoring easier at this stage, there are no TypeScript types to aid in changing this code, so changing many layers of code seemed brittle./api/metrics/vis/data
API endpoint has been fully migrated to the NP router. This is the endpoint things like Kibana's Visualize uses.What's not covered?
___LEGACY.server
.