Skip to content

[8.17] Fix empty node stats pipelines (backport #17185)#17198

Merged
kaisecheng merged 1 commit into8.17from
mergify/bp/8.17/pr-17185
Mar 5, 2025
Merged

[8.17] Fix empty node stats pipelines (backport #17185)#17198
kaisecheng merged 1 commit into8.17from
mergify/bp/8.17/pr-17185

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Feb 28, 2025

Release notes

Fixed an issue where the /_node/stats API displayed empty pipeline metrics when X-Pack monitoring was enabled

What does this PR do?

When monitoring is enabled, the metrics store does not initialise data for monitoring pipeline. The api has null pointer when fetching the pipeline config like workers, batch_size.

Why is it important/What is the impact to the user?

/_node/stats gives empty pipelines: {}

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files (and/or docker env variables)
  • I have added tests that prove my fix is effective or that my feature works

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs


This is an automatic backport of pull request #17185 done by [Mergify](https://mergify.com).

Fixed an issue where the `/_node/stats` API displayed empty pipeline metrics
when X-Pack monitoring was enabled

(cherry picked from commit 8678581)
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Feb 28, 2025

@elastic-sonarqube
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Mar 3, 2025

This pull request has not been merged yet. Could you please review and merge it @kaisecheng? 🙏

Copy link
Copy Markdown
Member

@robbavey robbavey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This is ready to merge now

@kaisecheng kaisecheng merged commit c36957c into 8.17 Mar 5, 2025
5 checks passed
@kaisecheng kaisecheng deleted the mergify/bp/8.17/pr-17185 branch March 5, 2025 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants