-
Notifications
You must be signed in to change notification settings - Fork 22
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 mysql container image. #533
Conversation
483272a
to
6484587
Compare
6484587
to
7194029
Compare
7194029
to
fb8d2cb
Compare
fb8d2cb
to
4984f4b
Compare
6f11be8
to
bba493b
Compare
aa58594
to
b059e1d
Compare
8be7724
to
7768a77
Compare
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[email protected]>
- name: Wait for MySQLCluster | ||
run: kubectl wait -n default --for=condition=Available mysqlcluster/test --timeout=180s |
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.
Currently we are only testing until MySQLCluster is ready.
When the MySQL image is pushed to ghcr.io, we will rewrite the E2E manifests and modify it to run the E2E test using the built mysqld image.
https://github.com/cybozu-go/moco/blob/main/e2e/testdata/single.yaml#L26
containers/mysql/README.md
Outdated
# MySQL container | ||
|
||
This directory provides a Dockerfile to build a MySQL container for [MOCO](https://github.com/cybozu-go/moco). | ||
This also provides `moco-init` command to initialize MySQL data for MOCO. |
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.
From this update, mysql images will not contain moco-init.
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.
Remove it: 027acfe
@@ -0,0 +1,2 @@ | |||
* | |||
!workspace |
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.
This exclusion is not necessary.
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.
Thanks, deleted.
2c2a3e4
Signed-off-by: d-kuro <[email protected]>
Signed-off-by: d-kuro <[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
refs: #535
Push mysqldcontainer image to ghcr.io.
This PR only pushes the image.
The pushed image needs to be changed to be used by MOCO, but this will be done in another PR.