Skip to content

Label replicated worker series as “Pholid Worker N” in container CPU/memory panels - #78

Merged
M-Nikox merged 3 commits into
masterfrom
copilot/update-worker-names
Apr 5, 2026
Merged

Label replicated worker series as “Pholid Worker N” in container CPU/memory panels#78
M-Nikox merged 3 commits into
masterfrom
copilot/update-worker-names

Conversation

Copilot AI commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Container CPU/Memory panels were showing raw Docker Compose container names for worker replicas (e.g. {name="...flamenco-worker-1"}), which made legends noisy and inconsistent with worker-specific panels. This updates legend labeling so worker replicas render as Pholid Worker 1/2.

Changes

  • Container Memory Usage panel
    • Extended label_replace chain to map worker replica names to friendly labels.
  • Container CPU Usage panel
    • Applied the same worker mapping for CPU series to keep naming consistent.
  • No scope expansion
    • Only the two Grafana panel expressions were updated.
label_replace(
  ...,
  "display_name", "Pholid Worker 1", "name", ".*pholid.*flamenco-worker-1.*"
)

Affected areas

  • Upload / render form
  • Live sessions panel
  • Previous jobs panel
  • Metrics / Grafana tab
  • Theme system
  • Modals
  • Header / navigation
  • CSS / layout
  • JavaScript modules

Screenshots

Before/after available in the issue discussion image reference.

Theme testing

  • Light
  • Dark
  • High Contrast
  • Midnight

Checklist

  • Tested in browser
  • No hardcoded colors (use CSS variables)
  • No inline styles where a CSS class would do
  • JS events use pholid: namespace prefix

@M-Nikox
M-Nikox marked this pull request as ready for review April 4, 2026 18:37
Copilot AI review requested due to automatic review settings April 4, 2026 18:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Grafana “Pholid Render Farm” dashboard so container CPU/Memory series for Flamenco worker replicas display friendly legend names (“Pholid Worker N”) instead of raw Docker Compose container names.

Changes:

  • Added worker-replica name → display label mapping in the Container Memory Usage PromQL expression.
  • Added the same worker-replica mapping in the Container CPU Usage PromQL expression to keep legends consistent.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread monitoring/grafana/dashboards/pholid-render-farm.json Outdated
Comment thread monitoring/grafana/dashboards/pholid-render-farm.json Outdated
Base automatically changed from copilot/analyze-error-handling to master April 4, 2026 18:53
M-Nikox and others added 2 commits April 4, 2026 20:59
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@M-Nikox
M-Nikox merged commit 7f7a843 into master Apr 5, 2026
6 checks passed
@M-Nikox
M-Nikox deleted the copilot/update-worker-names branch April 5, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants