Unverified Commit 6835dd7a authored by Neelay Shah's avatar Neelay Shah Committed by GitHub
Browse files

fix: updating test to not set port (#1823)

parent ece76a62
......@@ -276,10 +276,8 @@ class DynamoServeProcess(ManagedProcess):
# Handle multimodal deployments differently
if "multimodal" in graph.directory:
# Set DYNAMO_PORT environment variable for multimodal
env = os.environ.copy()
env["DYNAMO_PORT"] = str(port)
# Don't add health check on port since multimodal uses DYNAMO_PORT
# port is currently required to be 8000
assert port == 8000
else:
# Regular LLM deployments
command.extend(["--Frontend.port", str(port)])
......
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