Skip to content

fix: Fix backups bug for Linode Interfaces#13458

Open
coliu-akamai wants to merge 3 commits intolinode:developfrom
coliu-akamai:linode-backups-bug
Open

fix: Fix backups bug for Linode Interfaces#13458
coliu-akamai wants to merge 3 commits intolinode:developfrom
coliu-akamai:linode-backups-bug

Conversation

@coliu-akamai
Copy link
Contributor

@coliu-akamai coliu-akamai commented Mar 3, 2026

Description 📝

The Network Connection selection field is incorrectly disabled on the backups page. It should only be disabled if interface type is Configuration Profile Interfaces

Changes 🔄

  • keep previous interface generation type when selecting a Linode on the backups page
    • (we don't want to use linode.interface_generation for here because the selected linode might have a different interface generation type than what's allowed by our account setting )
  • Hide VLAN/VPC interface fields if unable to select Network Connection type + clear vpc/vlan fields in case they were already touched when selecting Config profile on backups tab to prevent hidden errors
  • moved up logic to disable Interface Type, since I'm also using this to determine when to hide VLAN/VPC stuff

Scope 🚢

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

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

Target release date 🗓️

next release?

Preview 📷

Before After
linode-backups-behavior.mov
linode-backups-updated-behavior.mov

How to test 🧪

Prerequisites

Have a Linode that has a backup

Reproduction steps

For all account setting options except for Configuration Profile Interface only:

  • Navigate to the the Create Linode from Backups page. Before selecting a Linode, note the interface generation type is correct (linode)
  • Select a Linode with a backup - notice how interface generation type becomes undefined
  • Networking Connection type is disabled (even though selected interface type is Linode in the UI, underlying value is undefined)

For an account setting with Configuration Profile Interface only:

  • Navigate to the the Create Linode from Backups page. Before selecting a Linode, note the interface generation type is correct (legacy_config)
  • Select a Linode with a backup - notice how interface generation type becomes undefined
  • Networking Connection type is disabled but
  • Network Interface type is incorrectly set to Linode even though it should be Configuration Profiles - this is because if interface generation type is undefined, Linode is now always chosen

Verification steps

  • Confirm when you select a Linode from the Backups table, the Interface Generation type is what you'd expect
    • Configuration Profile Interfaces for 'Configuration Profile Interface only' account setting
    • Linode Interfaces for all other Account setting options
  • and that the Interface Type is only disabled when Interface Generation is 'Configuration Profile'
  • Confirm that creating a Linode in the other flows (os, stackscript, etc) still works fine
  • Confirm that cloning a Linode still works fine

@coliu-akamai coliu-akamai self-assigned this Mar 3, 2026
@coliu-akamai coliu-akamai changed the title Fix backups bug for Linode Interfaces fix: Fix backups bug for Linode Interfaces Mar 3, 2026
@coliu-akamai coliu-akamai added the Linodes Dealing with the Linodes section of the app label Mar 3, 2026
@coliu-akamai coliu-akamai marked this pull request as ready for review March 3, 2026 21:48
@coliu-akamai coliu-akamai requested a review from a team as a code owner March 3, 2026 21:48
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 4 failing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
4 Failing865 Passing11 Skipped41m 42s

Details

Failing Tests
SpecTest
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Parent » can switch from Proxy user back to Parent account user from Billing page
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Child » can switch to another Child account as a Proxy user
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can create and delete object storage buckets
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can update bucket access

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/parentChild/account-switching.spec.ts,cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Linode Interfaces Linodes Dealing with the Linodes section of the app

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

2 participants