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

feat: creation MSSQL Terraform provider #1085

Merged
merged 13 commits into from
Aug 16, 2023

Conversation

zucchinidev
Copy link
Contributor

@zucchinidev zucchinidev commented Aug 15, 2023

#185151996

Checklist:

  • Have you added Release Notes in the docs repositories?
  • Have you ran make run-integration-tests and make run-terraform-tests?
  • Have you ran acceptance tests for the service under change?
  • Have you followed the Conventional Commits specification?

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)
@cf-gitbot
Copy link

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.

@zucchinidev zucchinidev changed the title Binding tf provider mssql 185151996 feat: creation MSSQL Terraform provider Aug 15, 2023
@zucchinidev zucchinidev marked this pull request as ready for review August 15, 2023 16:54
var _ = BeforeSuite(func() {
adminPassword = testhelpers.RandomPassword()
port = testhelpers.FreePort()
session := testhelpers.StartServer(adminPassword, port)

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.

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.

@pivotal-marcela-campo pivotal-marcela-campo merged commit e9368d4 into main Aug 16, 2023
@zucchinidev zucchinidev deleted the binding_tf_provider_mssql_185151996 branch August 16, 2023 13:50
@fnaranjo-vmw
Copy link
Contributor

Created a PR to replace the modules mentioned in my comments:
#1106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

4 participants