Skip to content

Commit 4655270

Browse files
Enrico Usaienrico-usai
authored andcommitted
Add missing condition for MasterPublicIp Output
The Master Public IP from the MasterServer resource is only available if MasterPublicIp condition is verified. We lost this condition when created two substacks for the master and the computes. Signed-off-by: Enrico Usai <[email protected]>
1 parent fd716b1 commit 4655270

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

cloudformation/aws-parallelcluster.cfn.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2790,7 +2790,7 @@
27902790
"MasterENI": {
27912791
"Ref": "MasterENI"
27922792
},
2793-
"MasterPublicIp": {
2793+
"UseMasterPublicIp": {
27942794
"Fn::If": [
27952795
"MasterPublicIp",
27962796
true,

cloudformation/master-server-substack.cfn.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ Parameters:
2424
Type: String
2525
MasterENI:
2626
Type: String
27-
MasterPublicIp:
27+
UseMasterPublicIp:
2828
Type: String
2929
ImageId:
3030
Type: AWS::EC2::Image::Id
@@ -221,8 +221,8 @@ Conditions:
221221
- !Equals
222222
- !Ref 'UpdateWaiterFunctionArn'
223223
- NONE
224-
MasterPublicIp: !Equals
225-
- !Ref 'MasterPublicIp'
224+
HasMasterPublicIp: !Equals
225+
- !Ref 'UseMasterPublicIp'
226226
- 'true'
227227
Resources:
228228
MasterServer:
@@ -627,7 +627,7 @@ Outputs:
627627
MasterPublicIP:
628628
Description: Public IP Address of the Master host
629629
Value: !GetAtt 'MasterServer.PublicIp'
630-
Condition: MasterPublicIp
630+
Condition: HasMasterPublicIp
631631
MasterPrivateDnsName:
632632
Description: Private DNS name of the Master host
633633
Value: !GetAtt 'MasterServer.PrivateDnsName'

0 commit comments

Comments
 (0)