Commit 2c6b7b1a authored by Barnabas Busa's avatar Barnabas Busa Committed by GitHub

fix(apache): only set static port if wanted (#610)

parent b96e5021
......@@ -740,8 +740,8 @@ xatu_sentry_params:
# Apache params
# Apache public port to port forward to local machine
# Default to port 40000, only set if apache additional service is activated
apache_port: 40000
# Default to port None, only set if apache additional service is activated
apache_port: null
# Global tolerations that will be passed to all containers (unless overridden by a more specific toleration)
# Only works with Kubernetes
......
......@@ -79,10 +79,13 @@ def launch_apache(
bootstrap_info_files_artifact_name = plan.render_templates(
template_and_data_by_rel_dest_filepath, "bootstrap-info"
)
public_ports = {
HTTP_PORT_ID: shared_utils.new_port_spec(apache_port, shared_utils.TCP_PROTOCOL)
}
public_ports = {}
if apache_port != None:
public_ports = {
HTTP_PORT_ID: shared_utils.new_port_spec(
apache_port, shared_utils.TCP_PROTOCOL
)
}
config = get_config(
config_files_artifact_name,
......
......@@ -668,7 +668,7 @@ def default_input_args():
"persistent": False,
"mev_type": None,
"xatu_sentry_enabled": False,
"apache_port": 40000,
"apache_port": None,
"global_tolerations": [],
"global_node_selectors": {},
"keymanager_enabled": False,
......
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