Skip to content

infra: deploy orcha with helm#27

Open
mairasalazar wants to merge 3 commits intoinveniosoftware:mainfrom
mairasalazar:dev-deployment
Open

infra: deploy orcha with helm#27
mairasalazar wants to merge 3 commits intoinveniosoftware:mainfrom
mairasalazar:dev-deployment

Conversation

@mairasalazar
Copy link
Copy Markdown
Collaborator

@mairasalazar mairasalazar commented Mar 20, 2026

❤️ Thank you for your review!

Description

Adds the Helm chart and a generic values.yaml with non-sensitive configuration for the Orcha deployment.

Orcha accessible here: https://orcha.app.cern.ch/

Addresses #3

Comment thread app/cli/main.py
"""Start the Temporal worker."""
typer.echo("Starting Temporal worker...")
result = subprocess.run(
["uv", "run", "python", "-m", "app.workers"],
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Changed the CLI command to use sys.executable instead of uv run python, because then we use the interpreter currently running without creating a cache and failing due to permissions

Comment thread Dockerfile

# Run the application.
CMD ["/app/.venv/bin/fastapi", "run", "app/main.py", "--port", "80"]
CMD ["/app/.venv/bin/fastapi", "run", "app/main.py", "--port", "8080"]
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Changed Dockerfile CMD port from 80 to 8080 because port 80 requires root privileges and OpenShift runs containers as non-root

@yashlamba yashlamba linked an issue Apr 9, 2026 that may be closed by this pull request
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.

Infra: deploy orcha with helm

1 participant