Define additional printer columns in the KafkaAccess CRD

[scholzj]

We should define some additional columns to the KafkaAccess CRD so that it shows a bit more useful information when you do kubectl get kafkaacces or kubectl get kafkaaccess -o wide. The Ready column / state would be the most obvious thing to add there.

We should probably also add it to the strimzi group to have it listed together with the other Strimzi CRs.

[ppatierno]

Triaged on 03.10.2024: we agreed on having the additional columns.

[MichaelMorrisEst]

Id like to volunteer to work on this

[scholzj]

You can definitely look into it. But unfortunately, because this is using the Java Operator SDK, I have no idea where to start. Not sure if @katheris has any idea.

[katheris]

I think Java Operator SDK uses the fabric8 CRD generator under the covers, so you should be able to use an annotation https://github.com/fabric8io/kubernetes-client/blob/main/doc/CRD-generator.md#iofabric8crdgeneratorannotationprintercolumn

[MichaelMorrisEst]

I'd suggest the following additional columns:

NAME              CLUSTER      LISTENER   USER      READY  
my-kafka-access   my-cluster   tls        my-user   False

Does this sound ok?

The cluster, listener and user can easily be added using the annotation suggested above. However the ready column cannot as the annotation does not support filters and hence there is no way to filter the conditions for the Ready condition (i.e. no way to specify conditions[?(@.type==\"Ready\")])

There is however a new annotation in development (see fabric8io/kubernetes-client#6390) that will allow the filter be specified.

We can wait for that to be released, or we can manually update 040-Crd-kafkaaccess.yaml until such time as that is released. If I'm not missing anything, it looks like 040-Crd-kafkaaccess.yaml is not automatically updated with the output from the crd generator as part of the build anyway?

[scholzj]

Thanks for looking into this. I think the suggested columns look good. But we should not modify the CRDs manually (and I think the recent PRs made it updated with the generated ones out of the box). We should either wait for the support to be added - or if you want we can add what we can right now and add the remaining columns in the future when available.

[katheris]

I'm also happy with the columns you proposed. And fine either way if we wait or add what is supported now and then add the final column in future.

[MichaelMorrisEst]

I have created a PR that addresses what we can for now.

Remaining:

• Add Ready column - can be addressed once fix: adds an additional printer column annotation fabric8io/kubernetes-client#6390 becomes available
• Add kafkaacces to Strimzi category (you state strimzi group in the initial description but I assume you mean the category). This is also not currently possible via annotation. However there is an issue for it to be added (CRDGenerator: Allow to configure categories fabric8io/kubernetes-client#5795). I will submit a PR on kubernetes-client to get this implemented as it is currently not prioritised