[java] Add sundrio to generate builders

[andreaTP]

Add sundrio to generate builder pattern for the model classes of the generated CRD.

With the upcoming version of kubernetes-client-bom-with-deps we should be able to remove the explicit versions of sundrio and lombok too.

[squakez]

We need to decline this one. We already discussed with the community the opportunity to use code generators such as Lombok. With a negative answer.

[claudio4j]

@andreaTP what would be the use case to manipulate Camel K k8s objects ?

[andreaTP]

@squakez in this case both Lombok and Sundrio are used only at compile time and are not transitive dependencies for the user.

@claudio4j :
@lburgazzoli asked to have builder pattern on the models and, as far as it works it's a much nicer user experience.

[squakez]

Okey. Maybe, given that this is an auto generated project where user is not meant to manually compile it, we can accept it. @oscerd wdyt?

[oscerd]

I hope we can remove lombok even if it's only a runtime dependency.

[squakez]

@andreaTP please, remind to remove the dependencies in the future when no longer necessary. Thanks!

[andreaTP]

@oscerd and @squakez I'm not sure about those last comments, both lombok and sundrio are marked as "provided".

This means that they will be available on the classpath only during compilation and they are not going to be considered transitive dependencies of the resulting artifact.

remove lombok even if it's only a runtime dependency.

It's not a runtime dependency and we can't remove it to get the generated builder pattern.

remind to remove the dependencies in the future when no longer necessary

Just to be clear, the only thing I will change over the current configuration is the removal of the explicit sundrio and lombok version and relevant "property".

[oscerd]

Our idea is to avoid Lombok as much as we can. Since this is a separated context for generating stuff, that's fine, but we would like to avoid the spreading of Lombok around the codebase. It's provided and that's fine.

[andreaTP]

agreed