kedacore / keda-olm-operator

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

Remove extra kubectl invocations #176

Closed joelsmith closed 1 year ago

joelsmith commented 1 year ago

Because make will run ifeq shell commands on every invocation, every target causes kubectl to be invoked to list namespaces, even targets which never would have reached the targets in question. Instead, update such targets to use kubectl apply instead of kubectl create so that creation will succeed even if the namespace already exists.

Checklist

joelsmith commented 1 year ago

@gauron99 @zroubalik please take a look, thanks!

gauron99 commented 1 year ago

nice change! I didnt think about that