feat: [UIE-9534] - IAM: fix perm for nodebalancer by aaleksee-akamai · Pull Request #13058 · linode/manager

aaleksee-akamai · 2025-11-05T13:04:38Z

Description 📝

This PR adds the tooltips for the disabled buttons, removes a notification banner

Changes 🔄

List any change(s) relevant to the reviewer.

Add tooltips
Remove notification banner

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 🗓️

11/18

Preview 📷

Before	After

How to test 🧪

Prerequisites

(How to setup test environment)

use iam account (not account_admin) with nodebalancer_viewer

Reproduction steps

(How to reproduce the issue, if applicable)

go to nodebalancers/[nodebalanceID]/summary
go to nodebalancers/[nodebalanceID]/configurations
go to nodebalancers/[nodebalanceID]/settings

Verification steps

(How to verify changes)

There is no a notification banner that You don't have permissions to edit this NodeBalancer
All disabled buttons have the tooltops

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

…cers, remove notification banner

linode-gh-bot · 2025-11-06T10:07:22Z

Cloud Manager UI test results

🔺 7 failing tests on test run #3 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
`7 Failing`	`844 Passing`	`13 Skipped`	`45m 55s`

Details

Failing Tests
	Spec	Test
❌	`create-database.spec.ts`	Cloud Manager Cypress Tests→create a database cluster, mocked data » creates a g7-premium-2 MySQL v8.x 3-node cluster
❌	`create-stackscripts.spec.ts`	Cloud Manager Cypress Tests→Create stackscripts » creates a StackScript and deploys a Linode with it
❌	`search-images.spec.ts`	Cloud Manager Cypress Tests→Search Images » "before all" hook for "creates two images and make sure they show up in the table and are searchable"
❌	`create-firewall.spec.ts`	Cloud Manager Cypress Tests→create firewall » "before all" hook for "creates a firewall without a linode"
❌	`create-firewall.spec.ts`	Cloud Manager Cypress Tests→restricted user cannot create firewall » "before each" hook for "confirms the create button is disabled on the Firewall Landing page"
❌	`update-stackscripts.spec.ts`	Cloud Manager Cypress Tests→Update stackscripts » updates a StackScript
❌	`clone-linode.spec.ts`	Cloud 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/databases/create-database.spec.ts,cypress/e2e/core/stackscripts/create-stackscripts.spec.ts,cypress/e2e/core/images/search-images.spec.ts,cypress/e2e/core/firewalls/create-firewall.spec.ts,cypress/e2e/core/stackscripts/update-stackscripts.spec.ts,cypress/e2e/core/linodes/clone-linode.spec.ts"

* feat: [UIE-9534] - IAM: add tooltips * Added changeset: IAM: add tooltips for disabled buttons for nodebalancers, remove notification banner * update text * remove tooltip

