netscaler / netscaler-helm-charts

NetScaler helm charts
https://github.com/netscaler/netscaler-helm-charts
Apache License 2.0
21 stars 31 forks source link

Getting error - line feed on citrix-k8s-ingress.yaml:12 #193

Closed MalekGD86 closed 1 week ago

MalekGD86 commented 1 week ago

in the commit 52c95e3, a line feed has been inserted causing charts not to work anymore on install / upgrade. this error has been raised :

Error: UPGRADE FAILED: parse error at (citrix-cloud-native/charts/cic/templates/citrix-k8s-ingress.yaml:12): unterminated quoted string

subashd commented 1 week ago

Hi @MalekGD86 , Could you please share output of helm template cic netscaler/citrix-ingress-controller?

MalekGD86 commented 1 week ago

`---

Source: citrix-ingress-controller/templates/rbac.yaml

apiVersion: v1 kind: ServiceAccount metadata: name: cic-citrix-ingress-controller namespace: default

Source: citrix-ingress-controller/templates/configmap.yaml

apiVersion: v1 kind: ConfigMap metadata: name: cic-cic-configmap namespace: default data: LOGLEVEL: "info" JSONLOG: "false" NS_PROTOCOL: "https" NS_PORT: "443"

Source: citrix-ingress-controller/templates/rbac.yaml

kind: ClusterRole apiVersion: rbac.authorization.k8s.io/v1 metadata: name: cic-citrix-ingress-controller rules:

subashd commented 1 week ago

Thank you @MalekGD86 The above output looks fine. Could you please share the command you tried to upgrade or install? Also which version of CIC are you using and upgrading to which version?

subashd commented 1 week ago

Also if you can fill the Requirement Gathering Questionnaire.

MalekGD86 commented 1 week ago

I've installed this repo:

It installed the last version of the helm chart, i guess. The yaml file didn't change since i've installed it sucessfully

the output of the command is the error raised in the main comment.

subashd commented 1 week ago

Could you please share the cic_ipam.yaml as well?

MalekGD86 commented 1 week ago

Ip's are masked on purpose :) adcCredentialSecret and entityPrefix has been changed too.

ao: adc: IP: 1.2.3.4 loginSecret: nslogin mgmtHTTPPort: 80 mgmtHTTPSPort: 443 cic: enabled: true nsPort: 443 nsProtocol: HTTPS ipam: true nsIP: 1.2.3.4 adcCredentialSecret: test entityPrefix: test image: 'quay.io/citrix/citrix-k8s-ingress-controller:1.40.12' pullPolicy: IfNotPresent license: accept: 'Yes' ingressClass:

subashd commented 1 week ago

Thank you @MalekGD86 I have fixed the issue. Could you please try once?

MalekGD86 commented 1 week ago

work perfectly ! thank you for your quick fix!

subashd commented 1 week ago

Thank you @MalekGD86 Also, you can provide more details on your requirement over an email