You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The CI/CD pipeline checks if it needs to build the action server docker image or not.
It does this by checking if any of the changed files of the most recent commits require a re-build.
This is not good for two reasons:
It is not fail safe, because it could be that there are no model changes in the current commit, but there were in the previous commit, and if the build during the previous commit failed, the docker image that is stored on ECR is old.
We're duplicating logic, because docker build & docker push already include all the required logic to build or push changes.
It is proposed that we simply always build & push the action server image, and rely on smarts available in the docker cli to build & push anything that needs to be updated.
The text was updated successfully, but these errors were encountered:
@b-quachtran
Can you also let me know what you think about this proposal?
It is related to #122 , which is for training of models, while this is issue for building the action server image. I created two separate issues, because the solution is different for each.
@ArjaanBuijk In this proposal, would images be built & pushed irrespective of changes to actions/ or is it still conditioned on changes to actions/ where the build process will run on every commit?
@b-quachtran
It would always run, because even if during the current commit there are not changes in the actions/, that does not guarantee that the docker image in ECR is the latest, because the CI/CD pipeline could have failed during a previous commit.
Hmm, what about changing the check to be on pull_request and checking for changes in actions/ in that case? That way builds can only succeed / fail when there are relevant changes present and they'll happen on every commit for a given PR that requires a new image build.
The CI/CD pipeline checks if it needs to build the action server docker image or not.
It does this by checking if any of the changed files of the most recent commits require a re-build.
This is not good for two reasons:
It is proposed that we simply always build & push the action server image, and rely on smarts available in the docker cli to build & push anything that needs to be updated.
The text was updated successfully, but these errors were encountered: