Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helm Chart - runMonitoring: maxRuntimeSeconds not configurable #26641

Open
dbrocha opened this issue Dec 20, 2024 · 0 comments
Open

Helm Chart - runMonitoring: maxRuntimeSeconds not configurable #26641

dbrocha opened this issue Dec 20, 2024 · 0 comments
Labels
deployment: k8s Related to deploying Dagster to Kubernetes type: bug Something isn't working

Comments

@dbrocha
Copy link

dbrocha commented Dec 20, 2024

What's the issue?

I'd like to add a max runtime seconds to my helm chart. Referencing the dagster.yaml config described here: https://docs.dagster.io/deployment/run-monitoring#general-run-timeouts, I added the following to my helm chart -

runMonitoring:
    enabled: true
    # Timeout for runs to start (avoids runs hanging in STARTING)
    startTimeoutSeconds: 300
    # Timeout for runs to cancel (avoids runs hanging in CANCELING)
    # cancelTimeoutSeconds: 300
    # How often to check on in progress runs
    pollIntervalSeconds: 120
    maxRuntimeSeconds: 10800

This setting does not show in the front end configuration display, however:

run_monitoring:
  enabled: true
  start_timeout_seconds: 300
  max_resume_run_attempts: 0
  poll_interval_seconds: 120
  free_slots_after_run_end_seconds: 0

What did you expect to happen?

No response

How to reproduce?

No response

Dagster version

1.9.6

Deployment type

Dagster Helm chart

Deployment details

No response

Additional information

No response

Message from the maintainers

Impacted by this issue? Give it a 👍! We factor engagement into prioritization.
By submitting this issue, you agree to follow Dagster's Code of Conduct.

@dbrocha dbrocha added the type: bug Something isn't working label Dec 20, 2024
@garethbrickman garethbrickman added the deployment: k8s Related to deploying Dagster to Kubernetes label Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment: k8s Related to deploying Dagster to Kubernetes type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants