Commit c8b9b19c authored by pk910's avatar pk910 Committed by GitHub

fix: allow vc properties in participants_matrix.cl (#715)

allow setting vc_* properties in participants_matrix.cl:

```
participants_matrix:
  el:
    - el_type: nethermind
      el_image: nethermindeth/nethermind:pectra
    - el_type: geth
      el_image: ethpandaops/geth:lightclient-prague-devnet-1
  cl:
    - cl_type: teku
      cl_image: ethpandaops/teku:master
    - cl_type: nimbus
      cl_image: ethpandaops/nimbus-eth2:unstable
    - cl_type: lodestar
      cl_image: ethpandaops/lodestar:electra-fork
      vc_max_mem: 1024
    - cl_type: grandine
      cl_image: ethpandaops/grandine:electra
```

in the example above, lodestar vc needs more than 500M memory for key
import, so I'd like to override the vc_max_mem accordingly.
This is theoretically allowed by the logic, but the new sanity check
prevents it.
parent b824cac8
...@@ -82,6 +82,18 @@ PARTICIPANT_MATRIX_PARAMS = { ...@@ -82,6 +82,18 @@ PARTICIPANT_MATRIX_PARAMS = {
"cl_min_mem", "cl_min_mem",
"cl_max_mem", "cl_max_mem",
"use_separate_vc", "use_separate_vc",
"vc_type",
"vc_image",
"vc_count",
"vc_log_level",
"vc_extra_env_vars",
"vc_extra_labels",
"vc_extra_params",
"vc_tolerations",
"vc_min_cpu",
"vc_max_cpu",
"vc_min_mem",
"vc_max_mem",
], ],
"vc": [ "vc": [
"vc_type", "vc_type",
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment