kedacore / keda-olm-operator

Operator for deploying KEDA Controller on OperatorHub.io/OLM
Apache License 2.0
31 stars 23 forks source link

logTimeFormat is not present in KEDA v2, encoder is #45

Closed zroubalik closed 3 years ago

zroubalik commented 3 years ago

logTimeFormat was removed but zap-encoder was added: https://github.com/kedacore/keda/blob/main/BUILD.md#keda-operator-logging

We should modify the KedaController resource, the property for zap-encoder should be probably named logEncoder https://github.com/kedacore/keda-olm-operator/blob/e42474f1fe8ee3e35e2ee175878def776e897bac/api/v1alpha1/kedacontroller_types.go#L38

Don't forget to update Readme and samples in config/samples https://github.com/kedacore/keda-olm-operator#kedacontroller-spec