IA-4812 Optimize CI: Deploy for IASO#3013
Conversation
dd70aed to
5a0bf7d
Compare
tdethier
left a comment
There was a problem hiding this comment.
Still need to test, so I'm leaving comments only
| @@ -0,0 +1,15 @@ | |||
| branch-defaults: | |||
There was a problem hiding this comment.
Should this file be part of the repo? Because I can use eb commands on the upgrade server without needing to include this file, I'm wondering why we have it
There was a problem hiding this comment.
It's needed for testing the CI of this branch, but we need to remove once it's ready tomerge
5a0bf7d to
4dbb0de
Compare
tdethier
left a comment
There was a problem hiding this comment.
Alright I think we've more or less covered the deployment CI, let's prepare the one that runs in PRs
| WFP_EMAIL_RECIPIENTS_NEW_ACCOUNT: ${WFP_EMAIL_RECIPIENTS_NEW_ACCOUNT} | ||
| WORKER: ${WORKER} | ||
|
|
||
| - APP_TITLE |
There was a problem hiding this comment.
Note for later: white listing env vars is very annoying for plugins because their env vars need to be white listed too. I guess each plugin will need another docker compose file that extends this one in order to do that?
There was a problem hiding this comment.
yes that is a great idea, the base (vanilla IASO) docker compose can extend the plugins. I guess that's the next step
dd43aea to
a3887a6
Compare
What problem is this PR solving?
Create an optimised CI setup for IASO that auto deploys to ElasticBeanstalk, using UV (~60% faster) and Docker HUB cache
Related JIRA tickets
IA-4812
Changes
Explain the changes that were made.
How to test
Push any change to this PR, this action will trigger
Notes
deploy_docker_manual.ymlin this PR is a WIP, and is intended to replacedeploy.ymlas in when it is ready.Doc
README.md