-
-
Notifications
You must be signed in to change notification settings - Fork 97
1918 fix frontend build action #1943
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
Conversation
updated node setup config
updated checkout action version
…cchrty/VRMS into 1918-fix-frontend-build-action reject changes
update aws-actions/configure-aws-credentials to v3
update aws-actions/configure-aws-credentials to v3
fix: type on env variable
updated aws action versions to latest
add force deployment step for missing service
updated AWS_APP_NAme to match previous passed test
wrong file path for dockerfile
updated role and variable access
added permission token per error
match devops incubator test
updated build and push to ECR steps
pulling updates from dev
merged updates from development into 1918 fix frontend build action
|
@trillium Please review this PR whenever you have a moment :) Once fixed, then we should potentially rebuild PROD with the latest feature updates for Bonnie to review on Thursday's call. |
trillium
left a comment
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.
Hey hey, stoked that this is working, there's a knick-knack or two that need to be addressed though, please see comments.
I see that the deploy works though (re checking out the actions) thank you!!!
| fi | ||
| outputs: | ||
| AWS_APPENV: ${{ env.AWS_APPENV }} | ||
| AWS_APPENV: ${{ env.AWS_APPENV }}git checkou |
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.
Pretty sure this is a typo
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.
Yep! fixed.
Removes command line typo
Merging updates from development
1918 fix frontend build action
|
@trillium Can you please review this PR this week to unblock rebuilding Dev / Prod? |
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.
Pull Request Overview
This PR modernizes the frontend deployment workflow by switching to AWS OIDC role assumptions, updating naming and paths, and streamlining build/deploy steps.
- Switches
aws-actions/configure-aws-credentialsto v3 with OIDC role assumption and removes static secrets - Updates
AWS_APP_NAMEand Dockerfile path, and consolidates Docker build into a custom shell step - Replaces multiple ECS action steps with a single AWS CLI
update-servicecall
Comments suppressed due to low confidence (3)
.github/workflows/aws-frontend-deploy.yml:18
- The Dockerfile path and build context may be mismatched: you cd into
${{ env.DOCKER_PATH }}(client) but referenceDockerfile.prodat the repo root. Either move the Dockerfile into the client directory or updateDOCKER_PATHto.so the file is found in the build context.
DOCKERFILE: Dockerfile.prod
.github/workflows/aws-frontend-deploy.yml:63
- Consider using the
${{ env.AWS_REGION }}variable here instead of hardcodingus-west-2, to keep region configuration consistent and centralized.
aws-region: us-west-2
.github/workflows/aws-frontend-deploy.yml:73
- [nitpick] The
lscommand appears to be leftover debugging output and isn’t required for the build step. Removing it will clean up the workflow logs.
ls
trillium
left a comment
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.
Approved!
Thanks for getting this working again @lcchrty
| AWS_APP_NAME: vrms-frontend | ||
| AWS_REGION: us-west-2 | ||
| DOCKERFILE: client/Dockerfile.prod | ||
| DOCKERFILE: Dockerfile.prod |
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.
Hey there, can you clarify which file you're pointing to here? I don't see a Dockerfile.prod in the base dir of the monorepo
Fixes #1918
What changes did you make and why did you make them ?