comparison roundup/scripts/roundup_demo.py @ 7925:f5f09ed9a783

fix: roundup_demo should use port assignment There are three port values: urlport (-P) - used in the URL when docker maps real port to external port port (-p) - used to set port roundup listens on DEFAULT_PORT = 8917 used for -p if not set. Before the -p argument was not used if -P was not set. So you can override the default 8917.
author John Rouillard <rouilj@ieee.org>
date Thu, 02 May 2024 21:21:40 -0400
parents c73a1177c2b2
children 05f36a527a45
comparison
equal deleted inserted replaced
7924:141225d2981e 7925:f5f09ed9a783
168 print("Unknown template: %s. Exiting." % template) 168 print("Unknown template: %s. Exiting." % template)
169 exit(1) 169 exit(1)
170 # install 170 # install
171 demo.install_demo(home, backend, 171 demo.install_demo(home, backend,
172 admin.AdminTool().listTemplates()[template]['path'], 172 admin.AdminTool().listTemplates()[template]['path'],
173 use_port=cli_args.urlport or DEFAULT_PORT) 173 use_port=cli_args.urlport or cli_args.port or DEFAULT_PORT)
174 else: 174 else:
175 # make sure that no options are specified that are only useful on initialization. 175 # make sure that no options are specified that are only useful on initialization.
176 if ( cli_args.backend or cli_args.template or 176 if ( cli_args.backend or cli_args.template or
177 cli_args.backend_db ): 177 cli_args.backend_db ):
178 usage(home, cli, msg=( 178 usage(home, cli, msg=(

Roundup Issue Tracker: http://roundup-tracker.org/