Skip to content

charts/apisix: Remove .Values.apisix.enabled check from service-control - #936

Closed
mdnfiras wants to merge 1 commit into
apache:masterfrom
mdnfiras:master
Closed

mdnfiras wants to merge 1 commit into
apache:masterfrom
mdnfiras:master

Conversation

@mdnfiras

Copy link
Copy Markdown

The field .Values.apisix.enabled does not exist in helm values and causes the control API's service to not be created even when .Values.control.enabled is set.

…ol.yaml

Signed-off-by: f.medini <f.medini@nyris.io>
@Baoyuantop Baoyuantop added the enhancement New feature or request label Apr 9, 2026
@AlinsRan AlinsRan closed this Sep 30, 2026
@AlinsRan AlinsRan reopened this Sep 30, 2026
@AlinsRan

Copy link
Copy Markdown
Contributor

Closing as covered by #923, which has been merged and replaces the non-existent .Values.apisix.enabled gate in service-control.yaml with .Values.control.enabled. Thanks for the contribution!

@AlinsRan AlinsRan closed this Sep 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants