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

support connecting via VPNs/bastion instances #24

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

rukai
Copy link
Member

@rukai rukai commented Oct 23, 2023

Progress towards #23

Currently aws-throwaway will connect to instances over the internet via the instances public ip address.
This is very convenient as it means the user does not have to setup any kind of VPN or bastion instance.
However being able to connect to a private ip via a VPN or bastion instance is very useful and even required in many environments.
To support both of these use cases, this PR introduces a config setting that decides whether to connect to instances private or public ip address.

This PR:

  • refactors the Aws struct to be created by a builder struct: AwsBuilder
  • adds a use_public_addresses method to the AwsBuilder struct
    • refer to the documentation of the method for more info.

I also changed aws-throwaway/examples/aws-throwaway-test.rs so that it doesnt leave files around making it more suitable for testing the project.

@rukai rukai marked this pull request as ready for review October 23, 2023 05:36
@rukai rukai force-pushed the use_private_addresses branch 2 times, most recently from d6d304f to d20e5ef Compare October 23, 2023 05:39
@rukai rukai requested a review from conorbros October 23, 2023 05:49
@rukai rukai changed the title add aws builder with use_private_addresses support running via VPNs/bastion instances Oct 23, 2023
@rukai rukai changed the title support running via VPNs/bastion instances support connecting via VPNs/bastion instances Oct 23, 2023
@rukai rukai merged commit 75a80f4 into shotover:main Oct 23, 2023
3 checks passed
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.

3 participants