Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: [M3-7739, M3-7741, M3-7746, M3-7747] - Cherry pick zero price fixes for release #10177

Conversation

jdamore-linode
Copy link
Contributor

@jdamore-linode jdamore-linode commented Feb 12, 2024

Description 📝

Eliminates errors displayed in the UI for $0 pricing ($0 is valid) and adds test coverage to flows.

Changes 🔄

  • Linode Migrate dialog displays $0 pricing
  • Kubernetes Resize and Add a Node displays $0 pricing and no errors'
  • Enable All Backups drawer and enable backups for an individual linode is possible and displays $0 pricing without errors

How to test 🧪

Prerequisites

(How to setup test environment)

  • Have a linode in es-madrid.

Reproduction steps

(How to reproduce the issue, if applicable)

  • In prod, go through the migrate, enable backups, and kubernetes add/resize node pool flows. Note the error messages visible for $0 pricing and in some cases (enable backups on a linode) an inability to complete the action in that region with $0 pricing.

Verification steps

(How to verify changes)

As an Author I have considered 🤔

Check all that apply

  • 👀 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

mjac0bs and others added 4 commits February 12, 2024 11:28
… Add a Node Pool drawers (linode#10157)

* Allow -zsh LKE prices without error notices in Resize Pool and Add Pool drawers

* Fix loading spinner displaying above what was supposed to be loading

* Fix conditional to render notice if either price is invalid

* Add test coverage

* Added changeset: Hide error notices for /bin/sh regions for LKE Resize and Add Node Pools

* Fix changeset wording

* Address feedback: use invalid price util
…awer (linode#10161)

* Remove error indicator for Linodes in $0 regions

* Fix $0 total price display issue

* Cover $0 pricing cases in Cypress backup tests

* Add BackupLinodeRow tests to account for error states and $0 regions

* Add unit tests for BackupDrawer component
…10166)

* Add unit tests for MigrationPricing component

* Accounting for $0 prices in MigrationPricing component
…ns with $0 price (linode#10153)

* Fix error when enabling backups for Linodes in regions with $0 price

* Add unit tests for EnableBackupsDialog

---------

Co-authored-by: Mariah Jacobs <114685994+mjac0bs@users.noreply.github.com>
@jdamore-linode jdamore-linode requested review from a team as code owners February 12, 2024 16:39
@jdamore-linode jdamore-linode requested review from cliu-akamai, cpathipa and abailly-akamai and removed request for a team February 12, 2024 16:39
Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

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

  • Able to enable backups for $0 Linodes
  • Able to Enable All Backups with a $0 Linode
  • Migrate dialog supports $0 Linodes
  • Linode Plan table with $0 Linodes
  • Kubernetes plan table with $0 Linodes
  • Linode resize table with $0 Linodes

Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

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

Using Madrid as a current $0 region, confirmed the following:

Desc Screenshot
Enable Backups for a Linode in Madrid Screenshot 2024-02-12 at 9 11 24 AM
Enable All Backups Screenshot 2024-02-12 at 9 11 07 AM
Migrate with $0 regions Screenshot 2024-02-12 at 9 12 26 AM Screenshot 2024-02-12 at 9 12 43 AM
Kubernetes Resize Pool and Add a Node Pool with cluster in Madrid Screenshot 2024-02-12 at 9 26 28 AM Screenshot 2024-02-12 at 9 25 20 AM

@jdamore-linode jdamore-linode merged commit 9d5fad5 into linode:staging-off-release Feb 12, 2024
16 of 17 checks passed
Copy link

Coverage Report:
Base Coverage: 81.28%
Current Coverage: 81.28%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants