Skip to content
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

Update build-deploy-ec2.yml #628

Closed
wants to merge 1 commit into from
Closed

Conversation

paulespinosa
Copy link
Member

Automatically build and deploy code that reaches the main branch. Allow only one deployment job to run at a time because there are no other synchronization mechanisms in place.

What changes did you make?

  • Changed the build-deploy-ec2 workflow so that it runs on merge to main and PR to main as the target.

Rationale behind the changes?

  • The api and app tests are now running on commits, merge to main, PR targeting the main branch. This gives some assurance that code can be deployed to the dev environment automatically without manual intervention. The caveat is when there are environment variable updates then the EC2 instance will need to be updated and the code manually deployed.

Testing done for these changes

  • Can't test until it reaches the main branch.

What did you learn or can share that is new?(optional)

  • GitHub workflows have concurrency controls to allow one workflow or job to run and one pending.

Automatically build and deploy code that reaches the main branch. Allow only one deployment job to run at a time because there are no other synchronization mechanisms in place.
@paulespinosa
Copy link
Member Author

Incubator deployment is underway. This version will not be needed.

@paulespinosa paulespinosa deleted the deploy-main-patch-1 branch August 29, 2024 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant