Merge pull request #579 from ministryofjustice/fix/reinstate-old-output

Reinstated original behaviour of security group output

README.md

@@ -185,6 +185,7 @@ In order to prevent older versions from being retained forever, in addition to t
 | <a name="output_bastion_launch_template"></a> [bastion\_launch\_template](#output\_bastion\_launch\_template) | Launch template of bastion |
 | <a name="output_bastion_s3_bucket"></a> [bastion\_s3\_bucket](#output\_bastion\_s3\_bucket) | S3 bucket of bastion |
 | <a name="output_bastion_security_group"></a> [bastion\_security\_group](#output\_bastion\_security\_group) | Security group of bastion |
+| <a name="output_bastion_security_group_map"></a> [bastion\_security\_group\_map](#output\_bastion\_security\_group\_map) | Security group details of bastion |
 <!-- END_TF_DOCS -->
 
 [Standards Link]: https://operations-engineering-reports.cloud-platform.service.justice.gov.uk/public-report/modernisation-platform-terraform-bastion-linux "Repo standards badge."

outputs.tf

@@ -1,5 +1,10 @@
 output "bastion_security_group" {
   description = "Security group of bastion"
+  value       = aws_security_group.bastion_linux.id
+}
+
+output "bastion_security_group_map" {
+  description = "Security group details of bastion"
   value       = aws_security_group.bastion_linux
 }
 

test/unit-test/outputs.tf

@@ -10,7 +10,7 @@ output "bastion_s3_bucket" {
 
 output "bastion_security_group" {
   description = "Bastion security groups"
-  value       = { for idx, bastion in module.bastion_linux : "bastion_${idx}" => bastion.bastion_security_group.id }
+  value       = { for idx, bastion in module.bastion_linux : "bastion_${idx}" => bastion.bastion_security_group_map.id }
 }
 
 output "bastion_kms_key" {