# HG changeset patch # User Goffi # Date 1630157578 -7200 # Node ID 7bc443253b7cc7981dde3cbdbd09ddc674d46786 # Parent 597a535ee18777b3b250e775a5e0f6c3182244ba# Parent 3c7a64d6f49fad2cf7761e985101e9d5c88ac6d8 merge branche "@" diff -r 597a535ee187 -r 7bc443253b7c doc/libervia-cli/pubsub.rst --- a/doc/libervia-cli/pubsub.rst Fri Aug 27 15:27:14 2021 +0200 +++ b/doc/libervia-cli/pubsub.rst Sat Aug 28 15:32:58 2021 +0200 @@ -346,7 +346,7 @@ else: print(item_raw.replace("SàT", "Libervia")) -And save it a some location, e.g. ``~/expand_sat.py`` (don't forget to make is executable +And save it a some location, e.g. ``~/expand_sat.py`` (don't forget to make it executable with ``chmod +x ~/expand_sat.py``). To be sure it's safe, you can first do a dry-run and check the result:: diff -r 597a535ee187 -r 7bc443253b7c sat/bridge/bridge_constructor/constructors/pb/pb_core_template.py --- a/sat/bridge/bridge_constructor/constructors/pb/pb_core_template.py Fri Aug 27 15:27:14 2021 +0200 +++ b/sat/bridge/bridge_constructor/constructors/pb/pb_core_template.py Sat Aug 28 15:32:58 2021 +0200 @@ -19,6 +19,7 @@ import dataclasses +from functools import partial from pathlib import Path from twisted.spread import jelly, pb from twisted.internet import reactor @@ -106,12 +107,8 @@ log.info("Init Perspective Broker...") self.root = PBRoot() conf = config.parseMainConf() - conn_type = config.getConfig( - conf, - "", - "bridge_pb_connection_type", - "unix_socket" - ) + getConf = partial(config.getConf, conf, "bridge_pb", "") + conn_type = getConf("connection_type", "unix_socket") if conn_type == "unix_socket": local_dir = Path(config.getConfig(conf, "", "local_dir")).resolve() socket_path = local_dir / "bridge_pb" @@ -120,12 +117,7 @@ str(socket_path), pb.PBServerFactory(self.root), mode=0o600 ) elif conn_type == "socket": - port = int(config.getConfig( - conf, - "", - "bridge_pb_port", - 8789 - )) + port = int(getConf("port", 8789)) log.info(f"using TCP Socket at port {port}") reactor.listenTCP(port, pb.PBServerFactory(self.root)) else: diff -r 597a535ee187 -r 7bc443253b7c sat/bridge/bridge_constructor/constructors/pb/pb_frontend_template.py --- a/sat/bridge/bridge_constructor/constructors/pb/pb_frontend_template.py Fri Aug 27 15:27:14 2021 +0200 +++ b/sat/bridge/bridge_constructor/constructors/pb/pb_frontend_template.py Sat Aug 28 15:32:58 2021 +0200 @@ -140,29 +140,15 @@ def bridgeConnect(self, callback, errback): factory = pb.PBClientFactory() conf = config.parseMainConf() - conn_type = config.getConfig( - conf, - "", - "bridge_pb_connection_type", - "unix_socket" - ) + getConf = partial(config.getConf, conf, "bridge_pb", "") + conn_type = getConf("connection_type", "unix_socket") if conn_type == "unix_socket": local_dir = Path(config.getConfig(conf, "", "local_dir")).resolve() socket_path = local_dir / "bridge_pb" reactor.connectUNIX(str(socket_path), factory) elif conn_type == "socket": - host = int(config.getConfig( - conf, - "", - "bridge_pb_host", - "localhost" - )) - port = int(config.getConfig( - conf, - "", - "bridge_pb_port", - 8789 - )) + host = getConf("host", "localhost") + port = int(getConf("port", 8789)) reactor.connectTCP(host, port, factory) else: raise ValueError(f"Unknown pb connection type: {conn_type!r}") diff -r 597a535ee187 -r 7bc443253b7c sat/bridge/pb.py --- a/sat/bridge/pb.py Fri Aug 27 15:27:14 2021 +0200 +++ b/sat/bridge/pb.py Sat Aug 28 15:32:58 2021 +0200 @@ -19,6 +19,7 @@ import dataclasses +from functools import partial from pathlib import Path from twisted.spread import jelly, pb from twisted.internet import reactor @@ -106,12 +107,8 @@ log.info("Init Perspective Broker...") self.root = PBRoot() conf = config.parseMainConf() - conn_type = config.getConfig( - conf, - "", - "bridge_pb_connection_type", - "unix_socket" - ) + getConf = partial(config.getConf, conf, "bridge_pb", "") + conn_type = getConf("connection_type", "unix_socket") if conn_type == "unix_socket": local_dir = Path(config.getConfig(conf, "", "local_dir")).resolve() socket_path = local_dir / "bridge_pb" @@ -120,12 +117,7 @@ str(socket_path), pb.PBServerFactory(self.root), mode=0o600 ) elif conn_type == "socket": - port = int(config.getConfig( - conf, - "", - "bridge_pb_port", - 8789 - )) + port = int(getConf("port", 8789)) log.info(f"using TCP Socket at port {port}") reactor.listenTCP(port, pb.PBServerFactory(self.root)) else: diff -r 597a535ee187 -r 7bc443253b7c sat/core/sat_main.py --- a/sat/core/sat_main.py Fri Aug 27 15:27:14 2021 +0200 +++ b/sat/core/sat_main.py Sat Aug 28 15:32:58 2021 +0200 @@ -86,7 +86,10 @@ trigger.TriggerManager() ) - bridge_name = self.memory.getConfig("", "bridge", "dbus") + bridge_name = ( + os.getenv("LIBERVIA_BRIDGE_NAME") + or self.memory.getConfig("", "bridge", "dbus") + ) bridge_module = dynamic_import.bridge(bridge_name) if bridge_module is None: diff -r 597a535ee187 -r 7bc443253b7c sat/tools/config.py --- a/sat/tools/config.py Fri Aug 27 15:27:14 2021 +0200 +++ b/sat/tools/config.py Sat Aug 28 15:32:58 2021 +0200 @@ -23,6 +23,7 @@ import os import csv import json +from typing import Any from configparser import ConfigParser, DEFAULTSECT, NoOptionError, NoSectionError from xdg import BaseDirectory from sat.core.log import getLogger @@ -142,3 +143,29 @@ except ValueError as e: raise exceptions.ParsingError("Error while parsing data: {}".format(e)) return value + + +def getConf( + conf: ConfigParser, + prefix: str, + section: str, + name: str, + default: Any +) -> Any: + """Get configuration value from environment or config file + + @param str: prefix to use for the varilable name (see `name` below) + @param section: config section to use + @param name: unsuffixed name. + For environment variable, `LIBERVIA__` will be prefixed (and name + will be set to uppercase). + For config file, `_` will be prefixed (and DEFAULT section will be + used). + Environment variable has priority over config values. If Environment variable + is set but empty string, config value will be used. + @param default: default value to use if varilable is set neither in environment, + nor in config + """ + # XXX: This is a temporary method until parameters are refactored + value = os.getenv(f"LIBERVIA_{prefix}_{name}".upper()) + return value or getConfig(conf, section, f"{prefix}_{name}", default) diff -r 597a535ee187 -r 7bc443253b7c sat_frontends/bridge/pb.py --- a/sat_frontends/bridge/pb.py Fri Aug 27 15:27:14 2021 +0200 +++ b/sat_frontends/bridge/pb.py Sat Aug 28 15:32:58 2021 +0200 @@ -140,29 +140,15 @@ def bridgeConnect(self, callback, errback): factory = pb.PBClientFactory() conf = config.parseMainConf() - conn_type = config.getConfig( - conf, - "", - "bridge_pb_connection_type", - "unix_socket" - ) + getConf = partial(config.getConf, conf, "bridge_pb", "") + conn_type = getConf("connection_type", "unix_socket") if conn_type == "unix_socket": local_dir = Path(config.getConfig(conf, "", "local_dir")).resolve() socket_path = local_dir / "bridge_pb" reactor.connectUNIX(str(socket_path), factory) elif conn_type == "socket": - host = int(config.getConfig( - conf, - "", - "bridge_pb_host", - "localhost" - )) - port = int(config.getConfig( - conf, - "", - "bridge_pb_port", - 8789 - )) + host = getConf("host", "localhost") + port = int(getConf("port", 8789)) reactor.connectTCP(host, port, factory) else: raise ValueError(f"Unknown pb connection type: {conn_type!r}")