-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: updated workflows for pull request and merge to main #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good, this is already reviewed.
.github/workflows/release.yaml
Outdated
@@ -0,0 +1,35 @@ | |||
name: Build and Deploy Lambda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update description.
.github/workflows/pull_request.yaml
Outdated
- name: Validate SAM Templates | ||
run: | | ||
for template in $(find . -name ".yaml" -o -name ".yml"); do | ||
echo "Validating template: $template" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see this getting logged, where can I see it.
run: sam build --template-file firehose-template.yaml --region us-east-2 | ||
|
||
- name: Package SAM Application | ||
run: sam package --s3-bucket unified-lambda-cft-1 --output-template-file firehose-template.yaml --region us-east-2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why this bucket? unified-lambda-cft-1
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this upload trigger template?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Since same bucket name cannot be created across different accounts as well , I put it unified-lambda-serverless-1. The original bucket is unified-lambda-serverless.
run: sam package --s3-bucket unified-lambda-cft-1 --output-template-file firehose-template.yaml --region us-east-2 | ||
|
||
- name: Upload CloudFormation Template to S3 | ||
run: aws s3 cp firehose-template.yaml s3://unified-lambda-serverless-1/firehose-template.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not the prod bucket. Are you planning to change it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Once we move to Jim Rupert, would change
validate sam template is failing, please look into this. |
This PR consists of workflows for pull request and release.
Pull Request workflow consists of:
a. Installing sam cli.
b. Validating all the templates using sam.
Release workflow consists of :
a. Install sam cli.
b. Build, package and publishing the template file to s3 bucket.