Rack groups and other small changes - #9
Merged
Merged
Conversation
- Rack.kind 'three-rack' becomes 'rack-group' (no migration, the format is still alpha); a group's `name` names the group and the new `memberNames` name its left, middle and right physical racks. - New racks take the next free "Rack <N>" above the highest in use (a group takes three), and switching a rack's kind keeps names unique. Validation warns about duplicate physical rack names. - The racks view draws a rack group's three elevations in one dashed box with the group name, like the topology already does. - New server groups default to the current MicroCloud H13 instead of the end-of-life X11. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The rack header keeps the name, rack type and, for a rack group, its member names as read-only text. The leaf model and count and a group's member name inputs move into Advanced next to height and power budget. Rack issues that can only be fixed there (height, power, a group's duplicate rack name, leaf model and count) carry target.field 'advanced'; clicking one unfolds the section, and its summary shows their badges. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The central rack keeps the counts (spines, exits, routers, superspines, storage leaves) and the management network's layer and redundancy in view, with the chosen models as a read-only line under the heading. The model pickers and mgmt leaves per rack move into Advanced, which is now grouped into hardware models, fabric links and rack defaults. Model issues and the mgmt spine fiber warning carry target.field 'advanced', so clicking one unfolds the central rack's Advanced too. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The central rack's visible fields are now fabric type, spines, exits (plus superspines with that tier) and the management network's layer and redundancy. Internet routers and storage leaves join the models in Advanced, whose first group becomes "Devices and models". In a rack section, "Add server group" now sits right below the server groups, above the Advanced section. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- A rack section's leaf ports, fabric ratio, issues and node count move
to the right of its header row, as in the central rack. The chips are
shorter ("Leaf ports 58 / 60", "115 nodes") with the details in hover
hints, so a rack group's header fits on one line.
- No long dashes in any UI text: prose uses a colon, comma or
parentheses, section subtitles a middle dot, empty cells an en dash.
src/uiText.test.ts fails on any outside comments.
- New README screenshots of the current UI (rack groups, folded
Advanced sections, current hardware) with updated captions.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Templates, Export JSON, Import JSON and Reset now form a right-aligned toolbar above the plan name and topology instead of sharing their row. The README's plan screenshot is retaken. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The spread test from #8 still set kind 'three-rack', which this branch renamed to 'rack-group'; build it with withRackKind like its siblings. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Used AI-Tools ✨