Merge pull request #23 from kurtosis-tech/gyani/remove-used-oprts

remove ports that were passed empty

docs/changelog.md

@@ -7,6 +7,7 @@
 - Renamed `num_validators_per_keynode` to `num_validator_keys_per_node`
 - Moved away from `load` infavor of `import_module`
 - Moved away from `store_files_from_service` to `store_service_files`
+- Removed empty `ports` from a few service configs as passing it is now optional
 
 # 0.0.1
 

src/participant_network/prelaunch_data_generator/prelaunch_data_generator_launcher/prelaunch_data_generator_launcher.star

@@ -26,8 +26,6 @@ def get_config(
 	files_artifact_mountpoints,
 ):
 	return struct(
-		# TODO remove this when ports is optional to pass
-		ports = {},
 		image = IMAGE,
 		entrypoint = ENTRYPOINT_ARGS,
 		files = files_artifact_mountpoints,

src/testnet_verifier/testnet_verifier.star

@@ -54,8 +54,6 @@ def get_asynchronous_verification_config(params, el_client_contexts, cl_client_c
 	return struct(
 		image = IMAGE_NAME,
 		cmd = commands,
-		# TODO remove this when ports is optional in add_service
-		ports = {},
 	)
 
 
@@ -63,6 +61,4 @@ def get_synchronous_verification_config():
 	return struct(
 		image = IMAGE_NAME,
 		entrypoint = SYNCHRONOUS_ENTRYPOINT_ARGS,
-		# TODO remove this when ports is optional in add_service
-		ports = {},
 	)

src/transaction_spammer/transaction_spammer.star

@@ -23,7 +23,6 @@ def get_config(prefunded_addresses, el_client_context):
 			"spam",
 			comma_separated_private_keys,
 			comma_separated_addresses
-		],
-		ports = {}
+		]
 	)