Skip to content

Modernize Orchestrator web UI and audit workflows - #122

Merged
renecannao merged 57 commits into
masterfrom
codex/ui-restorative-topology-worktree
Aug 19, 2026
Merged

Modernize Orchestrator web UI and audit workflows#122
renecannao merged 57 commits into
masterfrom
codex/ui-restorative-topology-worktree

Conversation

@renecannao

@renecannao renecannao commented Aug 12, 2026

Copy link
Copy Markdown

Pull Request

Related issue: none

Description

Modernizes Orchestrator's legacy web interface while preserving its operational behavior and existing API contracts.

This PR:

  • restores a clear clusters landing page and topology workspace with semantic instance cards, responsive graph sizing, and working command interactions;
  • replaces broken or mismatched framework assets with locally served Bootstrap 5 assets and scopes the new workspace CSS to avoid global regressions;
  • rebuilds failure-analysis, audit, recovery, status, agents, seeds, discovery, About, FAQ, and navigation surfaces into consistent operational workspaces;
  • updates project and documentation links for the ProxySQL Orchestrator repository and current MySQL/PostgreSQL support;
  • adds functional audit persistence and a bounded failover harness for the Docker lab;
  • hardens lab cleanup so it only restores mysql1 when the harness actually stopped it, and never broadly recreates MySQL services;
  • preserves the historical begin-maintenance response details while adding the exact MaintenanceKey used for safe cleanup.

The UI was exercised against the local three-node MySQL lab on desktop and narrow viewports, including populated audit, failure-detection, recovery-list, and recovery-detail states after controlled failovers.

Validation

  • go test ./... -count=1: all Go packages passed
  • Node UI tests: 26/26 passed across all six test files
  • functional smoke: 50 passed / 0 failed / 0 skipped
  • shell syntax checks and the combined functional/MariaDB Compose configuration passed
  • committed browser QA: 45/45 route/viewport entries passed at 1440×900, 794×900, and 390×844, with zero body overflow and zero new warning/error console entries
  • topology/browser interactions passed: collapse/expand, single View and Details activation, drag exclusion, modal dismissal, responsive navigation, audit pagination, and narrow internal canvas scrolling
  • malformed partial failure-analysis payloads now render the explicit unavailable state; audit/recovery API text is DOM-safe; one-row recovery lists and proxy-prefixed pagination have focused regression coverage\n- dynamic Bootstrap controls passed focused mutation coverage and live post-load alert dismissal (one activation, no duplicate action, and no console error)\n- controlled failover and final lab checks completed with topology restoration, unchanged MySQL container identities and roles, and no ProxySQL recreation
  • git diff --check origin/master...HEAD and git show --check --oneline HEAD passed
  • scoped final reviews found no Critical or Important issues

Related UI PRs

  • PR feat(ui): Bootstrap 5 polish — icons, D3 v7, asset cache-busting #125's local Bootstrap Icons, shared asset cache policy, Bootstrap 5 compatibility behavior, and D3 v7 topology work were reapplied deliberately. The integration also fixes the asset-version render order, duplicate modal activation, tree-coordinate restoration, agent grid structure, and icon-only accessibility findings; the overlapping PR was not merged wholesale.
  • PR UI/UX: Fix layout rendering issues in web interface #126's reliable local-asset and working-navigation intent is satisfied while retaining the approved Bootstrap 5 interface and current ProxySQL documentation/repository links. Its Bootstrap 3/IE8 and Openark rollback was not adopted.

Checklist

  • Code formatted with gofmt (please avoid goimports)
  • Tests added/updated
  • CI passes (unit, integration, system tests)
  • DCO sign-off included (git commit -s)
  • Related issue linked above

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 874431d0-df76-4e48-82c6-bb96444e3ee1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

…e-topology-worktree

# Conflicts:
#	go/http/render_test.go
@renecannao
renecannao marked this pull request as ready for review August 19, 2026 06:29
@renecannao
renecannao merged commit 6191b95 into master Aug 19, 2026
18 checks passed
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.

1 participant