fix: virtual-ns-prefix into sim CRD

[drmorr0]

Related Links

• follow-up from sk-120/sk-32 fix: Add ca_bundle validation for MutatingWebhookConfiguration #207

Description and Rationale

• Move the virtual ns prefix out of the CLI arguments and into the CRD, because the CRD is easier to interact with/change on a per-sim basis

Test Steps

• make test passes
• skctl run works as expected

Other Notes

• As in fix: make the driver secrets part of the CRD #208 this is arguably not a patch bump but since the CRD is not installed in "real" clusters I think it's fine

• I waffled back and forth on whether to make virtual_ns_prefix "required", "required with default", or "optional". Optional would be the most API-preserving, but I do think this "should" be a required field. Then I took the default out because the serde default stuff is a pain, but this way users can (if they want) write their own tools for interacting with simkube and get sane-ish behaviour. The default is then re-specified in skctl which is kindof annoying but whatever.

• Re-ordered some stuff to make it alphabetical

• I certify that this PR does not contain any code that has been generated with GitHub Copilot or any other AI-based code generation tool, in accordance with this project's policies.

[drmorr0]

Kubernetes Object DAG

%%{init: {'themeVariables': {'mainBkg': '#ddd'}}}%%
graph LR

classDef default color:#000
subgraph global
  direction LR
  global/simkube[<b>Namespace</b><br>simkube]
%% DELETED OBJECTS START
%% DELETED OBJECTS END
end

subgraph sk-tracer
  direction LR
  simkube/sk-tracer-svc[<b>Service</b><br>sk-tracer-svc]
  simkube/sk-tracer-depl[<b>Deployment</b><br>sk-tracer-depl]
  simkube/sk-tracer-sa[<b>ServiceAccount</b><br>sk-tracer-sa]
  sk-tracer/sk-tracer-crb[<b>ClusterRoleBinding</b><br>sk-tracer-crb]
  simkube/sk-tracer-tracer-config[<b>ConfigMap</b><br>sk-tracer-tracer-config]
  simkube/sk-tracer-sa--->simkube/sk-tracer-depl
  sk-tracer/sk-tracer-crb--->simkube/sk-tracer-depl
  simkube/sk-tracer-tracer-config--->simkube/sk-tracer-depl
%% DELETED OBJECTS START
%% DELETED OBJECTS END
end

subgraph sk-ctrl
  direction LR
  simkube/sk-ctrl-depl[<b>Deployment</b><br>sk-ctrl-depl]
  simkube/sk-ctrl-sa[<b>ServiceAccount</b><br>sk-ctrl-sa]
  sk-ctrl/sk-ctrl-crb[<b>ClusterRoleBinding</b><br>sk-ctrl-crb]
  simkube/sk-ctrl-sa--->simkube/sk-ctrl-depl
  sk-ctrl/sk-ctrl-crb--->simkube/sk-ctrl-depl
%% DELETED OBJECTS START
%% DELETED OBJECTS END
end

global--->sk-tracer
global--->sk-ctrl

%% STYLE DEFINITIONS START
%% STYLE DEFINITIONS END

Loading

New object
Deleted object
Updated object
Updated object (causes pod recreation)

Detailed Diff

[codecov]

Codecov Report

❌ Patch coverage is 25.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.60%. Comparing base (97cabf7) to head (c4fb213).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
sk-cli/src/run.rs	0.00%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
- Coverage   75.63%   75.60%   -0.04%     
==========================================
  Files          58       58              
  Lines        3292     3291       -1     
  Branches      164      164              
==========================================
- Hits         2490     2488       -2     
- Misses        696      697       +1     
  Partials      106      106              

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[drmorr0]

@ogorman89 just FYI, this is the change we discussed in the last PR