Skip to content

Fix Task Completion Time Without Backpressure Grafana panel metric name#60481

Merged
bveeramani merged 3 commits intoray-project:masterfrom
kriyanshii:fix-data-dashboard-task-completion-time-panel
Jan 28, 2026
Merged

Fix Task Completion Time Without Backpressure Grafana panel metric name#60481
bveeramani merged 3 commits intoray-project:masterfrom
kriyanshii:fix-data-dashboard-task-completion-time-panel

Conversation

@kriyanshii
Copy link
Contributor

Description

Fixes the broken "Task Completion Time Without Backpressure" metrics chart in the Ray Data Grafana dashboard.

The panel was querying ray_data_task_completion_time_without_backpressure, which no longer exists. PR #57788 renamed the underlying metric to task_completion_time_excl_backpressure_s in op_runtime_metrics.py, but the Grafana panel in data_dashboard_panels.py was not updated. This PR updates the panel's Prometheus expr to use ray_data_task_completion_time_excl_backpressure_s so the chart displays data again.

Change: Single-line fix in data_dashboard_panels.py — replace the old metric name with the correct one in the panel's expr. The formula (average task completion time excluding backpressure over a 5-minute window) is unchanged.

Related issues

Fixes the regression from #57788 (metric rename). Related to Ray Data monitoring / dashboard.
Closes: #60163

Additional information

  • Metric flow: op_runtime_metrics.task_completion_time_excl_backpressure_s → Stats uses data_{name} → Metrics agent adds ray_ namespace → ray_data_task_completion_time_excl_backpressure_s
  • Manual verification: Run a Ray Data job with Grafana + Prometheus (see cluster metrics), then confirm the "Task Completion Time Without Backpressure" panel shows data.

@kriyanshii kriyanshii requested a review from a team as a code owner January 25, 2026 06:30
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly fixes a broken Grafana panel by updating the metric name for 'Task Completion Time Without Backpressure'. The change replaces the old metric name ray_data_task_completion_time_without_backpressure with the new one, ray_data_task_completion_time_excl_backpressure_s, which aligns with the recent metric renaming in op_runtime_metrics.py. The fix is accurate and addresses the issue described. The code change is clean and no further issues were found.

Update the panel's Prometheus expr to use ray_data_task_completion_time_excl_backpressure_s instead of ray_data_task_completion_time_without_backpressure. The metric was renamed in ray-project#57788 (op_runtime_metrics) but the data_dashboard_panels.py panel was not updated, causing the chart to show no data.

Signed-off-by: kriyanshii <[email protected]>
@kriyanshii kriyanshii force-pushed the fix-data-dashboard-task-completion-time-panel branch from 99dd985 to 9d5a78a Compare January 25, 2026 06:33
@ray-gardener ray-gardener bot added data Ray Data-related issues observability Issues related to the Ray Dashboard, Logging, Metrics, Tracing, and/or Profiling community-contribution Contributed by the community labels Jan 25, 2026
Copy link
Member

@bveeramani bveeramani left a comment

Choose a reason for hiding this comment

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

ty!

@bveeramani
Copy link
Member

@kriyanshii did you get a chance to build the dashboard locally and verify if the chart displays correctly?

@kriyanshii
Copy link
Contributor Author

@kriyanshii did you get a chance to build the dashboard locally and verify if the chart displays correctly?

Yes. The metrics are there after the code change.

image

@bveeramani bveeramani enabled auto-merge (squash) January 27, 2026 22:46
@bveeramani
Copy link
Member

Nice. Just enabled merge

