[Docs] Illustrate cloud/identity/organizations with Orbital Labs and Stellar Dynamics examples#983
Conversation
|
There was a problem hiding this comment.
Code Review
This pull request establishes a continuous narrative for the Layer5 Cloud documentation, introducing a cast of characters and organizations to provide consistent examples across the site. The changes include a new 'About' section, a research plan for further integration, and the addition of mascot assets. My feedback focuses on improving the portability of the implementation plan by removing hardcoded absolute paths and ensuring consistency in character and team naming across the design specifications.
| - [ ] **Step 1: Create the target directory** | ||
|
|
||
| ```bash | ||
| mkdir -p /Users/l/code/docs/static/images/five |
There was a problem hiding this comment.
This file contains multiple shell commands with hardcoded absolute paths, such as /Users/l/code/docs. This practice is not portable and tightly couples the plan to a specific user's machine setup. It also leaks user and directory information. These paths should be replaced with relative paths (e.g., assuming commands are run from the repo root) or environment variables to make the plan generic and usable by any contributor.
| mkdir -p /Users/l/code/docs/static/images/five | |
| mkdir -p static/images/five |
| | `orbital-production` | Orbital Labs | Infrastructure | Prod access controls, environment assignment | | ||
| | `orbital-staging` | Orbital Labs | Infrastructure, Development | Cross-team workspace sharing | | ||
| | `orbital-dev` | Orbital Labs | Development | Team-scoped workspace, design iteration | | ||
| | `stellar-main` | Stellar Dynamics | Marcus's team | Enterprise org, cross-org scenarios | |
There was a problem hiding this comment.
The team with access to the stellar-main workspace is listed as "Marcus's team". For consistency with the 'Team & Org Hierarchy' table in this same document (line 94), this should be specified as the 'Platform' team.
| | `stellar-main` | Stellar Dynamics | Marcus's team | Enterprise org, cross-org scenarios | | |
| | `stellar-main` | Stellar Dynamics | Platform | Enterprise org, cross-org scenarios | |
| | `cloud/identity/users` | Five, Maya | Five's profile card; Maya as Org Admin | SVG 1 (meet Five) | Low | Open | | ||
| | `cloud/identity/organizations` | Maya, Marcus | Orbital Labs org creation; Stellar Dynamics as enterprise tenant | — | Low | Open | | ||
| | `cloud/identity/teams` | Zara, Rex, Jordan | Infrastructure vs. Dev team setup | team-of-fives.svg | Low | Open | | ||
| | `cloud/security/roles` | Dr. Aiko, Zara | Provider Admin → Org Admin → Team Admin chain | business-mode.svg | Medium | Open | |
There was a problem hiding this comment.
There's an inconsistency in the 'Integration Map'. For the cloud/security/roles section, the characters are listed as 'Dr. Aiko, Zara', but the scenario is 'Provider Admin → Org Admin → Team Admin chain'. According to the cast list, Maya Chen is the Org Admin. The character list should likely be 'Dr. Aiko, Maya, Zara' to accurately reflect the roles in the scenario.
| | `cloud/security/roles` | Dr. Aiko, Zara | Provider Admin → Org Admin → Team Admin chain | business-mode.svg | Medium | Open | | |
| | `cloud/security/roles` | Dr. Aiko, Maya, Zara | Provider Admin → Org Admin → Team Admin chain | business-mode.svg | Medium | Open | |
2ee7250 to
8d052af
Compare
…s examples (#968) Signed-off-by: Lee Calcote <lee.calcote@layer5.io>
8d052af to
c0a86ce
Compare
Closes #968
Adds an 'Example: The Orbital Labs Ecosystem' section to
cloud/identity/organizationsillustrating multi-tenancy across three tiers: Constellation Cloud (MSP/provider), Orbital Labs (startup tenant), and Stellar Dynamics (enterprise tenant). Includes a cross-org access explainer and link to the narrative reference.Depends on: #979 (narrative page)
Changes
/cloud/aboutnarrative referenceTest plan
hugo --quietbuilds clean/cloud/identity/organizations//cloud/aboutworks (after [Docs] Establish Five and Orbital Labs as continuous docs narrative (issue #457) #979 merges)