From 92e14a6e7b09bf4ea454c241a7643a89d4b12ac8 Mon Sep 17 00:00:00 2001 From: Edward Evans Date: Thu, 14 May 2020 15:44:38 +1000 Subject: [PATCH] Add formatting filter for BESU_ENV_OPTS It allows for the `besu_env_opts` variable to be either a whitespace separated string or a list Signed-off-by: Edward Evans --- README.md | 2 +- filter_plugins/besu_filters.py | 18 ++++++++++++++++++ templates/besu.service.j2 | 2 +- 3 files changed, 20 insertions(+), 2 deletions(-) create mode 100644 filter_plugins/besu_filters.py diff --git a/README.md b/README.md index 2d4ef4e..b3f2fc2 100644 --- a/README.md +++ b/README.md @@ -82,7 +82,7 @@ All variables which can be overridden are stored in [defaults/main.yml](defaults | `besu_permissions_accounts_contract_address` | ___unset___ | The contract address for onchain accounts permissioning | | `besu_permissions_nodes_contract_address` | ___unset___ | The contract address for onchain nodes permissioning | | `besu_cmdline_args` | "" | Command line args that are passed in as overrides | -| `besu_env_opts` | "" | Environmental variable BESU_OPTS that gets passed to the JVM. eg: -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005 | +| `besu_env_opts` | [] | Settings passed to the JVM through `BESU_OPTS` environment variable. eg: `-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005` | | `orion_version` | ___unset___ | Version of Orion installed. When supplied, adds required privacy config items and requires `orion_config_dir` and `orion_key_name` to be set | | `orion_config_dir` | ___unset___ | Path to Orion configuration directory | | `orion_key_name` | ___unset___ | Name of the Node Key | diff --git a/filter_plugins/besu_filters.py b/filter_plugins/besu_filters.py new file mode 100644 index 0000000..7bdba9e --- /dev/null +++ b/filter_plugins/besu_filters.py @@ -0,0 +1,18 @@ +#! /usr/bin/env python + +from typing import Union, List + + +class FilterModule(object): + def filters(self): + return { + 'besu_opts_format': self.besuOptsFormat, + } + + def besuOptsFormat(self, besu_env_opts: Union[str, List[str]]): + if isinstance(besu_env_opts, str): + besu_env_opts = besu_env_opts.split() + + escaped_besu_env_opts = [item.replace('"', '\\"') for item in besu_env_opts] + + return f'''"{'" "'.join(escaped_besu_env_opts)}"''' diff --git a/templates/besu.service.j2 b/templates/besu.service.j2 index 610265c..9dd5d23 100644 --- a/templates/besu.service.j2 +++ b/templates/besu.service.j2 @@ -9,7 +9,7 @@ Environment=HOME=/home/{{ besu_user }} {% if besu_log4j_config_file %} Environment=LOG4J_CONFIGURATION_FILE={{ besu_log4j_config_file }} {% endif %} -Environment=BESU_OPTS={{ besu_env_opts }} +Environment='BESU_OPTS={{ besu_env_opts|besu_opts_format }}' Type=simple {% if besu_cmdline_args %} ExecStart=/bin/sh -c "{{ besu_current_dir }}/bin/besu --config-file={{ besu_config_dir }}/config.toml {{ besu_cmdline_args|quote }} >> {{besu_log_dir}}/besu.log 2>&1"