Provide example on how to create users in tenant chart

[aqeelat]

The docs say that it is a list of secrets. It does not say that the list should be of objects that contain a name.

I had to go through the crd schema to understand why my tenant was not being created.

Incorrect:

tenant:
  users:
    - my-user

Correct:

tenant:
  users:
    - name: my-user

[ramondeklein]

@aqeelat Which docs are you refering too?

[aqeelat]

@ramondeklein https://min.io/docs/minio/kubernetes/upstream/operations/install-deploy-manage/deploy-minio-tenant.html#minio-k8s-deploy-minio-tenant

[ramondeklein]

I can't find any reference on creating users on that page.

[aqeelat]

Sorry I meant https://min.io/docs/minio/kubernetes/upstream/reference/tenant-chart-values.html#minio-tenant-chart-values

Regardless, its not very straightforward to find

[ramondeklein]

I think the reporter of this issue has a valid point. We should provide an example here.

[ravindk89]

https://github.com/minio/operator/blob/master/helm/tenant/values.yaml

We generate that page using the values in the actual values.yaml, so to update this line with more details.

Looking at the chart , that all boils down to the crd where we accept a LocalObjectReference .

We could try to do a pass through the CRD and back that into the values.yaml for those areas where we look for some sort of K8s-standard object. Might take us a bit to schedule this, but we will try to get to it as we have spare cycles.

[ramondeklein]

We can't update that value, because that would create a user by default and we don't want to do that.

[ravindk89]

@ramondeklein we can just add to the commented line above it, I should have been more specific.

We use an autoyaml plugin that just takes whatever is in that comment block and pops it into the RST. No change to the actual value is required.