Skip to content

feat: [UIE-9051] - IAM RBAC block non-beta route access - #12656

Merged
abailly-akamai merged 9 commits into
linode:developfrom
corya-akamai:UIE-9051-block-non-beta-iam-alt
Aug 13, 2025
Merged

feat: [UIE-9051] - IAM RBAC block non-beta route access#12656
abailly-akamai merged 9 commits into
linode:developfrom
corya-akamai:UIE-9051-block-non-beta-iam-alt

Conversation

@corya-akamai

Copy link
Copy Markdown
Contributor

Description 📝

Block non-beta users from the IAM screens and redirect to the account screens accordingly. Redirect beta users from the account screens to the IAM screens respectively.

Changes 🔄

  • Block non-beta users from the IAM screens and redirect to the account screens accordingly.
  • Redirect beta users from the account screens to the IAM screens respectively.

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

Target release date 🗓️

Preview 📷

re-routes.mov

How to test 🧪

Prerequisites

  • Devcloud IAM account
  • Devcloud non-IAM account

Reproduction steps

Navigate to routes

  • /iam/users <-> /account/users
  • /iam/roles <-> (no equivalent) /account/users
  • /iam/users/corya-iam/details <-> /account/users/corya-iam/profile
  • /iam/users/corya-iam/roles <-> /account/users/corya-iam/permissions
  • /iam/users/corya-iam/entities <-> /account/users/corya-iam/permissions

Verification steps

Verify redirect

  • /iam/users <-> /account/users
  • /iam/roles <-> (no equivalent) /account/users
  • /iam/users/corya-iam/details <-> /account/users/corya-iam/profile
  • /iam/users/corya-iam/roles <-> /account/users/corya-iam/permissions
  • /iam/users/corya-iam/entities <-> /account/users/corya-iam/permissions
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

Comment thread packages/manager/src/features/IAM/hooks/useIsIAMEnabled.ts
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🔺 2 failing tests on test run #5 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
2 Failing704 Passing4 Skipped126m 54s

Details

Failing Tests
SpecTest
qemu-reboot-upgrade-notice.spec.tsCloud Manager Cypress Tests→QEMU reboot upgrade notification » should display maintenance banner in 'Linode' landing page when one or more Linodes get impacted.
smoke-linode-landing-table.spec.tsCloud Manager Cypress Tests→linode landing checks » checks the landing page side menu items

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/notificationsAndEvents/qemu-reboot-upgrade-notice.spec.ts,cypress/e2e/core/linodes/smoke-linode-landing-table.spec.ts"

@aaleksee-akamai aaleksee-akamai added the Add'tl Approval Needed Waiting on another approval! label Aug 13, 2025

@kwojtowiakamai kwojtowiakamai left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Block non-beta users from the IAM screens and redirect to the account screens accordingly.
✅ Redirect beta users from the account screens to the IAM screens respectively.

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Aug 13, 2025
@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Aug 13, 2025

@coliu-akamai coliu-akamai 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.

✅ routes with a devcloud IAM account - stayed on iam routes
✅ routes with a devcloud non-iam account - redirected
✅ routes with a devcloud IAM account but IAM feature flags off - redirected

thanks @corya-akamai!

@abailly-akamai
abailly-akamai merged commit 2dffce0 into linode:develop Aug 13, 2025
34 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Aug 13, 2025
@corya-akamai
corya-akamai deleted the UIE-9051-block-non-beta-iam-alt branch September 10, 2025 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! IAM (Identity & Access Management)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

7 participants