Skip to content

Commit 005dfec

Browse files
authored
Merge branch 'main' into x/uv-in-docker
2 parents 7b811c0 + e373614 commit 005dfec

File tree

2 files changed

+7
-13
lines changed

2 files changed

+7
-13
lines changed

src/agentex/lib/cli/handlers/deploy_handlers.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
from rich.console import Console
1212

1313
from agentex.lib.utils.logging import make_logger
14-
from agentex.lib.constants.ports import ACP_SERVER_PORT
1514
from agentex.lib.cli.utils.exceptions import HelmError, DeploymentError
1615
from agentex.lib.cli.utils.path_utils import PathResolutionError, calculate_docker_acp_module
1716
from agentex.lib.environment_variables import EnvVarKeys
@@ -240,12 +239,12 @@ def add_acp_command_to_helm_values(helm_values: dict[str, Any], manifest: AgentM
240239
try:
241240
docker_acp_module = calculate_docker_acp_module(manifest, manifest_path)
242241
# Create the uvicorn command with the correct module path
243-
helm_values["command"] = ["uvicorn", f"{docker_acp_module}:acp", "--host", "0.0.0.0", "--port", str(ACP_SERVER_PORT)]
242+
helm_values["command"] = ["uvicorn", f"{docker_acp_module}:acp", "--host", "0.0.0.0", "--port", "8000"]
244243
logger.info(f"Using dynamic ACP command: uvicorn {docker_acp_module}:acp")
245244
except (PathResolutionError, Exception) as e:
246245
# Fallback to default command structure
247246
logger.warning(f"Could not calculate dynamic ACP module ({e}), using default: project.acp")
248-
helm_values["command"] = ["uvicorn", "project.acp:acp", "--host", "0.0.0.0", "--port", str(ACP_SERVER_PORT)]
247+
helm_values["command"] = ["uvicorn", "project.acp:acp", "--host", "0.0.0.0", "--port", "8000"]
249248

250249

251250
def merge_deployment_configs(

src/agentex/lib/environment_variables.py

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
12
from __future__ import annotations
23

34
import os
@@ -7,12 +8,6 @@
78
from dotenv import load_dotenv
89

910
from agentex.lib.utils.logging import make_logger
10-
from agentex.lib.constants.ports import (
11-
ACP_SERVER_PORT as _DEFAULT_ACP_PORT,
12-
TEMPORAL_ADDRESS as _DEFAULT_TEMPORAL_ADDRESS,
13-
HEALTH_CHECK_PORT as _DEFAULT_HEALTH_CHECK_PORT,
14-
AGENTEX_API_BASE_URL as _DEFAULT_AGENTEX_BASE_URL,
15-
)
1611
from agentex.lib.utils.model_utils import BaseModel
1712

1813
PROJECT_ROOT = Path(__file__).resolve().parents[2]
@@ -61,9 +56,9 @@ class Environment(str, Enum):
6156

6257
class EnvironmentVariables(BaseModel):
6358
ENVIRONMENT: str = Environment.DEV
64-
TEMPORAL_ADDRESS: str | None = _DEFAULT_TEMPORAL_ADDRESS
59+
TEMPORAL_ADDRESS: str | None = "localhost:7233"
6560
REDIS_URL: str | None = None
66-
AGENTEX_BASE_URL: str | None = _DEFAULT_AGENTEX_BASE_URL
61+
AGENTEX_BASE_URL: str | None = "http://localhost:5003"
6762
# Agent Identifiers
6863
AGENT_NAME: str
6964
AGENT_DESCRIPTION: str | None = None
@@ -73,12 +68,12 @@ class EnvironmentVariables(BaseModel):
7368
AGENT_INPUT_TYPE: str | None = None
7469
# ACP Configuration
7570
ACP_URL: str
76-
ACP_PORT: int = _DEFAULT_ACP_PORT
71+
ACP_PORT: int = 8000
7772
# Workflow Configuration
7873
WORKFLOW_TASK_QUEUE: str | None = None
7974
WORKFLOW_NAME: str | None = None
8075
# Temporal Worker Configuration
81-
HEALTH_CHECK_PORT: int = _DEFAULT_HEALTH_CHECK_PORT
76+
HEALTH_CHECK_PORT: int = 80
8277
# Auth Configuration
8378
AUTH_PRINCIPAL_B64: str | None = None
8479
# Build Information

0 commit comments

Comments
 (0)