Skip to content

fix: use --cconfig for coordinator cluster config - #10

Merged
mattisonchao merged 1 commit into
mainfrom
mattison/coordinator-cconfig-watch
Apr 15, 2026
Merged

fix: use --cconfig for coordinator cluster config#10
mattisonchao merged 1 commit into
mainfrom
mattison/coordinator-cconfig-watch

Conversation

@mattisonchao

@mattisonchao mattisonchao commented Apr 14, 2026

Copy link
Copy Markdown
Member

Motivation

  • stop wiring cluster config through the coordinator server config file
  • rely on the coordinator configmap watch path directly instead of mounting the cluster config into the pod

Modifications

  • remove cluster.configPath from coordinator.yaml
  • pass --cconfig=configmap:<namespace>/<release>-coordinator to the coordinator
  • keep coordinator using --sconfig for server settings and --profile when pprofEnabled is set
  • remove the extra coordinator cluster-config volume mount from the pod

Testing

  • helm template test-release charts/oxia-cluster

@mattisonchao mattisonchao changed the title Use --cconfig for coordinator cluster config fix: use --cconfig for coordinator cluster config Apr 14, 2026
@mattisonchao mattisonchao self-assigned this Apr 14, 2026
@mattisonchao
mattisonchao merged commit 05f2097 into main Apr 15, 2026
3 checks passed
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.

1 participant