change: [UIE-9888] - Make firewall selection mandatory while creating linode and its interfaces by grevanak-akamai · Pull Request #13410 · linode/manager

grevanak-akamai · 2026-02-18T09:40:11Z

Description 📝

Make firewall selection mandatory while creating linode and its interfaces. If there are no firewalls or user doesn't wants to assign a firewall to a linode, "No firewall" option can be selected but the firewall field cannot be left blank.

Changes 🔄

List any change(s) relevant to the reviewer.

Firewall selection is made mandatory while creating linode and adding interfaces to linode.
Added new option "No firewalls" under select firewall dropdown to allow users to not assign a firewall(Not recommended though). Added a warning notice if "No firewall" option is selected.
Updated linode schema to make firewall field mandatory.

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

Mar 2026

Preview 📷

1. If no firewall is selected

2. "No firewall" option in the dropdown

How to test 🧪

Prerequisites

Navigate to Create linode page.

Verification steps

Ensure create linode submission fails if no option is selected under firewall dropdown.
Select any firewall or select "No Firewall" option in the dropdown and ensure linode is created/cloned.

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

grevanak-akamai · 2026-02-18T11:32:23Z

packages/validation/src/linodes.schema.ts


 export const CreateLinodeInterfaceSchema = object({
-  firewall_id: number().nullable(),
+  firewall_id: number()


API accepts both -1 and null(will be deprecated soon and will be utilized only to assign default firewall if any) to not to assign a firewall to a linode and hence firewall field has been made mandatory in UI.

linode-gh-bot · 2026-02-22T19:08:23Z

Cloud Manager UI test results

🔺 3 failing tests on test run #5 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
`3 Failing`	`865 Passing`	`11 Skipped`	`40m 58s`

Details

Failing Tests
	Spec	Test
❌	`search-images.spec.ts`	Cloud Manager Cypress Tests→Search Images » creates two images and make sure they show up in the table and are searchable
❌	`account-switching.spec.ts`	Cloud Manager Cypress Tests→Parent/Child account switching→From Parent to Child » can search child accounts
❌	`smoke-community-stackscripts.spec.ts`	Cloud Manager Cypress Tests→Community Stackscripts integration tests » deploys a new linode as expected

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/images/search-images.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts,cypress/e2e/core/stackscripts/smoke-community-stackscripts.spec.ts"

grevanak-akamai · 2026-02-23T15:11:11Z

packages/manager/src/features/Account/DefaultFirewalls.tsx

                  label="Linode Interfaces - Public Interface Firewall"
                  onChange={(e, firewall) => field.onChange(firewall.id)}
                  placeholder={DEFAULT_FIREWALL_PLACEHOLDER}
+                  showNoFirewallOption={false}


In future, default firewalls will also support "No Firewalls" option.

grevanak-akamai · 2026-02-23T15:24:13Z

Cloud Manager UI test results

🔺 3 failing tests on test run #5 ↗︎

❌ Failing ✅ Passing ↪️ Skipped 🕐 Duration
3 Failing 865 Passing 11 Skipped 40m 58s

Details

Failing Tests
Spec Test
❌ search-images.spec.ts Cloud Manager Cypress Tests→Search Images » creates two images and make sure they show up in the table and are searchable
❌ account-switching.spec.ts Cloud Manager Cypress Tests→Parent/Child account switching→From Parent to Child » can search child accounts
❌ smoke-community-stackscripts.spec.ts Cloud Manager Cypress Tests→Community Stackscripts integration tests » deploys a new linode as expected

Troubleshooting

Use this command to re-run the failing tests:
pnpm cy:run -s "cypress/e2e/core/images/search-images.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts,cypress/e2e/core/stackscripts/smoke-community-stackscripts.spec.ts"

Failure in smoke-community-stackscripts.spec.ts is related I guess. Rest 2 are unrelated.

packages/validation/src/linodes.schema.ts

packages/manager/src/features/Linodes/LinodeCreate/Networking/utilities.ts

… linode and its interfaces

coliu-akamai

Thanks @grevanak-akamai! Had a couple of questions/comments below.

Also, this PR is missing a changeset - ~~do we still need changesets for the release process?~~ looked at some other open PRs and they had changesets - could we add one to this PR?

coliu-akamai · 2026-02-26T19:05:44Z

...ures/Linodes/LinodesDetail/LinodeNetworking/LinodeInterfaces/AddInterfaceDrawer/utilities.ts

+      firewall_id: number().required(
+        'Select an option or create a new Firewall.'
+      ),


Should we just have this part say 'Select an option.' only? There's no option to create a firewall on the Add Interface drawer:

coliu-akamai · 2026-02-26T19:07:06Z

...inodesDetail/LinodeNetworking/LinodeInterfaces/EditInterfaceDrawer/EditInterfaceFirewall.tsx

          <FirewallSelect
            errorText={fieldState.error?.message}
            onChange={(e, firewall) => field.onChange(firewall?.id ?? null)}
+            showNoFirewallOption={false}


maybe out of scope for this PR, but given that we're making firewall selection mandatory while creating a linode & its interfaces, should we also make it mandatory when editing an interface?

coliu-akamai · 2026-02-26T19:34:32Z

packages/manager/src/features/Linodes/LinodeCreate/resolvers.ts

      values.linodeInterfaces = values.linodeInterfaces.map(
        getCleanedLinodeInterfaceValues
      );
+      if (tab === 'Clone Linode' && !values.firewall_id) {


lines 41-43 look the same as lines 72-75 - do we need this check here underneath if (context?.isLinodeInterfacesEnabled) if we're running it anyway at lines 72-75?

Also, regarding this comment:
The Clone Linode flow does not have the firewall_id field under interfaces object, so we set firewall_id to -1 to bypass the firewall requirement in the validation schema.

just to double check - since we're keeping the original firewall schema which lets us have undefined values, do we still need this condition & the ones below?

coliu-akamai · 2026-02-26T19:38:08Z

packages/validation/src/linodes.schema.ts

          }),
    }),
  metadata: MetadataSchema.notRequired().default(undefined),
