From 3499b973013381ff669095406274233583fe2b77 Mon Sep 17 00:00:00 2001 From: cruizba Date: Wed, 7 Oct 2020 23:07:27 +0200 Subject: [PATCH] deployment-openvidu-pro: Only replace OPENVIDU_PRO_AWS_S3_BUCKET with created s3 bucket name if Recording is s3 --- .../pro/aws/cfn-openvidu-server-pro-no-market.yaml.template | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openvidu-server/deployments/pro/aws/cfn-openvidu-server-pro-no-market.yaml.template b/openvidu-server/deployments/pro/aws/cfn-openvidu-server-pro-no-market.yaml.template index 752eb10a..9b913da4 100644 --- a/openvidu-server/deployments/pro/aws/cfn-openvidu-server-pro-no-market.yaml.template +++ b/openvidu-server/deployments/pro/aws/cfn-openvidu-server-pro-no-market.yaml.template @@ -314,7 +314,7 @@ Rules: RuleCondition: Fn::Or: [ !Equals [!Ref Recording, 'disabled' ], !Equals [!Ref Recording, 'local' ] ] Assertions: - - AssertDescription: If recording Storage is 'disabled', you don't need to specify a S3 bucket. + - AssertDescription: If recording Storage is 'disabled' or 'local', you don't need to specify a S3 bucket. Assert: Fn::Equals: [ !Ref S3RecordingsBucketName, "" ] @@ -501,7 +501,7 @@ Resources: sed -i "s/#OPENVIDU_PRO_RECORDING_STORAGE=/OPENVIDU_PRO_RECORDING_STORAGE=${Recording}/" $WORKINGDIR/.env if [ ! -z "${S3RecordingsBucketName}" ]; then sed -i "s/#OPENVIDU_PRO_AWS_S3_BUCKET=/OPENVIDU_PRO_AWS_S3_BUCKET=${S3RecordingsBucketName}/" $WORKINGDIR/.env - else + elif [ "${Recording}" == "s3" ]; then sed -i "s/#OPENVIDU_PRO_AWS_S3_BUCKET=/OPENVIDU_PRO_AWS_S3_BUCKET=${s3BucketName}/" $WORKINGDIR/.env fi fi