Skip to content

Conversation

@grevanak-akamai
Copy link
Contributor

Description 📝

Change the default selection of network interface type from legacy 'configuration profile' to 'linode interface' in Linode create flow.

Changes 🔄

  • Changed default value of interface generation type to 'linode' in getDefaultInterfaceGenerationFromAccountSetting utility function

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

NA

Preview 📷

Before After
Screenshot 2025-12-23 at 12 13 09 PM Screenshot 2025-12-23 at 12 28 07 PM

How to test 🧪

Prerequisites

  • In account settings, ensure the network interface type for new linodes is "config profile but allow Linode interface"

Verification steps

  • Network interface type in Linode create flow should be "Linode interfaces" by default.
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 grevanak-akamai requested a review from a team as a code owner December 23, 2025 07:20
@grevanak-akamai grevanak-akamai changed the title change: [UIE-9860] - Change the default selection to linode interface in Linode create flow change: [UIE-9860] - Change the default selection of network interface type to linode interface in Linode create flow Dec 23, 2025
@grevanak-akamai grevanak-akamai force-pushed the feature/UIE-9860-change-default-selection-to-linode-interface branch from fb358c4 to fb8b1a6 Compare December 23, 2025 12:18
@grevanak-akamai grevanak-akamai added the Linodes Dealing with the Linodes section of the app label Dec 23, 2025
@grevanak-akamai grevanak-akamai self-assigned this Dec 23, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 7 failing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
7 Failing862 Passing11 Skipped51m 20s

Details

Failing Tests
SpecTest
create-linode-with-vpc.spec.tsCloud Manager Cypress Tests→Create Linode with VPCs (Linode Interfaces) » can assign existing VPCs during Linode Create flow (legacy)
create-linode-with-vpc.spec.tsCloud Manager Cypress Tests→Create Linode with VPCs (Linode Interfaces) » can assign new VPCs during Linode Create flow (legacy)
create-linode-with-firewall.spec.tsCloud Manager Cypress Tests→Create Linode with Firewall (Linode Interfaces) » can assign existing Firewall during Linode Create flow (legacy)
create-linode-with-firewall.spec.tsCloud Manager Cypress Tests→Create Linode with Firewall (Linode Interfaces) » can assign new Firewall during Linode Create flow (legacy)
create-linode-with-firewall.spec.tsCloud Manager Cypress Tests→Create Linode with Firewall (Linode Interfaces) » can generate and assign a compliant Firewall during Linode Create flow (legacy)
create-linode-with-vlan.spec.tsCloud Manager Cypress Tests→Create Linode with VLANs (Linode Interfaces) » can assign existing VLANs during Linode create flow (legacy)
create-linode-with-vlan.spec.tsCloud Manager Cypress Tests→Create Linode with VLANs (Linode Interfaces) » can assign new VLANs during Linode create flow (legacy)

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/create-linode-with-vpc.spec.ts,cypress/e2e/core/linodes/create-linode-with-firewall.spec.ts,cypress/e2e/core/linodes/create-linode-with-vlan.spec.ts"

@bnussman-akamai bnussman-akamai added the Related Cypress Test Failure Test failure(s) are the result of changes in this PR and must be addressed before merging. label Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Linodes Dealing with the Linodes section of the app Ready for Review Related Cypress Test Failure Test failure(s) are the result of changes in this PR and must be addressed before merging.

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

3 participants