Skip to content

Conversation

@herrBez
Copy link
Contributor

@herrBez herrBez commented Nov 28, 2025

Proposed commit message

[Elasticsearch][Package] Improve Cluster and Node View Dashboard by improving node.roles queries

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

  • In Elastic Cloud activate Stack Monitoring
  • Install the Elasticsearch Integration
  • Verify that the Dashboard Cluster & Node View correctly populates the field

For legacy data role:

Use the following manifest on ECK:

---
apiVersion: elasticsearch.k8s.elastic.co/v1
kind: Elasticsearch
metadata:
  name: monitored-sample
  namespace: default
spec:
  version: 9.2.1
  monitoring:
    metrics:
      elasticsearchRefs:
      - name: monitored-sample
        namespace: default
  nodeSets:
  - name: default
    count: 1
    config:
      node.store.allow_mmap: false
      node.roles: ["master", "data", "ingest"]
---
apiVersion: kibana.k8s.elastic.co/v1
kind: Kibana
metadata:
  name: quickstart
spec:
  version: 9.2.1
  count: 1
  elasticsearchRef:
    name: monitored-sample
  • Install the Elasticsearch integration and verify that all panels are empty
  • Update the integration with the fix and verify that the panels are populated.

Related issues

Screenshots

Example with legacy "data" role:

BEFORE:

image

AFTER:

image

@herrBez herrBez requested a review from a team as a code owner November 28, 2025 07:55
@andrewkroh andrewkroh added dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Integration:elasticsearch Elasticsearch Team:Stack Monitoring Stack Monitoring team [elastic/stack-monitoring] labels Nov 28, 2025
@consulthys
Copy link
Contributor

On the following line, the version must also be changed.

This issue was the reason why this had to be done manually. It seems to be fixed now, so we should now template the ingest pipeline name in the future.
cc @3kt

Copy link
Contributor

@consulthys consulthys left a comment

Choose a reason for hiding this comment

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

LGT Stack Monitoring

@herrBez herrBez enabled auto-merge (squash) December 2, 2025 16:26
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

Package elasticsearch 👍(3) 💚(1) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
slowlog 6578.95 5263.16 -1315.79 (-20%) 💔

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

@herrBez herrBez merged commit af0d889 into elastic:main Dec 2, 2025
7 checks passed
@elastic-vault-github-plugin-prod

Package elasticsearch - 1.19.1 containing this change is available at https://epr.elastic.co/package/elasticsearch/1.19.1/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Integration:elasticsearch Elasticsearch Team:Stack Monitoring Stack Monitoring team [elastic/stack-monitoring]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants