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 support & test cases for MSSQL Destination via SSH tunnels #5970

Closed
Tracked by #5901
alexandr-shegeda opened this issue Sep 10, 2021 · 0 comments · Fixed by #6503
Closed
Tracked by #5901

Add support & test cases for MSSQL Destination via SSH tunnels #5970

alexandr-shegeda opened this issue Sep 10, 2021 · 0 comments · Fixed by #6503

Comments

@alexandr-shegeda
Copy link
Contributor

Tell us about the problem you're trying to solve

We'd like to add support for SSH tunnels for the existing MSSQL destination. This will allow connecting to the destination from the different VPC.

Describe the solution you’d like

There are few main steps to accomplish this requirement:

  1. Create a DB in AWS through terraform or the UI. We already have a blueprint for how to create a database on AWS in terraform. You would need only to create a database without a public IP in the subnet which already exists, the same one the Postgres DB is currently linked to. Please talk to Oleksandra for getting access to the AWS dev environment where you can spin up a database. If you go this route, you can re-use many of the parameters (tunnel host, tunnel key, etc..) in the lastpass secrets Postgres destination ssh key creds and destination postgres test creds with sshkey. If Terraform is slowing you down too much feel free to create one in the UI.
  2. Wrap existing MSSQL destination with SshWrappeddestination and check that additional SSH configuration properties are present on UI
  3. Cover this change with an appropriate number of unit/integration/acceptance tests

Describe the alternative you’ve considered or used

Depending on whether you are adding a source or a destination the impl is slightly different. See the instructions for how to add SSH support to a connector

Example of adding SSH support to Postgres destination

Note that this PR has way more logic in them than what you'll need to implement for a single connector because they add common helpers and shared libraries and docs

@alexandr-shegeda alexandr-shegeda added area/connectors Connector related issues lang/java priority/high High priority type/enhancement New feature or request labels Sep 10, 2021
@andriikorotkov andriikorotkov removed their assignment Sep 20, 2021
sashaNeshcheret added a commit that referenced this issue Sep 29, 2021
#6503)

* 🎉 Destination MSSQL: Added support for connection via SSH tunnels (#5970)

* 🎉 Destination MSSQL: bumb image version

*  🎉 Destination MSSQL: update mssql.md file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants