-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Build and push Docker vitess/base
and component images to DockerHub from GitHub Actions
#14271
Conversation
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
4f0a0ef
to
857286d
Compare
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
- name: Set Docker tag name | ||
if: startsWith(github.ref, 'refs/tags/') && matrix.branch == 'latest' |
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.
For vitess/base
on a new tag, we only build vitess/base:vX.X.X
, using the default Dockerfile, we ignore all other flavor of MySQL/Percona
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
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
… from GitHub Actions (#14271) Signed-off-by: Florent Poinsard <[email protected]>
… from GitHub Actions (vitessio#14271) Signed-off-by: Florent Poinsard <[email protected]>
… from GitHub Actions (vitessio#14271) Signed-off-by: Florent Poinsard <[email protected]>
Description
This is a follow-up of #14243.
Goal
This PR adds a new workflow:
Docker Build Base
that runs on bothmain
and new tags. The goal of this workflow is:vitess/build
imagevitess/k8s
image, using both supported debian versionvitess/vtgate
, using both supported debian versionBuilt/pushed images
For each execution of this new workflow a total of 40 images are built and pushed. A full list can be found below. Note that if we are building and pushing due to a new git tag, the Docker Tag
latest
is replaced by the git tag name and all other Docker Tag are prefixed by the git tag name.vitess/base
(latest
,mysql57
,percona57
, andpercona80
)vitess/k8s
(latest
,bookworm
, andbullseye
)latest
,bookworm
, andbullseye
)Example
On
main
Here is a sample run on my own fork after a push to
main
: https://github.com/frouioui/vitess/actions/runs/6554596808. Thefrouioui/base
image can be found on DockerHub. Same thing for thefrouioui/k8s
image here. All other components are also visible in my list of repositories on DockerHub.On a new tag
An example run on my own fork is available on https://github.com/frouioui/vitess/actions/runs/6554972519.
Related Issue(s)
Checklist
Deployment Notes