search

canonical / kafka-k8s-operator

Kafka K8s operator
Apache License 2.0
7 stars 11 forks source link

[DPE-5544] - fix: set dir permissions & remove lost+found on storageattached #144

Closed marcoppenheimer closed 4 days ago

marcoppenheimer commented 1 month ago

Changes Made

fix: set dir permissions & remove lost+found on storageattached

deusebio commented 4 weeks ago

I have tested the fix on Azure AKS and the deployment worked fine. Attached the list of revisions tested:

App            Version  Status   Scale  Charm            Channel      Rev  Address       Exposed  Message
admin                   unknown      0  data-integrator  latest/edge   72  10.0.96.101   no
consumer                unknown      0  kafka-test-app   latest/edge   13  10.0.15.211   no           
kafka-k8s      3.6.1    waiting    3/0  kafka-k8s                       0  10.0.69.71    no       
producer                unknown      0  kafka-test-app   latest/edge   13  10.0.139.134  no
zookeeper-k8s  3.9.2    active     1/0  zookeeper-k8s    3/edge        67  10.0.124.149  no

(the controller was scaling down, hence the 0 scale units, but the process of producing and consuming worked fine, believe me!)

joalif commented 1 week ago

Hi @marcoppenheimer is there an ETA for this to be merged and released? I believe this fixes a bug a customer hits. Issue: When deploying kafka-k8s on microk8s with ceph backend the kafka unit and application get into blocked state. This does not happen with hostpath backend. After some digging I saw that when using ceph backend '/var/lib/kafka/data' is owned by root user instead of kafka and kafka cannot write to it. Thus I think this PR fixes it.