Skip to content

Commit f7a957b

Browse files
committed
fix(aibrix): add default_decision to aibrix values.yaml for PII fallback
The aibrix profile uses static YAML configuration (not CRDs). The PII policy checker has a fallback to 'default_decision' when no decision name is provided, but aibrix didn't define this decision. This caused all PII detection requests to be allowed (0% accuracy). Adding default_decision with PII enabled (pii_types_allowed: []) ensures that the fallback logic works correctly for aibrix, matching the behavior expected by the E2E PII detection tests. Related to #648 Signed-off-by: Yossi Ovadia <[email protected]>
1 parent 444fbf8 commit f7a957b

File tree

1 file changed

+17
-0
lines changed
  • deploy/kubernetes/aibrix/semantic-router-values

1 file changed

+17
-0
lines changed

deploy/kubernetes/aibrix/semantic-router-values/values.yaml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -380,6 +380,23 @@ config:
380380
system_prompt: "You are a thinking expert, should think multiple steps before answering. Please answer the question step by step."
381381
mode: "replace"
382382

383+
- name: default_decision
384+
description: "Default fallback decision for requests without specific routing"
385+
priority: 1
386+
rules:
387+
operator: "OR"
388+
conditions:
389+
- type: "domain"
390+
name: "other"
391+
modelRefs:
392+
- model: vllm-llama3-8b-instruct
393+
use_reasoning: false
394+
plugins:
395+
- type: "pii"
396+
configuration:
397+
enabled: true
398+
pii_types_allowed: []
399+
383400
# Strategy for selecting between multiple matching decisions
384401
# Options: "priority" (use decision with highest priority) or "confidence" (use decision with highest confidence)
385402
strategy: "priority"

0 commit comments

Comments
 (0)