@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Jan 27, 2026
@bveeramani bveeramani merged commit b2be3a3 into ray-project:master Jan 28, 2026
8 checks passed
jinbum-kim pushed a commit to jinbum-kim/ray that referenced this pull request Jan 29, 2026
…tric name (ray-project#60481)

## Description
Fixes the broken **"Task Completion Time Without Backpressure"** metrics
chart in the Ray Data Grafana dashboard.

The panel was querying
`ray_data_task_completion_time_without_backpressure`, which no longer
exists. PR ray-project#57788 renamed the underlying metric to
`task_completion_time_excl_backpressure_s` in `op_runtime_metrics.py`,
but the Grafana panel in `data_dashboard_panels.py` was not updated.
This PR updates the panel's Prometheus `expr` to use
`ray_data_task_completion_time_excl_backpressure_s` so the chart
displays data again.

**Change:** Single-line fix in `data_dashboard_panels.py` — replace the
old metric name with the correct one in the panel's `expr`. The formula
(average task completion time excluding backpressure over a 5-minute
window) is unchanged.

## Related issues
Fixes the regression from ray-project#57788 (metric rename). Related to Ray Data
monitoring / dashboard.
Closes: ray-project#60163

## Additional information
- **Metric flow:**
`op_runtime_metrics.task_completion_time_excl_backpressure_s` → Stats
uses `data_{name}` → Metrics agent adds `ray_` namespace →
**`ray_data_task_completion_time_excl_backpressure_s`**
- **Manual verification:** Run a Ray Data job with Grafana + Prometheus
(see [cluster
metrics](https://docs.ray.io/en/latest/cluster/metrics.html)), then
confirm the "Task Completion Time Without Backpressure" panel shows
data.

Signed-off-by: kriyanshii <[email protected]>
Signed-off-by: jinbum-kim <[email protected]>
limarkdcunha pushed a commit to limarkdcunha/ray that referenced this pull request Jan 29, 2026
…tric name (ray-project#60481)

## Description
Fixes the broken **"Task Completion Time Without Backpressure"** metrics
chart in the Ray Data Grafana dashboard.

The panel was querying
`ray_data_task_completion_time_without_backpressure`, which no longer
exists. PR ray-project#57788 renamed the underlying metric to
`task_completion_time_excl_backpressure_s` in `op_runtime_metrics.py`,
but the Grafana panel in `data_dashboard_panels.py` was not updated.
This PR updates the panel's Prometheus `expr` to use
`ray_data_task_completion_time_excl_backpressure_s` so the chart
displays data again.

**Change:** Single-line fix in `data_dashboard_panels.py` — replace the
old metric name with the correct one in the panel's `expr`. The formula
(average task completion time excluding backpressure over a 5-minute
window) is unchanged.

## Related issues
Fixes the regression from ray-project#57788 (metric rename). Related to Ray Data
monitoring / dashboard.
Closes: ray-project#60163 

## Additional information
- **Metric flow:**
`op_runtime_metrics.task_completion_time_excl_backpressure_s` → Stats
uses `data_{name}` → Metrics agent adds `ray_` namespace →
**`ray_data_task_completion_time_excl_backpressure_s`**
- **Manual verification:** Run a Ray Data job with Grafana + Prometheus
(see [cluster
metrics](https://docs.ray.io/en/latest/cluster/metrics.html)), then
confirm the "Task Completion Time Without Backpressure" panel shows
data.

Signed-off-by: kriyanshii <[email protected]>
400Ping pushed a commit to 400Ping/ray that referenced this pull request Feb 1, 2026
…tric name (ray-project#60481)

## Description
Fixes the broken **"Task Completion Time Without Backpressure"** metrics
chart in the Ray Data Grafana dashboard.

The panel was querying
`ray_data_task_completion_time_without_backpressure`, which no longer
exists. PR ray-project#57788 renamed the underlying metric to
`task_completion_time_excl_backpressure_s` in `op_runtime_metrics.py`,
but the Grafana panel in `data_dashboard_panels.py` was not updated.
This PR updates the panel's Prometheus `expr` to use
`ray_data_task_completion_time_excl_backpressure_s` so the chart
displays data again.

**Change:** Single-line fix in `data_dashboard_panels.py` — replace the
old metric name with the correct one in the panel's `expr`. The formula
(average task completion time excluding backpressure over a 5-minute
window) is unchanged.

## Related issues
Fixes the regression from ray-project#57788 (metric rename). Related to Ray Data
monitoring / dashboard.
Closes: ray-project#60163

## Additional information
- **Metric flow:**
`op_runtime_metrics.task_completion_time_excl_backpressure_s` → Stats
uses `data_{name}` → Metrics agent adds `ray_` namespace →
**`ray_data_task_completion_time_excl_backpressure_s`**
- **Manual verification:** Run a Ray Data job with Grafana + Prometheus
(see [cluster
metrics](https://docs.ray.io/en/latest/cluster/metrics.html)), then
confirm the "Task Completion Time Without Backpressure" panel shows
data.

Signed-off-by: kriyanshii <[email protected]>
Signed-off-by: 400Ping <[email protected]>
ans9868 pushed a commit to ans9868/ray that referenced this pull request Feb 18, 2026
…tric name (ray-project#60481)

## Description
Fixes the broken **"Task Completion Time Without Backpressure"** metrics
chart in the Ray Data Grafana dashboard.

The panel was querying
`ray_data_task_completion_time_without_backpressure`, which no longer
exists. PR ray-project#57788 renamed the underlying metric to
`task_completion_time_excl_backpressure_s` in `op_runtime_metrics.py`,
but the Grafana panel in `data_dashboard_panels.py` was not updated.
This PR updates the panel's Prometheus `expr` to use
`ray_data_task_completion_time_excl_backpressure_s` so the chart
displays data again.

**Change:** Single-line fix in `data_dashboard_panels.py` — replace the
old metric name with the correct one in the panel's `expr`. The formula
(average task completion time excluding backpressure over a 5-minute
window) is unchanged.

## Related issues
Fixes the regression from ray-project#57788 (metric rename). Related to Ray Data
monitoring / dashboard.
Closes: ray-project#60163

## Additional information
- **Metric flow:**
`op_runtime_metrics.task_completion_time_excl_backpressure_s` → Stats
uses `data_{name}` → Metrics agent adds `ray_` namespace →
**`ray_data_task_completion_time_excl_backpressure_s`**
- **Manual verification:** Run a Ray Data job with Grafana + Prometheus
(see [cluster
metrics](https://docs.ray.io/en/latest/cluster/metrics.html)), then
confirm the "Task Completion Time Without Backpressure" panel shows
data.

Signed-off-by: kriyanshii <[email protected]>
Signed-off-by: Adel Nour <[email protected]>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
…tric name (ray-project#60481)

## Description
Fixes the broken **"Task Completion Time Without Backpressure"** metrics
chart in the Ray Data Grafana dashboard.

The panel was querying
`ray_data_task_completion_time_without_backpressure`, which no longer
exists. PR ray-project#57788 renamed the underlying metric to
`task_completion_time_excl_backpressure_s` in `op_runtime_metrics.py`,
but the Grafana panel in `data_dashboard_panels.py` was not updated.
This PR updates the panel's Prometheus `expr` to use
`ray_data_task_completion_time_excl_backpressure_s` so the chart
displays data again.

**Change:** Single-line fix in `data_dashboard_panels.py` — replace the
old metric name with the correct one in the panel's `expr`. The formula
(average task completion time excluding backpressure over a 5-minute
window) is unchanged.

## Related issues
Fixes the regression from ray-project#57788 (metric rename). Related to Ray Data
monitoring / dashboard.
Closes: ray-project#60163

## Additional information
- **Metric flow:**
`op_runtime_metrics.task_completion_time_excl_backpressure_s` → Stats
uses `data_{name}` → Metrics agent adds `ray_` namespace →
**`ray_data_task_completion_time_excl_backpressure_s`**
- **Manual verification:** Run a Ray Data job with Grafana + Prometheus
(see [cluster
metrics](https://docs.ray.io/en/latest/cluster/metrics.html)), then
confirm the "Task Completion Time Without Backpressure" panel shows
data.

Signed-off-by: kriyanshii <[email protected]>
Signed-off-by: peterxcli <[email protected]>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
…tric name (ray-project#60481)

## Description
Fixes the broken **"Task Completion Time Without Backpressure"** metrics
chart in the Ray Data Grafana dashboard.

The panel was querying
`ray_data_task_completion_time_without_backpressure`, which no longer
exists. PR ray-project#57788 renamed the underlying metric to
`task_completion_time_excl_backpressure_s` in `op_runtime_metrics.py`,
but the Grafana panel in `data_dashboard_panels.py` was not updated.
This PR updates the panel's Prometheus `expr` to use
`ray_data_task_completion_time_excl_backpressure_s` so the chart
displays data again.

**Change:** Single-line fix in `data_dashboard_panels.py` — replace the
old metric name with the correct one in the panel's `expr`. The formula
(average task completion time excluding backpressure over a 5-minute
window) is unchanged.

## Related issues
Fixes the regression from ray-project#57788 (metric rename). Related to Ray Data
monitoring / dashboard.
Closes: ray-project#60163

## Additional information
- **Metric flow:**
`op_runtime_metrics.task_completion_time_excl_backpressure_s` → Stats
uses `data_{name}` → Metrics agent adds `ray_` namespace →
**`ray_data_task_completion_time_excl_backpressure_s`**
- **Manual verification:** Run a Ray Data job with Grafana + Prometheus
(see [cluster
metrics](https://docs.ray.io/en/latest/cluster/metrics.html)), then
confirm the "Task Completion Time Without Backpressure" panel shows
data.

Signed-off-by: kriyanshii <[email protected]>
Signed-off-by: peterxcli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Contributed by the community data Ray Data-related issues go add ONLY when ready to merge, run all tests observability Issues related to the Ray Dashboard, Logging, Metrics, Tracing, and/or Profiling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Data] "Task Completion Time Without Backpressure" Grafana panel is broken

2 participants