Skip to content

Add query-manager.required-workers guidance to KEDA documentation#409

Open
jmealo wants to merge 1 commit intotrinodb:mainfrom
jmealo:docs/keda-required-workers-guidance
Open

Add query-manager.required-workers guidance to KEDA documentation#409
jmealo wants to merge 1 commit intotrinodb:mainfrom
jmealo:docs/keda-required-workers-guidance

Conversation

@jmealo
Copy link
Copy Markdown

@jmealo jmealo commented Mar 27, 2026

Summary

  • Add documentation to the server.keda section in values.yaml explaining
    how to configure query-manager.required-workers and
    query-manager.required-workers-max-wait via additionalConfigProperties
    when using KEDA with scale-to-zero
  • Include a YAML example showing the recommended configuration
  • Regenerate README.md via helm-docs

Without these Trino-side settings, queries submitted while workers are
scaled to zero fail immediately with an "Insufficient active worker nodes"
error instead of entering a WAITING_FOR_RESOURCES state and waiting for
workers to start.

The referenced Trino properties are being documented in trinodb/trino#28902.

Test plan

  • Verify helm-docs generates the README correctly
  • Confirm the additionalConfigProperties example is valid

Document how to configure query-manager.required-workers and
query-manager.required-workers-max-wait in additionalConfigProperties
when using KEDA with scale-to-zero. Without these settings, queries
submitted while workers are scaled to zero fail immediately instead
of waiting for workers to start.
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Mar 27, 2026

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

@jmealo
Copy link
Copy Markdown
Author

jmealo commented Mar 27, 2026

CLA sent just prior to pening this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant