Skip to content

upcoming: [UIE-9301] - IAM Parent/Child - Default roles & Entity access routes#12981

Merged
abailly-akamai merged 10 commits intolinode:developfrom
abailly-akamai:UIE-9301
Oct 14, 2025
Merged

upcoming: [UIE-9301] - IAM Parent/Child - Default roles & Entity access routes#12981
abailly-akamai merged 10 commits intolinode:developfrom
abailly-akamai:UIE-9301

Conversation

@abailly-akamai
Copy link
Contributor

@abailly-akamai abailly-akamai commented Oct 13, 2025

Description 📝

This PR adds routing and basic components for the default roles and default entity access section of the delegation UI.

It only applies to Child Accounts (and if the delegation flag is enabled).

It does not add any true functionality, other then routing and gating the new section.

Changes 🔄

  • Add routing:
    • /iam/roles/defaults/roles
    • /iam/roles/defaults/entity-access
  • Add Basic placeholder components
  • Configure feature gating

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Preview 📷

Screenshot 2025-10-13 at 14 21 10

Screenshot 2025-10-13 at 14 21 17

How to test 🧪

Prerequisites

  • Have IAM enabled Parent/Child accounts
  • Have the iamDelegation?.enabled feature flag turned ON

Verification steps

👶 As a Child Account

  • Navigate to /iam/roles
    • ✅ Confirm the new "Default Roles for Delegate Users" panel
    • Confirm Call to action sends to iam/roles/defaults/roles
  • On /iam/roles/defaults/roles
    • ✅ Confirm both tabs + routing
  • Point browser to /iam/roles/default
    • ✅ Confirm redirect to /iam/roles/defaults/roles

👴 As a Parent Account

  • Navigate to /iam/roles
    • ✅ Confirm no "Default Roles for Delegate Users" panel is rendered
  • Point browser to /iam/roles/defaults, /iam/roles/defaults/roles and /iam/roles/defaults/entity-access
    • ✅ Confirm they all redirect to /iam/roles

👶👴 Either Account
Have the iamDelegation?.enabled feature flag turned OFF

  • Point browser to /iam/roles/defaults, /iam/roles/defaults/roles and /iam/roles/defaults/entity-access
    • ✅ Confirm they all redirect to /iam/roles
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support

  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@abailly-akamai abailly-akamai marked this pull request as ready for review October 13, 2025 12:48
@abailly-akamai abailly-akamai requested a review from a team as a code owner October 13, 2025 12:48
@abailly-akamai abailly-akamai changed the title [UIE-9301] upcoming: [UIE-9301] - IAM Parent/Child - Default roles & Entity access routes Oct 13, 2025
isACLPEnabled,
isDatabasesEnabled,
isPlacementGroupsEnabled,
profile,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since we will be relying more and more on user types, it's very helpful to have the profile in the route context to be able to gate content at the routing level.

@abailly-akamai abailly-akamai removed the request for review from a team October 13, 2025 13:23
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 2 failing tests on test run #7 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
2 Failing836 Passing11 Skipped45m 40s

Details

Failing Tests
SpecTest
timerange-verification.spec.tsCloud Manager Cypress Tests→Integration tests for verifying Cloudpulse custom and preset configurations » should implement and validate custom date/time picker for a specific date and time range
clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » can clone a Linode from Linode details page

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/cloudpulse/timerange-verification.spec.ts,cypress/e2e/core/linodes/clone-linode.spec.ts"

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Oct 14, 2025
Default Entity Access for Delegate Users
</Typography>
<Typography marginTop={2}>
View and update entities assigned to delegate users by default. Note
Copy link
Contributor

Choose a reason for hiding this comment

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

seems like that text is the same for both tabs, shouldn't be as a constant then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not exactly the same (roles VS entities) so i'll leave it as such for now. Not too big a deal

@abailly-akamai abailly-akamai merged commit e58707f into linode:develop Oct 14, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Oct 14, 2025
corya-akamai pushed a commit to corya-akamai/manager that referenced this pull request Dec 8, 2025
…ss routes (linode#12981)

* save progress

* better paths

* routing

* basic copy

* default route and cleanup

* better gating

* some coverage

* Added changeset: IAM Parent/Child - Default roles & Entity access routes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IAM Delegation Implementing the new Parent/Child delegation UI IAM (Identity & Access Management)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants