Skip to content

fix: [M3-10612] - Creating a firewall fails after creating a kubernetes - #12859

Merged
cpathipa merged 2 commits into
linode:hotfix/release-v1.150.1from
corya-akamai:M3-10612-create-firewall-fails-after-kubernetes
Sep 10, 2025
Merged

cpathipa merged 2 commits into
linode:hotfix/release-v1.150.1from
corya-akamai:M3-10612-create-firewall-fails-after-kubernetes

Conversation

@corya-akamai

Copy link
Copy Markdown
Contributor

Description 📝

Fix the error "Something went wrong" on firewall create.

Changes 🔄

  • Make sure we permissions for the given entity before creating the map

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

ASAP

Preview 📷

Screenshot 2025-09-10 at 1 25 51 PM
Screenshot 2025-09-10 at 1 28 25 PM

How to test 🧪

Prerequisites

  • Prod/devcloud

Reproduction steps

  • 1. Sign in as a restricted user with read_write access for LKE clusters and Firewalls
  • 2. Create at least one LKE cluster and wait for its nodes to provision
  • 3. Reload Cloud Manager and navigate to the Firewalls landing page at "/firewalls"
  • 4. Click "Create Firewall"
  • 5. Observe that Cloud Manager crashes upon opening the Firewall Create drawer

Verification steps

  • You are able to open the create firewall drawer
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

@corya-akamai
corya-akamai requested a review from a team as a code owner September 10, 2025 19:46
@corya-akamai
corya-akamai requested review from coliu-akamai and cpathipa and removed request for a team September 10, 2025 19:46
@corya-akamai corya-akamai added 🚨 Urgent Firewalls Related to Firewalls labels Sep 10, 2025
@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Sep 10, 2025

@mjac0bs mjac0bs 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.

Reapproving prior approved changes after branch updates.

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.

This is fine as long as it gets added to the changelog in #12858. 👍🏼

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

thanks @corya-akamai!

@cpathipa
cpathipa merged commit 83b3ef3 into linode:hotfix/release-v1.150.1 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

Firewalls Related to Firewalls 🚨 Urgent

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants