Skip to content

change: [M3-10586] - Remove update strategy from node pool summary item in LKE checkout bar - #12814

Merged
mjac0bs merged 2 commits into
linode:developfrom
mjac0bs:M3-10586-remove-update-strategy-from-checkout-bar
Sep 10, 2025
Merged

mjac0bs merged 2 commits into
linode:developfrom
mjac0bs:M3-10586-remove-update-strategy-from-checkout-bar

Conversation

@mjac0bs

@mjac0bs mjac0bs commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Description 📝

Currently, with Post LA changes, the kube checkout bar was updated to show the node pool's update strategy for each pool (for LKE-E clusters, where this selection is available).

After UX feedback, we've decided to keep the information in the checkout bar simple to avoid overwhelming the user with too many details and focus on pricing. Update strategy and firewall will not be listed.

Firewall was not added to begin with, I just forgot to remove update strategy earlier - since this is only a display change and has no functional impact, I don't think it's priority to squeeze this into the already cut 9/9 release, even if we flip the post LA flag before 9/23.

Changes 🔄

  • Remove the displayed update strategy from the UI and clean up related RHF logic no longer needed

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

9/23

Preview 📷

Before After
Screenshot 2025-09-04 at 11 05 02 AM Screenshot 2025-09-04 at 11 03 40 AM

How to test 🧪

Prerequisites

  • Have LKE-Enterprise enabled (feature flags: enabled, la, postLa) + customer tag

Reproduction steps

(How to reproduce the issue, if applicable)

  • Check out develop and go to http://localhost:3000/kubernetes/create
  • Select an LKE-E tier, go through the create flow, and add at least one node pool to the cluster
  • Observe the checkout bar and note the update strategy is listed

Verification steps

(How to verify changes)

  • Check out this branch
  • Select an LKE-E tier, go through the create flow, and add at least one node pool to the cluster
  • Observe the checkout bar and note the update strategy is not listed
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

@mjac0bs
mjac0bs marked this pull request as ready for review September 4, 2025 18:27
@mjac0bs
mjac0bs requested a review from a team as a code owner September 4, 2025 18:27
@mjac0bs
mjac0bs requested review from bnussman-akamai and cpathipa and removed request for a team September 4, 2025 18:27

@bnussman-akamai bnussman-akamai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this better. Nice

@mjac0bs mjac0bs added the Add'tl Approval Needed Waiting on another approval! label Sep 4, 2025
@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Sep 10, 2025
@cpathipa cpathipa added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Sep 10, 2025
@mjac0bs
mjac0bs merged commit c686656 into linode:develop Sep 10, 2025
34 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Sep 10, 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! LKE-Enterprise

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants