Skip to content

Apply kubernetes service parameters during reconciliation#548

Merged
enzocandotti merged 1 commit into
Wind-River:masterfrom
wasnio:fix/apply-param
Jul 14, 2026
Merged

Apply kubernetes service parameters during reconciliation#548
enzocandotti merged 1 commit into
Wind-River:masterfrom
wasnio:fix/apply-param

Conversation

@wasnio

@wasnio wasnio commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

DM creates kubernetes/kube_apiserver service parameters in the sysinv database but never issues the service-parameter-apply API call. Unlike most service parameters that are consumed by puppet classes during host unlock, kube_apiserver parameters are only applied by the change_k8s_control_plane_params.py script, which is exclusively triggered via the service-parameter-apply API.

This results in the parameters being stored in the database while kube-apiserver continues running with default settings.

Issue the service-parameter-apply for the kubernetes service when kube_apiserver parameters are present in the deployment configuration. Update the gophercloud dependency to fix the Apply function which incorrectly expected HTTP 204 and attempted to decode an empty response body.

Test Plan:

PASS: Deploy subcloud with kubernetes/kube_apiserver service
      parameters (tls-min-version, tls-cipher-suites). Verify
      kube-apiserver static pod manifest reflects the configured
      values after deployment completes.
PASS: Deploy system without kubernetes service parameters. Verify
      no apply is issued and reconciliation completes normally.
PASS: Deploy dc system controller with kubernetes service parameters
      applied beforehand during bootstrap. Verify no apply is issued
      and reconciliation completes normally.

DM creates kubernetes/kube_apiserver service parameters in the sysinv
database but never issues the service-parameter-apply API call. Unlike
most service parameters that are consumed by puppet classes during
host unlock, kube_apiserver parameters are only applied by the
change_k8s_control_plane_params.py script, which is exclusively
triggered via the service-parameter-apply API.

This results in the parameters being stored in the database while
kube-apiserver continues running with default settings.

Issue the service-parameter-apply for the kubernetes service when
kube_apiserver parameters are present in the deployment configuration.
Update the gophercloud dependency to fix the Apply function which
incorrectly expected HTTP 204 and attempted to decode an empty
response body.

Test Plan:
PASS: Deploy subcloud with kubernetes/kube_apiserver service
      parameters (tls-min-version, tls-cipher-suites). Verify
      kube-apiserver static pod manifest reflects the configured
      values after deployment completes.
PASS: Deploy system without kubernetes service parameters. Verify
      no apply is issued and reconciliation completes normally.
PASS: Deploy dc system controller with kubernetes service parameters
      applied beforehand during bootstrap. Verify no apply is issued
      and reconciliation completes normally.

Signed-off-by: Wallysson <wallysson.silva@windriver.com>
@wasnio
wasnio requested a review from enzocandotti July 14, 2026 12:49
@enzocandotti
enzocandotti merged commit 12330e6 into Wind-River:master Jul 14, 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.

2 participants