Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

do not enable "disable SSL cert verif" #276

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ymeillier
Copy link
Contributor

if a cert is specified for NSX Manager CA, then one cannot click the box "disable SS Certificate verification"
AD per the guide from pivotal pp184 "NSX Manager CA Cert" amd "Disable SSL certificate validation" are mutually exclusive [...] ... if you populate the certificate field and disable certificate validation the pks installation will fail

if a cert is specified for NSX Manager CA, then one cannot click the box "disable SS Certificate verification"
AD per the guide from pivotal pp184 "NSX Manager CA Cert" amd "Disable SSL certificate validation" are mutually exclusive [...] ... if you populate the certificate field and disable certificate validation the pks installation will fail
@afewellvmware afewellvmware added bug/minor You found a simple problem and know the resolution, but materials need to be updated do-not-merge/hold enhancement/newFeature New feature or request state/planning and removed bug/minor You found a simple problem and know the resolution, but materials need to be updated labels Feb 20, 2019
@afewellvmware
Copy link
Collaborator

I want to investigate this a bit more, I see your point about the docs and would like to follow the docs, but it has me curious as this seems to work fine without this change, which seems to contradict the docs. In any case this does appear to work fine without the update, so I am going to keep this pr open while waiting further research and feedback

@natereid72
Copy link
Contributor

should this PR from 2019 be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/minor You found a simple problem and know the resolution, but materials need to be updated do-not-merge/hold enhancement/newFeature New feature or request state/planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants