Anubhav Jindal has posted comments on this change. ( http://gerrit.cloudera.org:8080/24342 )
Change subject: IMPALA-14976: Add optional Kudu Helm resources ...................................................................... Patch Set 13: (5 comments) Done http://gerrit.cloudera.org:8080/#/c/24342/10//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/24342/10//COMMIT_MSG@36 PS10, Line 36: Implemented and validated manually; Cursor-assisted. > For now, please add "Generated-by: Cursor (model name)" to this commit mess Done http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-master-deployment.yaml File helm/impala/templates/kudu-master-deployment.yaml: http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-master-deployment.yaml@30 PS10, Line 30: replicas: {{ .Values.kudu.master.replicas }} > Even if replicas is intentionally set to 0 here, helm will fallback to the Done http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-tserver-deployment.yaml File helm/impala/templates/kudu-tserver-deployment.yaml: http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-tserver-deployment.yaml@30 PS10, Line 30: replicas: {{ .Values.kudu.tserver.replicas }} > Even if replicas is intentionally set to 0 here, helm will fallback to the Done http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-validation.yaml File helm/impala/templates/kudu-validation.yaml: http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-validation.yaml@21 PS10, Line 21: {{- if ne (int .Values.kudu.master.replicas) 1 }} > I believe we are trying to enforce a replica count of 1 here, but an edge c Done http://gerrit.cloudera.org:8080/#/c/24342/10/helm/impala/templates/kudu-validation.yaml@24 PS10, Line 24: {{- if ne (int .Values.kudu.tserver.replicas) 1 }} > I believe we are trying to enforce a replica count of 1 here, but an edge c Done -- To view, visit http://gerrit.cloudera.org:8080/24342 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I50671c46bbc30beb9d643880910adbd3f77407b6 Gerrit-Change-Number: 24342 Gerrit-PatchSet: 13 Gerrit-Owner: Anubhav Jindal <[email protected]> Gerrit-Reviewer: Anubhav Jindal <[email protected]> Gerrit-Reviewer: Gokul Kolady <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Comment-Date: Wed, 03 Jun 2026 22:19:23 +0000 Gerrit-HasComments: Yes
