-
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
Deprecate mysqld
in vitess/lite
and use mysql:8.0.30
image for the operator
#14990
Conversation
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
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 <florent.poinsard@outlook.fr>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #14990 +/- ##
=======================================
Coverage 47.31% 47.32%
=======================================
Files 1142 1142
Lines 239009 239009
=======================================
+ Hits 113091 113106 +15
+ Misses 117327 117312 -15
Partials 8591 8591 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
mysql:8.0.30
image for the operatormysqld
in vitess/lite
and use mysql:8.0.30
image for the operator
Do we still want to keep the MySQL packages with the binaries installed in the container? Or should that also be removed? |
That will be removed in >= v20 |
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.
🥳
Description
This PR uses the
mysql:8.0.30
image in the vitess-operator example and deprecates themysqld
binary in thevitess/lite
Docker image.The plan as of now is to fully remove the installation of
mysqld
in thevitess/lite
image byv20
.Checklist
Deployment Notes