CENGN/netbox-kubernetes

no reference for REDIS_CACHE_HOST

Closed this issue · 7 comments

lethargosapatheia commented

Hello,

You're not referencing REDIS_CACHE_HOST in the configmap, so when you deploy it netbox, it doesn't work.

mohahmed13 commented

Hi @lethargosapatheia , I think it is mentioned here: https://github.com/CENGN/netbox-kubernetes/blob/master/netbox-deployment.yaml#L108

Let me know if you think that's missing. I will just note that there's a better way to do this by referencing all values from a ConfigMap using the envFrom spec: https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/pod-v1/#environment-variables. We have future plans to upgrade these manifests and share it with the community.

  • 👍1
lethargosapatheia commented

Thanks for replying. I must have missed it, I'm not sure why.

Regarding the envFrom spec - doesn't netbox expect the variables to be read from /run/secrets? I'm not sure how you can mount the volumes which you reference in the envFrom.

mohahmed13 commented

Ah yeah, I meant that with the envFrom spec, all values in the ConfigMap would be mounted as environment variables automatically. That would eliminate the need to do this for every environment variable:

        - name: ALLOWED_HOSTS
          valueFrom:
            configMapKeyRef:
              key: ALLOWED_HOSTS
              name: netbox-configmap

For the /run/secrets, like you said, you would still need to mount them as volumes.

  • 👍1
ac185391 commented

It still doesn't work, REDIS_CACHE_HOST is missing in configmap and referenced in deployment which is causing this

mohahmed13 commented

Upon a second look, you're right. The variable name needs to be changed here: https://github.com/CENGN/netbox-kubernetes/blob/master/netbox-configmap.yaml#L24 - it should actually be REDIS_CACHE_HOST

We have some updated manifests compatible with v2.11.12. Should be rolled out in a couple of days!

  • 👍1
NeckBeardPrince commented

Upon a second look, you're right. The variable name needs to be changed here: https://github.com/CENGN/netbox-kubernetes/blob/master/netbox-configmap.yaml#L24 - it should actually be REDIS_CACHE_HOST

We have some updated manifests compatible with v2.11.12. Should be rolled out in a couple of days!

Any updates on this being rolled out?

  • 👍1
Eric-CENGN commented

Sorry for the wait, update was rolled out, and REDIS_CACHE_HOST env variable was added