-  firewall_id: number().nullable().notRequired(),


should we add back in the notRequired() for this schema, since the API will accept undefined? (Referring to Banks' earlier comment here - #13410 (comment))

github-project-automation bot added this to Cloud Manager Feb 18, 2026

github-project-automation bot moved this to Review in Cloud Manager Feb 18, 2026

grevanak-akamai force-pushed the feature/UIE-10132-block-interface-creation-without-firewall branch from c096c03 to 4844424 Compare February 18, 2026 11:07

grevanak-akamai commented Feb 18, 2026

View reviewed changes

grevanak-akamai force-pushed the feature/UIE-10132-block-interface-creation-without-firewall branch from 3926c9e to 262df83 Compare February 22, 2026 18:18

grevanak-akamai commented Feb 23, 2026

View reviewed changes

grevanak-akamai marked this pull request as ready for review February 23, 2026 15:21

grevanak-akamai requested review from a team as code owners February 23, 2026 15:21

grevanak-akamai requested review from bnussman-akamai, coliu-akamai and jdamore-linode and removed request for a team February 23, 2026 15:21

grevanak-akamai self-assigned this Feb 23, 2026

grevanak-akamai added Ready for Review Linodes Dealing with the Linodes section of the app labels Feb 23, 2026

bnussman-akamai reviewed Feb 23, 2026

View reviewed changes

packages/validation/src/linodes.schema.ts Outdated Show resolved Hide resolved

bnussman-akamai reviewed Feb 23, 2026

View reviewed changes

grevanak-akamai added 2 commits February 25, 2026 18:32

change: [UIE-9888] - Make firewall selection mandatory while creating…

702ee48

… linode and its interfaces

Fixed e2e test cases

0560992

grevanak-akamai force-pushed the feature/UIE-10132-block-interface-creation-without-firewall branch from 5724cc4 to b700773 Compare February 25, 2026 13:13

grevanak-akamai requested a review from bnussman-akamai February 25, 2026 15:04

Addressed review comments

b5ffc6d

grevanak-akamai force-pushed the feature/UIE-10132-block-interface-creation-without-firewall branch from b700773 to b5ffc6d Compare February 26, 2026 15:45

coliu-akamai reviewed Feb 26, 2026

View reviewed changes

Conversation

grevanak-akamai commented Feb 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description 📝

Changes 🔄

Scope 🚢

Target release date 🗓️

Preview 📷

How to test 🧪

Prerequisites

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!

grevanak-akamai Feb 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

linode-gh-bot commented Feb 22, 2026

Cloud Manager UI test results

🔺 3 failing tests on test run #5 ↗︎

Details

Troubleshooting

Uh oh!

grevanak-akamai Feb 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

grevanak-akamai commented Feb 23, 2026

Cloud Manager UI test results

🔺 3 failing tests on test run #5 ↗︎

Details

Troubleshooting

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coliu-akamai left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

coliu-akamai Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

coliu-akamai Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

coliu-akamai Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

coliu-akamai Feb 26, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

grevanak-akamai commented Feb 18, 2026 •

edited

Loading

grevanak-akamai Feb 18, 2026 •

edited

Loading

grevanak-akamai Feb 23, 2026 •

edited

Loading

coliu-akamai left a comment •

edited

Loading