Description of problem: Client is using LDAP as identity provider without TLS. For insecure connection ca certificate declaration should be ignored but it is taking into effect. https://docs.openshift.com/container-platform/3.10/install_config/syncing_groups_with_ldap.html#ldap-client-configuration Here is the details: master-config.yaml ------------------- oauthConfig: assetPublicURL: https://master.example.comn:8443/console/ grantConfig: method: auto identityProviders: - challenge: true login: true mappingMethod: claim name: ldap_provider_name provider: apiVersion: v1 attributes: email: - mail id: - dn name: - cn preferredUsername: - uid bindDN: '' bindPassword: "********" ca: /etc/origin/master/linux_provider_ca.crt <===== insecure: true <===== kind: LDAPPasswordIdentityProvider url: ldap://192.168.100.1/ou=People,o=unixteam?uid With this configuration the master-api did not start. After changing the value ca: ' ' i.e. empty it master-api pod started working. How reproducible: In master-config.yaml file , if following ca value is used , the master-api pod does not start. ~~ ca: /etc/origin/master/linux_provider_ca.crt ~~ Actual results: if insecure=true is used then ca value should be ignored as per documentation Expected results: ca='xxxx' whatever , master-api pod should be started Additional info:
The error is coming from loading the config and not being able to load the ca from that file (likely because it doesn't exist). The documentation is slightly off from the actual behavior: no matter what is set for `insecure`, if the url scheme is `ldaps` it will attempt to make a TLS connection using the given `ca`. This seems like reasonable behavior, so we should update the docs.
PR's here: https://github.com/openshift/openshift-docs/pull/12631 Jianlin, will you PTAL? Erica, I'd like your opinion on the new wording, too.
I commented on the PR.
Commits pushed to master at https://github.com/openshift/openshift-docs https://github.com/openshift/openshift-docs/commit/33a81aeb08b341cd6b70ceafff6fab7593cda19e bug 1639080 clarifying insecure connection override https://github.com/openshift/openshift-docs/commit/30c3e2a4bb1c12eb783ee552da037f7570cce996 Merge pull request #12631 from kalexand-rh/BZ1639080 bug 1639080 clarifying insecure connection override
Erica and Chuan Yu agreed to merge the PR. I'm waiting for the change to go live.
This change is live on docs.openshift, eg: https://docs.openshift.com/container-platform/3.11/install_config/syncing_groups_with_ldap.html#ldap-client-configuration and on the portal, eg: https://access.redhat.com/documentation/en-us/openshift_container_platform/3.10/html-single/configuring_clusters/index#ldap-client-configuration