Clarify expected config file name and target for Conan

[AbrilRBS]

Hi!

A PR to add the library to Conan Center Index was submitted based on your current one, and in the review process one question has arisen as to how to best move forward with that PR:

Your generated config files look like kubernetesConfig.cmake and the generated target is kubernetes::kubernetes, and it's like this how it's used outside of Conan

Meanwhile in your Conan documentation, you mention that users are expected to use the default Conan-generated file name of kubernetes_client_c, and target of kubernetes_client_c::kubernetes_client_c. This is because Conan was not told to generate different names in the recipe, but it could have been just as easily match what you already generate when compiling from sources.

In the process of moving to the central CCI repository, we wanted to clarify if we should make it so Conan also generates the kubernetes names and follow the upstream convention, which is something we usually do without much hesitation, but that this time, as you already have Conan docs, we thought we should ask first :)

[brendandburns]

cc @ityuhui

I'm not super familiar with Conan, I'm not sure that there was much thought put to this in the original PR, but @ityuhui may have a better perspective since he reviewed that change.

[ityuhui]

Hi @brendandburns @AbrilRBS

Thanks for starting this discussion !

I think kubernetes::kubernetes is better. That way we can follow Conan conventions. And the CMake and vcpkg https://github.com/kubernetes-client/c/blob/master/docs/vcpkg.md for this client library also use kubernetes as the target name.

Perhaps the original PR #230 simply wanted to use kubernetes_client_c::kubernetes_client_c to indicate that this was a C client for Kubernetes.

I think @davidsanfal could modify recipes/kubernetes_client_c/all/conanfile.py to replace kubernetes_client_c with kubernetes.

Eventually, I can merge that file back into https://github.com/kubernetes-client/c/blob/master/kubernetes/conanfile.py.

cc @ydcpp , the author of #230

[davidsanfal]

Hi! thanks for the clarification @ityuhui

I've already updated it to kubernetes::kubernetes, It was very easy to migrate the recipe to work on conan center index, it was very well written.

[ityuhui]

Thanks @davidsanfal ! I have seen your change.

After conan-io/conan-center-index#28042 is merged, I'll merge recipes/kubernetes/all/conanfile.py back here.