Commit 998063fa authored by Luca | Serenita's avatar Luca | Serenita Committed by GitHub

fix: provide `--network-custom-config-path` to Vero (#905)

As of `v0.9.0` Vero [requires a network to be specified as a CLI
argument](https://github.com/serenita-org/vero/releases/tag/v0.9.0).
This PR adds that to the Kurtosis definition for the Vero VC service.

(In case you're wondering - how did Vero not have a `--network` flag
before? It used to load all the spec values from the connected beacon
node(s) and would then dynamically create all necessary SSZ classes.)

I tested this manually with the `.github/tests/vero-all.yaml` config and
have been using this version for the last few months while preparing for
Pectra.
parent 53602f1b
...@@ -159,6 +159,7 @@ def launch( ...@@ -159,6 +159,7 @@ def launch(
fail("vero VC doesn't support the Keymanager API") fail("vero VC doesn't support the Keymanager API")
config = vero.get_config( config = vero.get_config(
participant=participant, participant=participant,
el_cl_genesis_data=launcher.el_cl_genesis_data,
image=image, image=image,
global_log_level=global_log_level, global_log_level=global_log_level,
beacon_http_url=beacon_http_url, beacon_http_url=beacon_http_url,
......
...@@ -14,6 +14,7 @@ VERBOSITY_LEVELS = { ...@@ -14,6 +14,7 @@ VERBOSITY_LEVELS = {
def get_config( def get_config(
participant, participant,
el_cl_genesis_data,
image, image,
global_log_level, global_log_level,
beacon_http_url, beacon_http_url,
...@@ -30,6 +31,10 @@ def get_config( ...@@ -30,6 +31,10 @@ def get_config(
) )
cmd = [ cmd = [
"--network=custom",
"--network-custom-config-path="
+ constants.GENESIS_CONFIG_MOUNT_PATH_ON_CONTAINER
+ "/config.yaml",
"--remote-signer-url={0}".format(remote_signer_context.http_url), "--remote-signer-url={0}".format(remote_signer_context.http_url),
"--beacon-node-urls=" + beacon_http_url, "--beacon-node-urls=" + beacon_http_url,
"--fee-recipient=" + constants.VALIDATING_REWARDS_ACCOUNT, "--fee-recipient=" + constants.VALIDATING_REWARDS_ACCOUNT,
...@@ -43,6 +48,10 @@ def get_config( ...@@ -43,6 +48,10 @@ def get_config(
# this is a repeated<proto type>, we convert it into Starlark # this is a repeated<proto type>, we convert it into Starlark
cmd.extend([param for param in participant.vc_extra_params]) cmd.extend([param for param in participant.vc_extra_params])
files = {
constants.GENESIS_DATA_MOUNTPOINT_ON_CLIENTS: el_cl_genesis_data.files_artifact_uuid,
}
public_ports = {} public_ports = {}
if port_publisher.vc_enabled: if port_publisher.vc_enabled:
public_ports_for_component = shared_utils.get_public_ports_for_component( public_ports_for_component = shared_utils.get_public_ports_for_component(
...@@ -61,6 +70,7 @@ def get_config( ...@@ -61,6 +70,7 @@ def get_config(
"ports": ports, "ports": ports,
"public_ports": public_ports, "public_ports": public_ports,
"cmd": cmd, "cmd": cmd,
"files": files,
"env_vars": participant.vc_extra_env_vars, "env_vars": participant.vc_extra_env_vars,
"labels": shared_utils.label_maker( "labels": shared_utils.label_maker(
client=constants.VC_TYPE.vero, client=constants.VC_TYPE.vero,
......
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