umotif-public / terraform-aws-bastion Goto Github PK

What is the current behavior?
Route53 isn't updating A record when Auto Scaling Group pushes the new EC2 instance

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Kill the EC2 process in Auto Scaling Group by setting desired capacity to 0 and then set it to 1 (min capacity should be set to 0)

What is the expected behavior?
After the new instance of EC2 is pushed, the A record in route53 should be updated with the latest EC2 public ip

Software versions?

I'm using this module to create a bastion host in my AWS environment. My setup involves having two separate AWS accounts: one for the environment resources (e.g. staging, production, including the bastion host) and another for management resources like Route 53.

Issue

The module, as currently designed, does not associate an Elastic IP with the bastion host, causing the public IP to be dynamic. Additionally, since the module attempts to create a Route 53 record in the same account as the bastion host, it doesn't fit well with my multi-account setup (AWS organisation) where Route 53 is hosted in a separate management account.

Desired Outcome

a. Add an option to associate an Elastic IP with the bastion host, or
b. Provide a way to specify a different AWS provider for creating the Route 53 A-record, so it can be created in a separate AWS account.

Would you consider adding these features? They'd be incredibly helpful for scenarios like mine.

Thank you

What is the current behavior?
Auto Scale Schedules are defined Etc/UTC timezone only

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Not a bug - feature request

What is the expected behavior?
The API allows us to define a local timezone so I'd like to do that in the module
https://aws.amazon.com/about-aws/whats-new/2021/03/amazon-ec2-auto-scaling-adds-support-for-local-time-zones-for-scheduled-scaling/

Software versions?
I had this requirement so I've forked the project and made a simple update. Would you like the addition as an MR?