Skip to content

Conversation

@prassanna-ravishankar
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR exposes the health check port for Temporal workers as an environment variable HEALTH_CHECK_PORT to make it configurable through agent configurations.

  • Added health_check_port field to TemporalConfig class with default value of None
  • Introduced HEALTH_CHECK_PORT environment variable with default value of 80
  • Updated worker initialization to read health check port from environment variable instead of hardcoded value

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/agentex/lib/types/agent_configs.py Added health_check_port field to TemporalConfig
src/agentex/lib/environment_variables.py Added HEALTH_CHECK_PORT environment variable definition
src/agentex/lib/core/temporal/workers/worker.py Updated worker to read health check port from environment
src/agentex/lib/cli/templates/temporal/manifest.yaml.j2 Added commented configuration example for health_check_port
src/agentex/lib/cli/handlers/run_handlers.py Added logic to set HEALTH_CHECK_PORT from temporal config

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@prassanna-ravishankar prassanna-ravishankar enabled auto-merge (squash) October 2, 2025 15:01
@prassanna-ravishankar prassanna-ravishankar merged commit e0236aa into main Oct 2, 2025
10 checks passed
@prassanna-ravishankar prassanna-ravishankar deleted the feat/temporal-health-check-ports branch October 2, 2025 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants