Skip to content

PMM-15240: enable All on Cluster filter in PostgreSQL Replication Overview - #5727

Open
mattiasimonato wants to merge 5 commits into
mainfrom
PMM-15240-fix-pg-replication-cluster-all
Open

PMM-15240: enable All on Cluster filter in PostgreSQL Replication Overview#5727
mattiasimonato wants to merge 5 commits into
mainfrom
PMM-15240-fix-pg-replication-cluster-all

Conversation

@mattiasimonato

@mattiasimonato mattiasimonato commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Ticket number: PMM-15240

Feature build: SUBMODULES-4505

  • Enable includeAll on the Cluster template variable and default it to All.
  • Stops the dashboard from being locked to one Patroni cluster so standalone (non-Patroni) PostgreSQL nodes can appear in Node Name / Service Name and replication panels.

Summary by CodeRabbit

  • New Features
    • The PostgreSQL Replication Overview dashboard now supports viewing data for all clusters by default.
    • Added the ability to select all clusters from the dashboard filter.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly states that the Cluster filter now supports the All option in the PostgreSQL Replication Overview dashboard.
Description check ✅ Passed The description includes the required ticket and feature build fields and clearly explains the dashboard change and its purpose.

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37.03%. Comparing base (31318c7) to head (ed7deb6).
⚠️ Report is 91 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5727      +/-   ##
==========================================
- Coverage   43.59%   37.03%   -6.56%     
==========================================
  Files         415      215     -200     
  Lines       43134     7168   -35966     
  Branches        0      585     +585     
==========================================
- Hits        18804     2655   -16149     
+ Misses      22454     4317   -18137     
+ Partials     1876      196    -1680     
Flag Coverage Δ
admin 34.96% <ø> (+0.17%) ⬆️
agent ?
managed ?
unittests 41.29% <ø> (?)
vmproxy ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mattiasimonato
mattiasimonato marked this pull request as ready for review August 4, 2026 06:38
@mattiasimonato
mattiasimonato requested review from a team and Nailya as code owners August 4, 2026 06:38
@mattiasimonato
mattiasimonato requested review from fabio-silva and matejkubinec and removed request for a team August 4, 2026 06:38
@mattiasimonato mattiasimonato added the auto-update-branch used by .github/workflows/auto-update-base.yaml label Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-update-branch used by .github/workflows/auto-update-base.yaml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants