🔴 Preview Environment stopped on Bunnyshell

See: Environment Details | Pipeline Logs

Available commands (reply to this comment):

• 🔵 /bns:start to start the environment
• 🚀 /bns:deploy to redeploy the environment
• ❌ /bns:delete to remove the environment

The EP #21835 is still to be discussed. I raised this PR to illustrate further what I am trying to achieve. Happy to add tests and documentation if the EP is approved.

Codecov Report

Attention: Patch coverage is 48.71795% with 20 lines in your changes missing coverage. Please review.

Project coverage is 55.56%. Comparing base (8a447d9) to head (3b3616f).
Report is 1 commits behind head on master.

@@            Coverage Diff             @@
##           master   #21846      +/-   ##
==========================================
- Coverage   55.60%   55.56%   -0.05%     
==========================================
  Files         339      339              
  Lines       56869    56893      +24     
==========================================
- Hits        31622    31612      -10     
- Misses      22596    22630      +34     
  Partials     2651     2651              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I'm not sure that the namespace selector should cause namespaces to be ignored. It seems like I would want to have the list of namespaces + labeled namespaces. WDYT?

Yeah, I considered both options and ended up proposing to make it exclusive since labelling namespaces is straightforward and aligns better with Kubernetes-native practices.

This approach also makes it easier to phase out the CSV method for specifying namespaces over time.

That said, I'm happy with either option—I'll leave it to the maintainers, as you have more insight into how this is being used and where you want to take it in the future.