Skip to content

[RDSC-4241] Improving observability documentation#2343

Merged
andy-stark-redis merged 8 commits intoredis:mainfrom
ilianiliev-redis:RDSC-4241-updating-rdi-observability-docs
Mar 4, 2026
Merged

[RDSC-4241] Improving observability documentation#2343
andy-stark-redis merged 8 commits intoredis:mainfrom
ilianiliev-redis:RDSC-4241-updating-rdi-observability-docs

Conversation

@ilianiliev-redis
Copy link
Contributor

@ilianiliev-redis ilianiliev-redis commented Nov 4, 2025

Ticket: https://redislabs.atlassian.net/browse/RDSC-4241

IMPORTANT: Do not merge or delete before the merge of https://github.com/RedisLabs/redis-data-integration/pull/2288 and the release of the respectful RDI version


Note

Low Risk
Documentation-only updates that change how users are instructed to access RDI metrics endpoints; no product code or runtime behavior is modified.

Overview
Adds a new “Accessing the metrics” section that distinguishes VM vs Helm installs, documents the correct metrics endpoints for each component, and provides Helm values.yaml examples to enable Prometheus ServiceMonitors (with a note about discovery delays and a pre-1.16.0 collector caveat).

Removes the old per-section endpoint callouts for collector/stream processor, and updates the operator metrics section to note most metrics are standard controller-runtime metrics while highlighting the RDI-specific ones.

Written by Cursor Bugbot for commit 99540b6. This will update automatically on new commits. Configure here.

@jit-ci
Copy link

jit-ci bot commented Nov 4, 2025

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

A few suggestions, mainly to fit better with docs heading structure and markup conventions. Otherwise, LGTM.

ilianiliev-redis and others added 8 commits March 4, 2026 11:08
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
@ilianiliev-redis ilianiliev-redis force-pushed the RDSC-4241-updating-rdi-observability-docs branch from 08deb40 to 99540b6 Compare March 4, 2026 10:41
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

LGTM

@andy-stark-redis andy-stark-redis merged commit 2ee28ab into redis:main Mar 4, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants