From 160bf0e0e8fa1cddd5900fff69d704f03ee4ae0b Mon Sep 17 00:00:00 2001 From: Will Greenberg Date: Fri, 9 Feb 2024 10:57:50 -0800 Subject: [PATCH] Remove unused `--acme-server` argument Since this argument is never set and always defaults to 'pebble', just remove it to simplify assumptions about which test server's being used. --- certbot-ci/certbot_integration_tests/utils/acme_server.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/certbot-ci/certbot_integration_tests/utils/acme_server.py b/certbot-ci/certbot_integration_tests/utils/acme_server.py index 3802c6197..748910ea2 100755 --- a/certbot-ci/certbot_integration_tests/utils/acme_server.py +++ b/certbot-ci/certbot_integration_tests/utils/acme_server.py @@ -283,10 +283,6 @@ def main() -> None: # pylint: disable=missing-function-docstring parser = argparse.ArgumentParser( description='CLI tool to start a local instance of Pebble or Boulder CA server.') - parser.add_argument('--server-type', '-s', - choices=['pebble', 'boulder-v2'], default='pebble', - help='type of CA server to start: can be Pebble or Boulder. ' - 'Pebble is used if not set.') parser.add_argument('--dns-server', '-d', help='specify the DNS server as `IP:PORT` to use by ' 'Pebble; if not specified, a local mock DNS server will be used to ' @@ -297,7 +293,7 @@ def main() -> None: args = parser.parse_args() acme_server = ACMEServer( - args.server_type, [], http_proxy=False, stdout=True, + 'pebble', [], http_proxy=False, stdout=True, dns_server=args.dns_server, http_01_port=args.http_01_port, )