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

ethpandaops · May 13, 2024 · 2c6b7b1 · 2c6b7b1
1 parent b96e502
commit 2c6b7b1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -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

diff --git a/src/apache/apache_launcher.star b/src/apache/apache_launcher.star
@@ -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,

diff --git a/src/package_io/input_parser.star b/src/package_io/input_parser.star
@@ -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,