Skip to content

Commit af7dfcd

Browse files
committed
fix(e2e): add default catch-all decision to dynamic-config CRD
The dynamic-config E2E profile uses Kubernetes CRDs (config_source: kubernetes) instead of config/testing/config.e2e.yaml, so the default decision added to the YAML file was being ignored. Root cause: E2E tests send model="MoM" which triggers auto-routing, but when no domain matches, no decision is selected, causing PII detection to be disabled ("No decision specified, PII detection disabled"). This adds a priority=1 catch-all decision to the CRD that ensures PII detection is always enabled for unmatched requests, blocking all PII types by default. Signed-off-by: Yossi Ovadia <[email protected]>
1 parent c298067 commit af7dfcd

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

e2e/profiles/dynamic-config/crds/intelligentroute.yaml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -344,3 +344,20 @@ spec:
344344
enabled: true
345345
system_prompt: "You are an engineering expert with knowledge across multiple engineering disciplines including mechanical, electrical, civil, chemical, software, and systems engineering. Apply engineering principles, design methodologies, and problem-solving approaches to provide practical solutions. Consider safety, efficiency, sustainability, and cost-effectiveness in your recommendations. Use technical precision while explaining concepts clearly, and emphasize the importance of proper engineering practices and standards."
346346
mode: "replace"
347+
348+
- name: "default_decision"
349+
priority: 1
350+
description: "Default catch-all decision - blocks all PII for safety"
351+
signals:
352+
operator: "OR"
353+
conditions:
354+
- type: "always"
355+
modelRefs:
356+
- model: "base-model"
357+
loraName: "science-expert"
358+
useReasoning: false
359+
plugins:
360+
- type: "pii"
361+
configuration:
362+
enabled: true
363+
pii_types_allowed: []

0 commit comments

Comments
 (0)