Skip to content

[redisenterprise_otel] Add redisenterprise_otel content pack#18002

Merged
ishleenk17 merged 4 commits intoelastic:mainfrom
Linu-Elias:redisenterprise_otel
Apr 9, 2026
Merged

[redisenterprise_otel] Add redisenterprise_otel content pack#18002
ishleenk17 merged 4 commits intoelastic:mainfrom
Linu-Elias:redisenterprise_otel

Conversation

@Linu-Elias
Copy link
Copy Markdown
Contributor

@Linu-Elias Linu-Elias commented Mar 24, 2026

Proposed commit message

See title

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

Related issues

Screenshots

redisenterprise_otel-overview redisenterprise_otel-shard-diagnostics redisenterprise_otel-proxy-listener redisenterprise_otel-database-performance redisenterprise_otel-node-health

@Linu-Elias Linu-Elias requested a review from a team as a code owner March 24, 2026 10:17
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

Vale Linting Results

Summary: 3 warnings, 1 suggestion found

⚠️ Warnings (3)
File Line Rule Message
packages/redisenterprise_otel/docs/README.md 35 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'for example' instead of 'e.g'.
packages/redisenterprise_otel/docs/README.md 36 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'for example' instead of 'e.g'.
packages/redisenterprise_otel/docs/README.md 37 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
💡 Suggestions (1)
File Line Rule Message
packages/redisenterprise_otel/docs/README.md 119 Elastic.WordChoice Consider using 'can, might' instead of 'may', unless the term is in the UI.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@andrewkroh andrewkroh added New Integration Issue or pull request for creating a new integration package. dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. labels Mar 24, 2026
@muthu-mps
Copy link
Copy Markdown
Contributor

Can you fix the null values in the table?
Screenshot 2026-04-07 at 3 18 10 PM

Comment on lines +576 to +577
- FROM metrics-redisenterprise.otel-default
- WHERE bdb_shard_cpu_user IS NOT NULL
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Medium kibana/redisenterprise_otel-database-performance.yaml:576

The EVAL expression at line 578 adds bdb_shard_cpu_system to bdb_shard_cpu_user, but the WHERE clause only filters on bdb_shard_cpu_user IS NOT NULL. When bdb_shard_cpu_system is NULL, the addition produces NULL and those data points are silently dropped from the chart. Consider adding AND bdb_shard_cpu_system IS NOT NULL to the WHERE clause.

-      - WHERE bdb_shard_cpu_user IS NOT NULL
+      - WHERE bdb_shard_cpu_user IS NOT NULL AND bdb_shard_cpu_system IS NOT NULL
🤖 Copy this AI Prompt to have your agent fix this:
In file packages/redisenterprise_otel/_dev/shared/kibana/redisenterprise_otel-database-performance.yaml around lines 576-577:

The EVAL expression at line 578 adds `bdb_shard_cpu_system` to `bdb_shard_cpu_user`, but the WHERE clause only filters on `bdb_shard_cpu_user IS NOT NULL`. When `bdb_shard_cpu_system` is NULL, the addition produces NULL and those data points are silently dropped from the chart. Consider adding `AND bdb_shard_cpu_system IS NOT NULL` to the WHERE clause.

type: line
query:
- FROM metrics-redisenterprise.otel-default
- WHERE bdb_avg_latency_max IS NOT NULL
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 High kibana/redisenterprise_otel-database-performance.yaml:264

The Max Latency Over Time panel filters on bdb_avg_latency_max IS NOT NULL, but this field doesn't exist — the actual fields are bdb_avg_read_latency_max, bdb_avg_write_latency_max, and bdb_avg_other_latency_max. The query will never match documents, so the panel always displays empty. Consider filtering on bdb_avg_read_latency_max IS NOT NULL instead, matching the pattern used in the non-max latency panel.

Suggested change
- WHERE bdb_avg_latency_max IS NOT NULL
- WHERE bdb_avg_read_latency_max IS NOT NULL
🤖 Copy this AI Prompt to have your agent fix this:
In file packages/redisenterprise_otel/_dev/shared/kibana/redisenterprise_otel-database-performance.yaml around line 264:

The `Max Latency Over Time` panel filters on `bdb_avg_latency_max IS NOT NULL`, but this field doesn't exist — the actual fields are `bdb_avg_read_latency_max`, `bdb_avg_write_latency_max`, and `bdb_avg_other_latency_max`. The query will never match documents, so the panel always displays empty. Consider filtering on `bdb_avg_read_latency_max IS NOT NULL` instead, matching the pattern used in the non-max latency panel.

- FROM metrics-redisenterprise.otel-default
- WHERE bdb_read_hits IS NOT NULL
- STATS hits = AVG(bdb_read_hits), misses = AVG(bdb_read_misses) BY bdb
- EVAL hit_rate = ROUND((hits / (hits + misses)) * 100, 1)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 High kibana/redisenterprise_otel-overview.yaml:304

The EVAL hit_rate formula on line 304 divides by hits + misses without guarding against zero. When a database has no read activity, both values are 0, causing division by zero that produces null/NaN and breaks panel rendering. Consider adding a zero-guard using CASE or ensuring the denominator is non-zero before division.

-      - EVAL hit_rate = ROUND((hits / (hits + misses)) * 100, 1)
+      - EVAL hit_rate = CASE(hits + misses > 0, ROUND((hits / (hits + misses)) * 100, 1), 0)
🤖 Copy this AI Prompt to have your agent fix this:
In file packages/redisenterprise_otel/_dev/shared/kibana/redisenterprise_otel-overview.yaml around line 304:

The `EVAL hit_rate` formula on line 304 divides by `hits + misses` without guarding against zero. When a database has no read activity, both values are 0, causing division by zero that produces null/NaN and breaks panel rendering. Consider adding a zero-guard using `CASE` or ensuring the denominator is non-zero before division.

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

@muthu-mps muthu-mps requested a review from ishleenk17 April 9, 2026 04:25
@ishleenk17 ishleenk17 merged commit 19f916e into elastic:main Apr 9, 2026
12 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

@andrewkroh andrewkroh added the Integration:redisenterprise_otel Redis Enterprise OpenTelemetry Assets label Apr 9, 2026
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. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:redisenterprise_otel Redis Enterprise OpenTelemetry Assets New Integration Issue or pull request for creating a new integration package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants