Hapag-Lloyd / terraform-aws-bastion-host-ssm

Terraform module to create an enterprise grade bastion host: High availability, SSM access only, encrypted disk and flexible resource naming.
Apache License 2.0
9 stars 4 forks source link

ci: do not update the AMI on `apply` #190

Closed kayman-mk closed 9 months ago

kayman-mk commented 10 months ago

Description

What is the overall goal of your PR? Which problem does it solve? Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #(issue number)

Migrations required

yes: please describe the migration no: please delete the whole paragraph

Verification

Please mention the examples you have verified. Did you check the change in your environment?

Checklist

github-actions[bot] commented 10 months ago

Hey @kayman-mk! 👋

Thank you for your contribution to the project. Please refer to the contribution rules for a quick overview of the process.

Make sure that this PR clearly explains:

With submitting this PR you confirm that you hold the rights of the code added and agree that it will published under the Apache 2.0 license.

The following ChatOps commands are supported:

Simply add a comment with the command in the first line. If you need to pass more information, separate it with a blank line from the command.

This message was generated automatically. You are welcome to improve it.

kayman-mk commented 9 months ago

Superseded by #199

There are so many edge cases --> let the user decide, which image to use.