Skip to content

upcoming: [M3-10125] - Add feature flag support for Cloud NAT - #12388

Merged
hana-akamai merged 2 commits into
linode:developfrom
hana-akamai:M3-10125-create-cloud-nat-feature-flag
Jun 17, 2025
Merged

hana-akamai merged 2 commits into
linode:developfrom
hana-akamai:M3-10125-create-cloud-nat-feature-flag

Conversation

@hana-akamai

Copy link
Copy Markdown
Contributor

Description 📝

Add feature flag support for Cloud NAT

Preview 📷

Dev Tools LD response
Screenshot 2025-06-16 at 11 40 39 AM Screenshot 2025-06-16 at 11 40 13 AM

How to test 🧪

Verification steps

(How to verify changes)

  • Checkout this PR or PR preview link
  • Verify Cloud NAT is present in the Dev Tools Feature Flags section
  • Verify Cloud NAT in the LD response
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

@hana-akamai hana-akamai self-assigned this Jun 16, 2025
@hana-akamai
hana-akamai requested a review from a team as a code owner June 16, 2025 16:25
@hana-akamai
hana-akamai requested review from harsh-akamai and pmakode-akamai and removed request for a team June 16, 2025 16:25
@hana-akamai
hana-akamai requested a review from bill-akamai June 16, 2025 16:26

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

Looks good @hana-akamai. Do we need a changeset?
✅ Verify Cloud NAT is present in the Dev Tools Feature Flags section
✅Verify Cloud NAT in the LD response

@hana-akamai

Copy link
Copy Markdown
Contributor Author

@bill-akamai yes, thanks for catching!

@hana-akamai hana-akamai added the Add'tl Approval Needed Waiting on another approval! label Jun 16, 2025
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 659 passing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing659 Passing4 Skipped116m 58s

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Jun 17, 2025
@pmakode-akamai pmakode-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Jun 17, 2025
@hana-akamai
hana-akamai merged commit 94f338c into linode:develop Jun 17, 2025
@hana-akamai
hana-akamai deleted the M3-10125-create-cloud-nat-feature-flag branch June 17, 2025 14:52
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Jun 17, 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! Cloud NAT

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants