-
Notifications
You must be signed in to change notification settings - Fork 16
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
feat: creation MSSQL Terraform provider #1085
feat: creation MSSQL Terraform provider #1085
Conversation
Proof of concept to understand how bindings work in MSSQL. The provider developed in the Azure Brokerpak has been used as a coding base with slight modifications for the connection chain. The database is not created when the servier is created. So it is necessary to execute the create database sentence. It is necessary to use an option group called "contained database authentication" to create the contained database. [#185149821](https://www.pivotaltracker.com/story/show/185149821)
It is a copy of the provider used in Azure, with minimal changes for the AWS adaptation. The idea is to keep it practically identical until the final integration. [#185151996](https://www.pivotaltracker.com/story/show/185151996)
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/185836110 The labels on this github issue will be updated when the story is started. |
var _ = BeforeSuite(func() { | ||
adminPassword = testhelpers.RandomPassword() | ||
port = testhelpers.FreePort() | ||
session := testhelpers.StartServer(adminPassword, port) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This StartServer set up creates a database. So strictly speaking we are only testing that we can connect and create bindings when the database is already there. We are missing testing the scenario when the database has not yet been created.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed with @zucchinidev and agreed to provide the tests in a separate pull request to unblock other tasks. The scanario missing is tested with acceptance tests as well and those pass, so we have confidence this works.
Created a PR to replace the modules mentioned in my comments: |
#185151996
Checklist:
make run-integration-tests
andmake run-terraform-tests
?