Merge pull request #10 from malikparvez/work

remove iam role
malikparvez · Jul 12, 2024 · e894ad5 · e894ad5
2 parents 491f4b4 + e2cb403
commit e894ad5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -13,17 +13,17 @@ This project is a Serverless application that parses CloudFront logs stored in a
 ## Setup using github actions
 
 ### Step1: Fork this repo
+Update the serverless.yml with desired s3 bucket name
 
 ### Step2: Add secrets:
 Add the following secrets to your GitHub repository settings:
 
 ```bash
-AWS_ACCESS_KEY
-AWS_SECRET_KEY
+MY_AWS_ACCESS_KEY
+MY_AWS_SECRET_KEY
 OPENSEARCH_HOST
 OPENSEARCH_USERNAME
 OPENSEARCH_PASSWORD
-BUCKET_NAME
 ```
 
 ### Step3: Deploy the Application:

diff --git a/serverless.yml b/serverless.yml
@@ -1,6 +1,6 @@
 frameworkVersion: "3"
 
-service: cloudfront-logs-parser-lambda-opensearch
+service: cloudfrontLogs
 useDotenv: true
 
 provider:
@@ -14,9 +14,6 @@ provider:
     OPENSEARCH_USERNAME: ${env:OPENSEARCH_USERNAME}
     OPENSEARCH_PASSWORD: ${env:OPENSEARCH_PASSWORD}
   region: ap-south-1
-  iam:
-    role:
-      name: cloudfront-logs-role 
 
 package:
   patterns: