Skip to content

upcoming: [M3-10389] - Feature flag New Chip and Beta Chip for Linode Interfaces - #12846

Merged
bnussman-akamai merged 7 commits into
linode:developfrom
bnussman-akamai:M3-10389-support-new-chip-with-linode-interfaces
Sep 12, 2025
Merged

bnussman-akamai merged 7 commits into
linode:developfrom
bnussman-akamai:M3-10389-support-new-chip-with-linode-interfaces

Conversation

@bnussman-akamai

@bnussman-akamai bnussman-akamai commented Sep 9, 2025

Copy link
Copy Markdown
Member

Description 📝

  • Updates Linode Interface UI to allow the New and Beta chips to be toggled via feature flags

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

Preview 📷

Beta Chip

This is the current UI because the feature is in beta right now

Linode Create Flow Account Settings Upgrade Interfaces Dialog (Linode Details)
Screenshot 2025-09-09 at 1 33 34 PM Screenshot 2025-09-09 at 1 33 43 PM Screenshot 2025-09-09 at 1 33 58 PM

New Chip

This is what the UI will look like in a few weeks once we toggle the feature flag

Linode Create Flow Account Settings Upgrade Interfaces Dialog (Linode Details)
Screenshot 2025-09-09 at 1 31 12 PM Screenshot 2025-09-09 at 1 30 53 PM Screenshot 2025-09-09 at 1 30 31 PM

How to test 🧪

  • Using Cloud Manager's local dev tools, toggle the beta and new flags under linodeInterfaces
  • Verify the correct chips show in the UI
  • @coliu-akamai verify I got all of the chips and did not miss any
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

@bnussman-akamai
bnussman-akamai marked this pull request as ready for review September 10, 2025 13:29
@bnussman-akamai
bnussman-akamai requested a review from a team as a code owner September 10, 2025 13:29
@bnussman-akamai
bnussman-akamai requested review from coliu-akamai and dwiley-akamai and removed request for a team September 10, 2025 13:29

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

✅ beta chips when feature is using beta flag
✅ new chips when feature is using new flag

I didn't see any other spots where we use a beta chip for Linode interfaces ✅

thanks @bnussman-akamai! one qq below

Comment thread packages/manager/src/features/Account/NetworkInterfaceType.tsx
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 745 passing tests on test run #5 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing745 Passing6 Skipped36m 50s

@coliu-akamai coliu-akamai added the Add'tl Approval Needed Waiting on another approval! label Sep 11, 2025

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

Correct chip shows in the specified locations depending on which sub-flag is toggled ✅
Code review ✅

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Sep 12, 2025
@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Sep 12, 2025
@bnussman-akamai
bnussman-akamai merged commit edb112b into linode:develop Sep 12, 2025
35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Sep 12, 2025
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! Linode Interfaces

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants