Pinniped info review about tanzu-validated-solutions

[sendjainabhi]

I noticed we mention LDAP in the title of the Pinniped section, but there is no other mention in the doc. Everything is OIDC related only.

Also here's a more detailed diagram that shows the differences that exist today on how the LDAP and OIDC integrations happen. Also more detail on the client component to reflect the Tanzu CLI involvement https://onevmw.sharepoint.com/:i:/s/TKG-TSL/EZph_9wBnMRNnRXzh0E7e6EBAjR68AIjUMQsXEGg3iws9w?e=1f6h55 This is compliant with both TKGm 1.3.1 and 1.4.0 versions

i have removed detail pinniped integration steps document in latest pull request after field review comments . please check this https://github.com/vmware-tanzu-labs/tanzu-core-reference-architecture/pull/23 . is this addressing the issue ?

[jaimegag]

@sendjainabhi
I see now the team has decided to simplify this page and now OIDC and LDAP options are equally covered.
I think that works, since additional detail probably doesn't belong in a RA.
However I would change the link at the bottom of the page where it says:
Refer to Pinniped Docs for more information on how to integrate Pinniped into Tanzu Kubernetes Grid with OIDC providers and LDAP.
It should not point to the Pinniped docs, IMO it should point to the TKG docs:
1.3 -> https://docs.vmware.com/en/VMware-Tanzu-Kubernetes-Grid/1.3/vmware-tanzu-kubernetes-grid-13/GUID-mgmt-clusters-enabling-id-mgmt.html
1.4 -> https://docs.vmware.com/en/VMware-Tanzu-Kubernetes-Grid/1.4/vmware-tanzu-kubernetes-grid-14/GUID-mgmt-clusters-enabling-id-mgmt.html

[msegvich]

@srivar This looks to be a duplicate of #28. If that's the case this can be closed out.

[srivar]

This is being delivered in TKO RA v1.5