* initial commit * Added changeset: Address oAuth issue with IAM OAuthCallback * fix: [UIE-9456 & UIE-9457] - Typo + expose search filters on /iam/roles route (#13034) * UIE-9456 & UIE-9457 * test * Added changeset: Typo + expose search filters on /iam/roles route * feat: [UIE-9481] - IAM: add perm check for the drawer (#13043) * feat: [UIE-9481] - IAM: add perm check for the drawer * Added changeset: IAM: add a permission check for delete nodebalancer drawer * feat: [UIE-9503] - add empty state to Default Roles/Entities tab (#13042) * feat: add empty state to Default Roles/Entities table * Added changeset: IAM: Empty state for the Default Roles and Default Entities Access tabs * update text * add unit tests * review fix * unit test fix * fix: [UIE-9478] - IAM: disable tag edit (#13046) * fix: [UIE-9478] - IAM: disable tag edit * Added changeset: IAM: tags editing was enabled for restricted users * fix: [UIE-9478] - add tooltip, disable manage tags in Volume action menu * style fix * Cloud version 1.154.0, API v4 version 0.152.0, Validation version 0.78.0, UI version 0.23.0 * Cloud version 1.154.0, API v4 version 0.152.0, Validation version 0.78.0, UI version 0.23.0 * fix: [M3-10698] - Update linode/clone endpoint and label calculations for < 1 hour (#13045) * fix: [M3-9498] - Update linode/clone endpoint and label calculations for < 1 hour * Add changelogs --------- Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com> * fix: [UIE-9520] - IAM Child Account - user not found error handling (#13047) * try/cath child user query * consistent handling * types * Added changeset: IAM Child Account - user not found error handling * fix: [UIE-9477] - IAM: disable delete and detach popups for Volumes when user has no permissions (#13049) * fix: [UIE-9477] - IAM: Disable delete and detach popups for Volumes if no permissions * Added changeset: IAM: restricted users could access delete and detach popups without permissions * upcoming: [UIE-9434] - Create feature flag for generational compute plans (#13054) * upcoming: [UIE-9434] - Create feature flag for generational compute plans * Added changeset: Feature flag for Generational Compute plans * test: [DPS-34825] Add cypress tests for Create/Edit Stream forms and Streams Landing (#13008) * refactor: [DI-27807] - Change status of edit & delete button in alert list table based on status (#13052) * refactor: [DI-27807] - Change status of edit & delete button in alert list table based on status * Added comment * Added changeset * refactor: [DI-27807] - Updated test cases * upcoming: [UIE-9368] - Create a firewall Rulesets & Preflixlists feature flag (#13051) * Add firewall rs & pl feature flag * Added changeset: Feature flag for Firewall Rulesets & Prefixlists * feat: [UIE-9341, UIE-9342, UIE-9521] - IAM: fix perm for vpcs (#13050) * UIE-9342 * UIE-9341 * feat: [UIE-9341, UIE-9342, UIE-9521] - IAM: fix perm for vpc * Added changeset: IAM: fix permissiom's check for vpc for assigning/unassigning linodes * filter linodes * e2e deferred getLinodes call --------- Co-authored-by: Alban Bailly <abailly@akamai.com> * refactor: [UIE-9325] - Replace Formik with React Hook Form in AddAccessControlDrawer (#13044) ## Description 📝 Refactor Database Manage Networking drawer to use React Hook Form instead of Formik ## Changes 🔄 - There should be no visual external changes other than IP validation on blur instead of after clicking Update Access Controls ## How to test 🧪 ### Prerequisites (How to setup test environment) - Have the Database VPC feature flag enabled ### Verification steps (How to verify changes) - [ ] Test adding/editing IPs in the Database landing page via Manage Access Controls action dropdown - [ ] Test adding/editing IPs in the Database details page via the Networking tab - [ ] There should be no regressions compared to prod, error messages should display * [UIE-9533] -Hopeful fix: Race condition with Preferences overrides in PrimaryNav (#13056) * attempt to rule out any race condition * comments * code and fixes * cleanup * Added changeset: Race condition with Preferences overrides in PrimaryNav * feat: [UIE-9393] - IAM Parent/Child: delegate permissions for child account (#13033) * feat: [UIE-9393] - IAM Parent/Child: delegate permissions for child account * changesets * remove the unrestricted check for query * fix conflict * e2e gating * lint fix * better approach to fixing e2e * Mock IAM to be disabled in Account Switching test --------- Co-authored-by: Alban Bailly <abailly@akamai.com> Co-authored-by: Alban Bailly <130582365+abailly-akamai@users.noreply.github.com> Co-authored-by: Joe D'Amore <jdamore@akamai.com> * feat: [UIE-9534] - IAM: fix perm for nodebalancer (#13058) * feat: [UIE-9534] - IAM: add tooltips * Added changeset: IAM: add tooltips for disabled buttons for nodebalancers, remove notification banner * update text * remove tooltip * change:[UIE-9541] Add MSW crud support for types API (#13067) * change:[UIE-9541] Add MSW crud support for types API * Added changeset: Add MSW crud support for types API * Added changeset: Add MSW crud support for types API --------- Co-authored-by: aaleksee-akamai <aaleksee@akamai.com> Co-authored-by: mpolotsk-akamai <157619599+mpolotsk-akamai@users.noreply.github.com> Co-authored-by: Dajahi Wiley <dwiley@linode.com> Co-authored-by: Jaalah Ramos <125309814+jaalah-akamai@users.noreply.github.com> Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com> Co-authored-by: grevanak-akamai <145482092+grevanak-akamai@users.noreply.github.com> Co-authored-by: mduda-akamai <mduda@akamai.com> Co-authored-by: Nikhil Agrawal <165884194+nikhagra-akamai@users.noreply.github.com> Co-authored-by: Purvesh Makode <pmakode@akamai.com> Co-authored-by: Hana Xu <115299789+hana-akamai@users.noreply.github.com> Co-authored-by: Joe D'Amore <jdamore@akamai.com>

feat: [UIE-9534] - IAM: add tooltips

ed9018b

aaleksee-akamai self-assigned this Nov 5, 2025

aaleksee-akamai requested a review from a team as a code owner November 5, 2025 13:04

aaleksee-akamai added NodeBalancers Relating to NodeBalancers IAM (Identity & Access Management) labels Nov 5, 2025

github-project-automation Bot moved this to Review in Cloud Manager Nov 5, 2025

github-project-automation Bot added this to Cloud Manager Nov 5, 2025

Added changeset: IAM: add tooltips for disabled buttons for nodebalan…

192de19

…cers, remove notification banner

aaleksee-akamai requested review from abailly-akamai, kwojtowiakamai and mpolotsk-akamai November 5, 2025 13:05

bnussman-akamai reviewed Nov 5, 2025

View reviewed changes

Comment thread packages/manager/src/features/NodeBalancers/constants.ts Outdated

Comment thread packages/manager/src/features/NodeBalancers/NodeBalancerDetail/NodeBalancerSettings.tsx Outdated

update text

46ef4da

abailly-akamai approved these changes Nov 7, 2025

View reviewed changes

mpolotsk-akamai approved these changes Nov 7, 2025

View reviewed changes

github-project-automation Bot moved this from Review to Approved in Cloud Manager Nov 7, 2025

remove tooltip

9bb0945

aaleksee-akamai merged commit 8a1bd09 into linode:develop Nov 7, 2025
34 of 35 checks passed

github-project-automation Bot moved this from Approved to Merged in Cloud Manager Nov 7, 2025

This was referenced Nov 13, 2025

Release v1.155.0 - release → staging #13085

Merged

Release v1.155.0 - staging → master #13101

Merged

Navigation Menu

Uh oh!

feat: [UIE-9534] - IAM: fix perm for nodebalancer - #13058

feat: [UIE-9534] - IAM: fix perm for nodebalancer#13058
aaleksee-akamai merged 4 commits into
linode:developfrom
aaleksee-akamai:UIE-9534-iam-nodebalancer-perm-fix

aaleksee-akamai commented Nov 5, 2025

Uh oh!

Uh oh!

Uh oh!

linode-gh-bot commented Nov 6, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

aaleksee-akamai commented Nov 5, 2025

Description 📝

Changes 🔄

Scope 🚢

Target release date 🗓️

Preview 📷

How to test 🧪

Prerequisites

Reproduction steps

Verification steps

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

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

Uh oh!

Uh oh!

Uh oh!

linode-gh-bot commented Nov 6, 2025

Cloud Manager UI test results

🔺 7 failing tests on test run #3 ↗︎

Details

Troubleshooting

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants