Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Ec2 connect tunnel #325

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Ec2 connect tunnel #325

wants to merge 9 commits into from

Conversation

qrilka
Copy link
Contributor

@qrilka qrilka commented Apr 28, 2020

Please also note that these are not hard requirements, but merely serve to define
what maintainers are looking for in PR's. Including these will more likely lead
to your PR being reviewed and accepted.

  • Update the changelog
  • Make sure that modules and files are documented. This can be done inside the module and files.
  • Make sure that new modules directories contain a basic README.md file.
  • Make sure that the module is added to tests/main.tf
  • Make sure that the linting passes on CI.
  • Make sure that there is an up to date example for your code:
    - For new modules this would entail example code for how to use the module or some explanation in the module readme.
    - For new examples please provide a README explaining how to run the example. It's also ideal to provide a basic makefile to use the example as well.
  • Make sure that there is a manual CI trigger that can test the deployment.

This PR uses #232 and #319 so needs those to be merged first

Magicloud and others added 6 commits April 14, 2020 19:49
Tested in examples/single-node-asg-tester

The change in all three modules is compatible with old code that defines just one data volume.
Since it is single node, binding an EIP to the instance is possible. And
it eases other things since the public interface is constant.

Add assign_eip variable to single-node-asg. If turns it on, an EIP will
be allocated, and assocated with the instance.

Scope VPC is specified in case the account does not have a default VPC.
}

output "eip_address" {
value = module.asg.eip_address
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JoseD92 this output already exists as public_ip

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

Successfully merging this pull request may close these issues.

3 participants