[API] Clean up the cluster-service API

[foot]

Motivation: customers want to use our APIs! Lets tidy them up before giving users access

• Make sure all methods have docstrings
• Make sure all ambiguous param names have examples
• Tags are a mix of TitleCase and lowercase
• For most resources we want to have
  • "single pane of glass" endpoint, e.g. GET /api/v1/helm-releases, that returns all helm-releases on all cluster on all namespace that the user can see.
  • Then for accessing a specific resource you have to use the very long path: /v1/namespaces/{clusterNamespace}/clusters/{clusternname}/namespace/{resNamespace}/helm-releases/{resName}
• Some endpoints have legacy naming:
  • POST /api/clusters to create a PR from a template, this should be POST /v1/templates/pull-request
• Make sure all message fields are snake_case (@simon double check this std. w/ google etc.)
• Check docstrings all make sense, and omit the name of message (function) in the proto file.
• kebab-case the resource names in the api endpoints, e.g. /helm-releases/ not /helm_releases/ (this is an OSS resource)

Tasks here:

• review *.proto files in OSS
• review *.proto files in EE
• Update, re-gen

[foot]

Lets follow https://protobuf.dev/programming-guides/style/

[foot]

Review all the places the UI is calling the static Service object and make sure we are typing it properly and not using any.

[foot]

Review all the places the UI is calling the static Service object and make sure we are typing it properly and not using any.

Templates, which was the main concern, seems to be all typed now which is great!

Explorer is also typed, think we are all good with this one.

[foot]

Oops, chatting w/ @sarataha , we realised clusters are namespaced... so this URL doesn't have enough info in it:

• /api/v1/clusters/{clusterName}/namespaces/{namespace}/helm_releases/{name}
• clusterName

Options:

1. /api/v1/namespaces/{clusterNamespace}/clusters/{clusterName}/namespaces/{namespace}/helm_releases/{name}
2. /api/v1/clusters/{clusterNamespace}/{clusterName}/namespaces/{namespace}/helm_releases/{name}
3. /api/v1/clusters/{clusterNameThatIncludeNamespace}/namespaces/{namespace}/helm_releases/{name}
4. /api/v1/helmreleases # all helmrelease on all the clusters, can be filtered
5. /api/v1/helmreleases/clusters/{clusterName}/namespaces/{namespace}/{helmReleaseName}
6. /api/v1/helmreleases/clusters/{clusterName}
7. /api/v1/helmreleases/namespace/{namespace}

clusterNameThatIncludeNamespace is usually default%3dmy-cluster

Flux: https://127.0.0.1:6443/apis/helm.toolkit.fluxcd.io/v2beta1/namespaces/default/helmreleases?limit=500