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

Release v1.112.0 - release → staging #10178

Merged
merged 14 commits into from
Feb 12, 2024
Merged

Release v1.112.0 - release → staging #10178

merged 14 commits into from
Feb 12, 2024

Conversation

cpathipa
Copy link
Contributor

@cpathipa cpathipa commented Feb 12, 2024

Changelog

CloudManager [2024-02-12] - v1.112.0

Fixed:

  • Error when enabling backups for Linodes in regions with $0 pricing (#10153)
  • Error notices for $0 regions in LKE Resize and Add Node Pools drawers (#10157)
  • Error in Enable All Backups drawer when one or more Linode is in a $0 region (#10161)
  • Display $0.00 prices in Linode Migration dialog (#10166)

Added:

  • Support for IPv4 Ranges in VPC 'Assign Linodes to subnet' drawer (#10089)
  • VPC IPv4 address and range to Linode IP Address Table (#10108)
  • Support for VPC IPv4 Ranges data in Unassign Linodes drawer (#10114)
  • Support for VPC IPv4 Ranges in Linode Create flow and 'VPC IPv4 Ranges' column to inner Subnets table on VPC Detail page (#10116)

Validation

Changed:

  • ip_ranges field in LinodeInterfaceSchema no longer limited to 1 element (#10089)

hana-akamai and others added 12 commits January 26, 2024 16:58
… Table (#10108)

## Description 📝
Add support for displaying VPC IPv4 addresses and ranges in the Linode Details -> Network tab -> "IP Addresses" table

## Changes  🔄
List any change relevant to the reviewer.
- Add Support for VPC IPv4 addresses and ranges to the Linode IP Address table
  - If there is a VPC interface with 1:1 NAT, hide the Public IPv4 IP address row
- Improve mobile text display of the Type column

## How to test 🧪

### Prerequisites
(How to setup test environment)
- Have the `vpc-extra-beta` tag on your account and a VPC with at least one Subnet
- Assign a Linode to the Subnet and check both the checkboxes (Auto-assign a VPC IPv4 address & Assign a public IPv4 address)
- To add `ip_ranges`, check out #10089

### Verification steps 
(How to verify changes)
- Go to the Linode Details -> Network tab of the assigned Linode and check the IP Addresses table
- You should see IP Address rows for `IPv4 – VPC`, `VPC IPv4 – NAT`, and `IPv4 – VPC – Range` for each range
- You should _not_ see an IP address row for the type `IPv4 - Public`
```
yarn test LinodeIPAddressRow
```
…er (#10114)

## Description 📝
Support IPv4 Ranges data in "Unassign Linodes" drawer.

## Changes  🔄
- Use `RemovableSelectionsListTable` in Unassign drawer to display VPC IPv4 and VPC IPv4 Ranges data
- Added IPv4 VPC and ranges to CSV. Simplified out data structure to mimic subnet assign drawer

## How to test 🧪
### Verification steps 
Have the `vpc-extra-beta` tag on your account and a VPC containing at least one subnet with a linode assigned to it. Confirm that:

- You see a table in the "Linodes to be Unassigned from Subnet" section
- When you select a linode, the table is populated with its data
  - If you select one with no IPv4 Ranges, you should see "None" in that cell
 - Clicking the "X" for that table row removes it (and clicking "X" in the LinodeSelect should clear all selections from the table)
 - Ensure both assign/unassign download CSV data contains IPv4 VPC address and it's ranges (if you added any)

>[!note]
When adding ranges, you'll want to narrow things to `/32`, IE: `10.0.4.1/32`

---------

Co-authored-by: Dajahi Wiley <dwiley@linode.com>
Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
## Description 📝
Add support for VPC IPv4 Ranges in the Linode Create flow.

- "Add additional IPv4 ranges" section added in VPC panel
- Logic to include `ip_ranges` in creation payload

The inclusion of a "VPC IPv4 Ranges" column in the VPC Detail > Subnets inner table as per the mockups slipped through the cracks during ticket creation, but I noticed it and included it in this PR (to try to keep things more consolidated).

## How to test 🧪
### Verification steps 
With the `vpc-extra-beta` tag on your account, enter the Linode Create flow and confirm:

- The "Add additional IPv4 ranges" section appears in the VPC panel after a subnet is selected
- You can add and remove IP input fields using the "Add IPv4 Range" and the "X" buttons, respectively
- When you click "Create Linode," the POST request contains the IP ranges you specified in the form in the `ip_ranges` property
- Once the linode is created, go to the detail page of the VPC you assigned the linode to. Expand the inner subnet table and observe the "VPC IPv4 Ranges" column is populated with what you provided (or "None" if you didn't provide any).

---------

Co-authored-by: Dajahi Wiley <dwiley@linode.com>
Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
Co-authored-by: Hana Xu <hxu@akamai.com>
…xes for release (#10177)

* fix: [M3-7741] - Hide error notices for $0 regions in Resize Pool and Add a Node Pool drawers (#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

* fix: [M3-7746] - Fix $0 region price error in "Enable All Backups" drawer (#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

* fix: [M3-7747] - Fix Linode Migration dialog hidden $0 price (#10166)

* Add unit tests for MigrationPricing component

* Accounting for $0 prices in MigrationPricing component

* fix: [M3-7739] - Fix error when enabling backups for Linodes in regions with $0 price (#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>

* Replace "toBeDisabled" with "toHaveAttribute" assertion

---------

Co-authored-by: Mariah Jacobs <114685994+mjac0bs@users.noreply.github.com>
@cpathipa cpathipa requested review from a team as code owners February 12, 2024 17:52
@cpathipa cpathipa requested review from jdamore-linode, bnussman-akamai and jaalah-akamai and removed request for a team February 12, 2024 17:52
@cpathipa cpathipa changed the title Staging off release Release v1.112.0 - release → staging Feb 12, 2024
@cpathipa cpathipa self-assigned this Feb 12, 2024
@cpathipa cpathipa added the Release → Staging Pre-Release: Release → Staging label Feb 12, 2024
Copy link

github-actions bot commented Feb 12, 2024

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

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.

e2e failure appears to be related to #10150, so I think we can disregard it

Looks like we may need to bump the version in validation's package.json

@jaalah-akamai jaalah-akamai added the Add'tl Approval Needed Waiting on another approval! label Feb 12, 2024
Copy link
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

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

Banks mentioned above, but just confirming for posterity that the e2e test failure* can be disregarded because it is 1) not related to a user-facing issue, and 2) is already fixed in develop

@cpathipa cpathipa merged commit 47870ab into staging Feb 12, 2024
19 of 20 checks passed
@bnussman-akamai bnussman-akamai deleted the staging-off-release branch February 16, 2024 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Add'tl Approval Needed Waiting on another approval! Release → Staging Pre-Release: Release → Staging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants