hotfix: Incorrect Region Options

[bnussman-akamai]

Description 📝

• Fixes bug introduced in fix: [M3-9022, M3-9027] - Filter regions by endpoints and fix unavailable regions #11432

Bug Summary 🐛

• One-off logic was added for Object Storage Gen2, but that logic was applied to all region selects throughout Cloud Manager resulting in users seeing incorrect region options outside of Object Storage 🌎

manager/packages/manager/src/components/RegionSelect/RegionSelect.utils.tsx

Lines 43 to 45 in 0b0fa66

	if (isObjectStorageGen2Enabled && WHITELISTED_REGIONS.has(region.id)) {
	return true;
	}

The Fix 🔧

• Add a condition to check that currentCapability === 'Object Storage' to the new one-off logic so that it only applies to Object Storage region selects

Preview 📷

Before

sg-sin-2 is shown as both a core region and a distributed region 😳

After

sg-sin-2 only shows up as a core region

How to test 🧪

• Check Region Selects throughout Cloud Manager
  • Verify correct regions are shown
  • Verify that on Linode Create, regions are not duplicated under "Core" and "Distributed"

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 unit tests are passing
• TypeScript compilation succeeded without errors
• Code passes all linting rules

[coliu-akamai]

thank you 🥲

[github-actions]

Coverage Report: ❌
Base Coverage: 86.85%
Current Coverage: 86.84%

[linode-gh-bot]

Cloud Manager UI test results

🔺 1 failing test on test run #6 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
1 Failing	465 Passing	2 Skipped	111m 55s

Details

Failing Tests
	Spec	Test
❌	linode-config.spec.ts	End-to-End » Clones a config

Troubleshooting

Use this command to re-run the failing tests:

yarn cy:run -s "cypress/e2e/core/linodes/linode-config.spec.ts"

[dwiley-akamai]

Tested and discussed during our mini-dogfooding session -- region options look as expected in Linode Create flow and in OBJ flows ✅