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

[Stack Monitoring] Make sure SM queries can handle both process and processes metrics across 7.x and 8.x indices #117614

Closed
Tracked by #113171 ...
matschaffer opened this issue Nov 5, 2021 · 4 comments
Labels
Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services won't fix

Comments

@matschaffer
Copy link
Contributor

In #113149 we'll add support for procesess metrics.

In this issue we need to address how to handle process metrics coming from older kibanas into kibana 8.0 stack monitoring UI.

We're anticipating the use of runtime fields to "upgrade" process metrics to processes but it's unclear if this has been attempted anywhere in the stack yet.

@matschaffer matschaffer added v8.0.0 Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services required-for-8.0 This work is required to be done before 8.0 lands, bc it relates to a breaking change or similar. labels Nov 5, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@jasonrhodes jasonrhodes changed the title [Stack Monitoring] Add data migration from for process to processes metrics from kibana 7.x [Stack Monitoring] Make sure SM queries can handle both process and processes metrics across 7.x and 8.x indices Nov 8, 2021
@jasonrhodes
Copy link
Member

@matschaffer I'm not 100% clear on which parts of these 2 tickets are absolutely required for 8.0 vs. things that are planned for later in order to support the eventual need for more than 1 process.

I think the idea in this ticket is to resolve how to read from old data that still references process, is that right? Can we query for both in 8.0+ and combine them in the code for now, as an option if needed?

@matschaffer
Copy link
Contributor Author

matschaffer commented Nov 15, 2021

Can we query for both in 8.0+ and combine them in the code for now

Working out a good way to query/display both is the intent of this issue.

As for what's required for 8.0, that might be a question for @Bamieh - I see #115200 is labeled 8.0.0. If we merge that and remove process from kibana in that version, we'll need this issue (as well as elastic/beats#27569) or we'll lose kibana process metrics.

@matschaffer
Copy link
Contributor Author

Closing this for the same reasons covered in #113149 (comment)

Rel #119560
Rel #104124

@matschaffer matschaffer added won't fix and removed v8.0.0 Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services required-for-8.0 This work is required to be done before 8.0 lands, bc it relates to a breaking change or similar. labels Nov 25, 2021
@botelastic botelastic bot added the needs-team Issues missing a team label label Nov 25, 2021
@matschaffer matschaffer added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Nov 25, 2021
@botelastic botelastic bot removed the needs-team Issues missing a team label label Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services won't fix
Projects
None yet
Development

No branches or pull requests

3 participants