-
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
Update CI to publish docker image #40
Conversation
3872c2c
to
fbe1fc6
Compare
fbe1fc6
to
daf052c
Compare
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.
🥇 Looks good and straightforward.
Could we create a staging branch or whatever the name to plug this workflow to try 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.
🚀
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.
LGTM,
Do not forget to cleanup this image if you do not use it : https://hub.docker.com/layers/lasuite/people/pr-40/images/sha256-cd1a93e3a04a9122bab215afbc6bbc4d2eebeffde38419dc793d2a471a5b7ec1?context=explore
daf052c
to
872b694
Compare
Added Docker Hub username and password, to shared secrets.
Refactored 'Hub' CI job for clarity, using 'docker/build-push-action.' This dedicated workflow efficiently manages image releases on push tag and main branch merges events. 'Hub' job was broken by Chat GPT translation from Circle CI. Images are pushed to a temporary Docker Hub repository, lasuite/people. Duplicated 'build-docker' job was removed from people workflow.
872b694
to
ae92de7
Compare
Purpose
Publish production Docker images for staging deployment.
Proposal
I have followed the basic usage of these configurations.
Images are pushed to this public Docker Hub repository.
It closes, #39 .