Skip to content

Commit

Permalink
init ocp test
Browse files Browse the repository at this point in the history
  • Loading branch information
leiicamundi committed May 14, 2024
1 parent 62182ad commit 49e4a73
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/actions/rosa-create-cluster/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ runs:
bucket_policy_path=".github/actions/rosa-create-cluster/bucket-policy.json"
# Replace placeholders in the bucket policy file and apply the policy
sed -i "s|\${{ inputs.s3-backend-bucket }}|${{ inputs.s3-backend-bucket }}|g" "$bucket_policy_path"
sed -i "s|{{ inputs.s3-backend-bucket }}|${{ inputs.s3-backend-bucket }}|g" "$bucket_policy_path"
aws s3api put-bucket-policy --bucket ${{ inputs.s3-backend-bucket }} --region ${{ inputs.aws-region }} --policy file://$bucket_policy_path
- name: Terraform Init
Expand Down
6 changes: 3 additions & 3 deletions .github/actions/rosa-create-cluster/bucket-policy.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"Effect": "Deny",
"Principal": "*",
"Action": "s3:PutObject",
"Resource": "arn:aws:s3:::${{ inputs.s3-backend-bucket }}/*",
"Resource": "arn:aws:s3:::{{ inputs.s3-backend-bucket }}/*",
"Condition": {
"StringNotEquals": {
"s3:x-amz-server-side-encryption": "AES256"
Expand All @@ -18,7 +18,7 @@
"Effect": "Deny",
"Principal": "*",
"Action": "s3:PutObject",
"Resource": "arn:aws:s3:::${{ inputs.s3-backend-bucket }}/*",
"Resource": "arn:aws:s3:::{{ inputs.s3-backend-bucket }}/*",
"Condition": {
"Null": {
"s3:x-amz-server-side-encryption": "true"
Expand All @@ -30,7 +30,7 @@
"Effect": "Deny",
"Principal": "*",
"Action": "s3:GetObject",
"Resource": "arn:aws:s3:::${{ inputs.s3-backend-bucket }}/*"
"Resource": "arn:aws:s3:::{{ inputs.s3-backend-bucket }}/*"
}
]
}

0 comments on commit 49e4a73

Please sign in to comment.