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

Add external openldap terraform scenario #1812

Merged
merged 2 commits into from
Nov 12, 2019
Merged

Conversation

snapp
Copy link
Contributor

@snapp snapp commented Nov 4, 2019

This PR adds an omnibus external openldap terraform scenario as requested in #1797

Description

This scenario instantiates an OpenLDAP Server followed by a Chef Infra Server utilizing an Omnibus built artifact downloaded from $upgrade_version_url as the install package.

The Chef Infra Server will receive a /etc/opscode/chef-server.rb configuration file that is setup to use the external LDAP server over TLS.

Once both servers are installed and configured, the pedant tests are run against the Chef Infra Server.

Note that smoke and pedant tests currently fail in this scenario.

Signed-off-by: Christopher A. Snapp [email protected]

Check List

@snapp snapp requested review from PrajaktaPurohit and a team November 4, 2019 22:36
@snapp snapp requested a review from a team as a code owner November 4, 2019 22:36
@snapp snapp force-pushed the csnapp/1797 branch 2 times, most recently from 2b615a1 to 42f04a4 Compare November 5, 2019 13:46
Copy link
Contributor

@markan markan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me. (Sorry for the delay in reviewing, had to refresh my memory on terraform syntax.)

Signed-off-by: Christopher A. Snapp <[email protected]>
Signed-off-by: Christopher A. Snapp <[email protected]>
@PrajaktaPurohit PrajaktaPurohit merged commit ed14ad9 into master Nov 12, 2019
@chef-expeditor chef-expeditor bot deleted the csnapp/1797 branch November 12, 2019 15:49
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.

4 participants