Skip to content

Kubernetes executionconfig API should return effective config#19014

Open
GWphua wants to merge 5 commits intoapache:masterfrom
GWphua:dynamic-config
Open

Kubernetes executionconfig API should return effective config#19014
GWphua wants to merge 5 commits intoapache:masterfrom
GWphua:dynamic-config

Conversation

@GWphua
Copy link
Contributor

@GWphua GWphua commented Feb 12, 2026

Description

#18591 introduced the capability to change the max number of K8s tasks created by Overlord, and we can track the values using /druid/indexer/v1/k8s/taskrunner/executionconfig` API.

Execution Config Returns Static Config by Default

Currently, calling GET on the /druid/indexer/v1/k8s/taskrunner/executionconfig API will return an empty payload if we did not submit any dynamic configs via POST on the same API. Users who depend on the API to retrieve the currently effective K8s task runner information will not see the information before the first dynamic config change is submitted.

Changed the API to return the effective config, so the users will always see what is currently being used.

Release note

Users now can retrieve the Kubernetes default configuration via GET on /druid/indexer/v1/k8s/taskrunner/executionconfig when there's no dynamic configs submitted.


This PR has:

  • been self-reviewed.
  • a release note entry in the PR description.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • been tested in a test Druid cluster.

@GWphua GWphua changed the title Dynamic config Kubernetes executionconfig API should return something even when no dynamic config is submitted Feb 12, 2026
@GWphua GWphua changed the title Kubernetes executionconfig API should return something even when no dynamic config is submitted Kubernetes executionconfig API should return effective config Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant