chrisingenhaag / helm Goto Github PK

Hello,

Instead of putting the username and password in the helm values file, we can use a reference to an existing secret.

I have opened a PR that covers this new feature.

#18

Please let me know what you think :)

Best Regards,
Alexander

HI @chrisingenhaag

I'm trying to use your chart to integrate CODE on my project.
I'm experimenting a little problem when I use Google Chrome

it seems to be not able to do synchronous call inside Cool.html

Uncaught DOMException: Failed to execute 'send' on 'XMLHttpRequest': Failed to load 'https://app.test.synbee.it/browser/eb73aa3/l10n/uno-localizations.json'.
at request_JSON (https://app.test.synbee.it/browser/eb73aa3/bundle.js:1:766277)
at String_ctr. (https://app.test.synbee.it/browser/eb73aa3/bundle.js:1:766621)
at https://app.test.synbee.it/browser/eb73aa3/bundle.js:1:770033
at https://app.test.synbee.it/browser/eb73aa3/bundle.js:1:770592

, request_JSON = function(uri) {
    if (browserless)
        return loadFromDisk(uri);
    var req = new XHR
      , data = {};
    req.open("GET", uri, FALSE);    <<-- HERE (if I put TRUE = a horrible local variable with value !0 ... it start correctly
    req.send(null);     
    try {
        data = JSON.parse(req.responseText)
    } catch (e) {
        setTimeout(function() {
            var l10n_err = new Error("Unable to load localization data: " + uri);
            l10n_err.name = "Localization Error";
            throw l10

any indication about this behavior?

If I use Mozilla Firefox ..all goes in the correct way

regards

For people running multiple ingress controllers, there is the possibility to specify the desired ingress controller via the ingressClassName in the ingress spec. You can find an example here. The old way of using annotations for this purpose no longer works on many recent deployments.

Can the ingressClassName field be added to the ingress fields in the values.yaml and rendered in the helm template?

Hello from CODE version 21.11.3.6 the variable domain has been changed in aliasgroupX and throw an error if is not defined.
see CODE changes

For the moment I have clone this charts and I have replace the env variable domain by aliasgroup1 to get a functional deployment, but I haven't real idea how manage this properly in the chart.

• with 2.6.0 I missed updating readme and schema file.
• additionally add artifacthub metadata file

Hello,

thanks a lot for the chart :)

not sure why but my configuration only works with
DONT_GEN_SSL_CERT: set
that is not allowed by helm because DONT_GEN_SSL_CERT is defined as a boolean.

I tried with DONT_GEN_SSL_CERT: false but it don't work either.

Is it possible to change DONT_GEN_SSL_CERT to accept any string ? Not sure what are the possible drawback.

If possible / enabled, it would be nice to install a ServiceMonitor for PrometheusOperator.

/cool/getMetrics` is the url for it with BasicAuth of your Variable.

apiVersion: monitoring.coreos.com/v1                                                                                                                                                                              kind: ServiceMonitor                                                                                                                                                                                              metadata:
  spec:
    endpoints:
      - port: "http"
        path: "/cool/getMetrics"
        basicAuth:
          username:
            name: "{{ include "collabora-code.fullname" . }}"
            key: username
          password:
            name: "{{ include "collabora-code.fullname" . }}"
            key: password
    selector:
      matchLabels:
        app.kubernetes.io/name: {{ include "collabora-code.name" . }}
        app.kubernetes.io/instance: {{ .Release.Name }}   

After the Helm-Chart reconfigure the ConfigMap, the Pod is not restartet.

A Solution could be, that the ConfigMap-Name becomes a hash.

kubernetes/kubernetes#22368 (comment)

Here a snipped:

name: configmap-{{ .Values.collabora | toYaml | sha256sum | trunc 8 }}