velero vSphere plugin docs for v1.3.0

[deepakkinni]

What this PR does / why we need it:
Updates the current docs.

Which issue(s) this PR fixes:
Fixes: #418

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

Signed-off-by: Deepak Kinni [email protected]

[svcbot-qecnsdp]

Sorry, some of the pre-check tests are failing for this PR !

Please check the Jenkins job for more details
https://container-dp.svc.eng.vmware.com/job/CNSDP-CI/649/

[lintongj]

A general comment here. I would suggest to add a separate page for the detail of creating the ConfigMap with credential. In the cluster flavor specific landing page, let's add one entry into the list of prerequisites with EMPHASIS.

[deepakkinni]

A general comment here. I would suggest to add a separate page for the detail of creating the ConfigMap with credential. In the cluster flavor specific landing page, let's add one entry into the list of prerequisites with EMPHASIS.

Actually, the ConfigMaps are different for guest and vanilla (no vc credentials secret necessary for guest), so having a common config map page wouldn't be possible.

[lintongj]

A general comment here. I would suggest to add a separate page for the detail of creating the ConfigMap with credential. In the cluster flavor specific landing page, let's add one entry into the list of prerequisites with EMPHASIS.

Actually, the ConfigMaps are different for guest and vanilla (no vc credentials secret necessary for guest), so having a common config map page wouldn't be possible.

Just maintain different sections in the separate page for vanilla and guest cluster flavor respectively. Otherwise, if put too many detail in the per-cluster landing page, users are easy to get lost.

[svcbot-qecnsdp]

Congratulations !!

All the pre-check tests passed
https://container-dp.svc.eng.vmware.com/job/CNSDP-CI/657/

[lintongj]

A general comment here. I would suggest to add a separate page for the detail of creating the ConfigMap with credential. In the cluster flavor specific landing page, let's add one entry into the list of prerequisites with EMPHASIS.

Actually, the ConfigMaps are different for guest and vanilla (no vc credentials secret necessary for guest), so having a common config map page wouldn't be possible.

Just maintain different sections in the separate page for vanilla and guest cluster flavor respectively. Otherwise, if put too many detail in the per-cluster landing page, users are easy to get lost.

@deepakkinni I took one another look at this PR. Ignore my previous comments. Generally, it looks good to organize this newly added section this way.

My only comment here is, would it be better to also add a corresponding section to supervisor.md and call out that it is not required to create such a ConfigMap for plugin in supervisor cluster?

[xing-yang]

LGTM

[svcbot-qecnsdp]

Congratulations !!

All the pre-check tests passed
https://container-dp.svc.eng.vmware.com/job/CNSDP-CI/658/