fix

Signed-off-by: jerryzhuang <[email protected]>
kaito-project · Dec 20, 2024 · ca9edad · ca9edad
1 parent 071d9fe
commit ca9edad
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/e2e-preset-configs.json b/.github/e2e-preset-configs.json
@@ -99,7 +99,7 @@
         "loads_adapter": false
       },
       {
-        "name": "qwen2-5-coder-7b-instruct",
+        "name": "qwen2.5-coder-7b-instruct",
         "node-count": 1,
         "node-vm-size": "Standard_NC12s_v3",
         "node-osdisk-size": 100,

diff --git a/.github/workflows/e2e-preset-test.yml b/.github/workflows/e2e-preset-test.yml
@@ -84,13 +84,15 @@ jobs:
             # for model in MATRIX:
             #     for config in CONFIGS:
             #         if config['name'] == model['name']:
+            #             # k8s doesn't allow '.' in names
+            #             model['name'] = model['name'].replace('.', '-')
             #             combined = {**model, **config}
             #             COMBINED_MATRIX.append(combined)
             #             break
 
             COMBINED_MATRIX=$(echo '${{ steps.affected_models.outputs.matrix }}' | jq --argjson configs "$CONFIGS" -c '
                 map(. as $model | $configs[] | select(.name == $model.name) | $model + .)
-            ')
+            ' | jq -c 'map(.name |= gsub("\\."; "-"))')
 
             echo "full_matrix=$COMBINED_MATRIX" >> $GITHUB_OUTPUT