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

Docs change for Dev VM, ldap config #3310

Merged
merged 1 commit into from May 13, 2022
Merged

Docs change for Dev VM, ldap config #3310

merged 1 commit into from May 13, 2022

Conversation

lbakerchef
Copy link
Contributor

@lbakerchef lbakerchef commented May 12, 2022

You must currently make the following change in defaults.yml, not config.yml, to enable ldap (it is unknown why the change is not being picked up in config.yml):

   ldap:
-    start: false
+    start: true

Note to docs team: Inward facing dev docs.

@lbakerchef lbakerchef requested review from a team as code owners May 12, 2022 20:59
@netlify
Copy link

netlify bot commented May 12, 2022

👷 Deploy Preview for chef-server processing.

Name Link
🔨 Latest commit fca2c55
🔍 Latest deploy log https://app.netlify.com/sites/chef-server/deploys/627d79e138652200083a35d0

@lbakerchef lbakerchef self-assigned this May 12, 2022
@lbakerchef lbakerchef changed the title Fix ldap for dev vm. Docs change for dev vm, ldap config May 12, 2022
@lbakerchef lbakerchef changed the title Docs change for dev vm, ldap config Docs change for Dev VM, ldap config May 12, 2022
You must currently make the following change in
dev/defaults.yml to enable ldap (it is unknown why
the change is not being picked up in config.yml):

   ldap:
-    start: false
+    start: true

Signed-off-by: Lincoln Baker <lbaker@chef.io>
@sonarcloud
Copy link

sonarcloud bot commented May 12, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jashaik jashaik merged commit dcd0c53 into main May 13, 2022
@jashaik jashaik deleted the lbaker/fix-dev-vm-ldap branch May 13, 2022 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants