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

Integration Testing for SSH using a docker container #5811

Closed
Tracked by #5901
cgardens opened this issue Sep 2, 2021 · 3 comments · Fixed by #6312
Closed
Tracked by #5901

Integration Testing for SSH using a docker container #5811

cgardens opened this issue Sep 2, 2021 · 3 comments · Fixed by #6312
Assignees
Labels
area/connectors Connector related issues priority/high High priority type/enhancement New feature or request

Comments

@cgardens
Copy link
Contributor

cgardens commented Sep 2, 2021

Tell us about the problem you're trying to solve

Improve the process of acceptance testing by allowing doing acceptance / integration testing using a bastion running in a docker container. Right now we have a dedicated host in AWS that we use.

Why do we care?

  1. We'd rather not maintain the bastion infra if we don't have to
  2. It means that the tests require secrets as an input which adds a bit more friction. If we use docker that would not be necessary.
  3. It requires having a static instance of the database we trying to test running! Because the bastion is static it also needs to be forwarding to some database in our VPC so we need a statically running db as well. This limits the tests we can right and means we're paying for and maintaining a database for no good reason. Ideally we could spin up dbs and bastion all in docker!

@sherifnada fyi

@cgardens cgardens added type/enhancement New feature or request area/connectors Connector related issues priority/high High priority labels Sep 2, 2021
@cgardens
Copy link
Contributor Author

cgardens commented Sep 2, 2021

I'm putting this as a high priority, as I really think we should fix this before we trying to develop SSH connectors for any other databases. Happy to adjust if others think this is not a pre-req.

@cgardens
Copy link
Contributor Author

Adding a diagram of the problem to make it a little clearer. https://docs.google.com/drawings/d/1StbJMqN9v5unzxxu121vdiZrxAKzLm3gEHCdlb_uXb4/

@sherifnada
Copy link
Contributor

thanks @cgardens!

cc @alexandr-shegeda

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues priority/high High priority type/enhancement New feature or request
Projects
No open projects
Archived in project
3 participants