Made the keypair parameter optional

* Removed stupid debug output * Made the keypair parameter optional
theonestack · Jul 2, 2021 · e46b8d0 · e46b8d0
1 parent 5f84e30
commit e46b8d0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/bastion.cfhighlander.rb b/bastion.cfhighlander.rb
@@ -8,7 +8,7 @@
     ComponentParam 'Ami', type: 'AWS::EC2::Image::Id'
     ComponentParam 'SpotPrice', ''
     ComponentParam 'InstanceType'
-    ComponentParam 'KeyName'
+    ComponentParam 'KeyName', ''
     ComponentParam 'DnsDomain'
     ComponentParam 'SubnetIds', type: 'CommaDelimitedList'
     ComponentParam 'VPCId', type: 'AWS::EC2::VPC::Id'

diff --git a/bastion.cfndsl.rb b/bastion.cfndsl.rb
@@ -3,6 +3,7 @@
   private_only = external_parameters.fetch(:private_only, false)
 
   Condition("SpotPriceSet", FnNot(FnEquals(Ref('SpotPrice'), '')))
+  Condition("KeypairSet", FnNot(FnEquals(Ref('KeyName'), '')))
 
   EC2_SecurityGroup('SecurityGroupBastion') do
     GroupDescription FnJoin(' ', [ Ref('EnvironmentName'), component_name ])
@@ -73,12 +74,10 @@
 
   if defined? userdata
     if userdata.is_a?(String)
-      puts("IS A STRING")
       bastion_userdata.push(*userdata.split("\n")) if defined? userdata
     end
 
     if userdata.kind_of?(Array)
-      puts("IS AN ARRAY")
       bastion_userdata.push(*userdata) if defined? userdata
     end
   end
@@ -88,7 +87,7 @@
     InstanceType Ref('InstanceType')
     AssociatePublicIpAddress true unless private_only.equal? true
     IamInstanceProfile Ref('InstanceProfile')
-    KeyName Ref('KeyName')
+    KeyName FnIf('KeypairSet', Ref('KeyName'), Ref('AWS::NoValue'))
     SpotPrice FnIf('SpotPriceSet', Ref('SpotPrice'), Ref('AWS::NoValue'))
     SecurityGroups [ Ref('SecurityGroupBastion') ]
     UserData FnBase64(FnJoin("",bastion_userdata))