-
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
Remove MySQL/Percona from the vitess/lite
Docker image
#15605
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
|
846c769
to
36ebcf0
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15605 +/- ##
==========================================
+ Coverage 65.80% 68.14% +2.33%
==========================================
Files 1561 1562 +1
Lines 195220 197558 +2338
==========================================
+ Hits 128474 134625 +6151
+ Misses 66746 62933 -3813 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Florent Poinsard <[email protected]> Switch from root to vitess users when installing packages Signed-off-by: Florent Poinsard <[email protected]> debug - use frouioui as the docker account Signed-off-by: Florent Poinsard <[email protected]> debug - use frouioui for the lite image Signed-off-by: Florent Poinsard <[email protected]> revert wip and debug Signed-off-by: Florent Poinsard <[email protected]>
fcef212
to
cd31ff4
Compare
Signed-off-by: Florent Poinsard <[email protected]>
Signed-off-by: Florent Poinsard <[email protected]>
@@ -1,11 +1,11 @@ | |||
# Copyright 2019 The Vitess Authors. |
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.
All the renames in this folder are weird. Did you actually use git mv
to rename these files? Or is GH inferring them as renames because of the similarity of content?
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.
The latter
@@ -7,7 +7,7 @@ RUN apt-get update | |||
RUN apt-get install -y sudo curl vim jq | |||
|
|||
# Install dependencies | |||
COPY docker/lite/install_dependencies.sh /vt/dist/install_dependencies.sh | |||
COPY docker/utils/install_dependencies.sh /vt/dist/install_dependencies.sh |
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.
Do we still need to have this script? It installs a number of packages, but are all those packages still relevant? or should we clean more in there too?
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 script is now only used by the local
and vttestserver
images so it is fine to keep it as it is since we still require mysqld/percona-server on those images.
Description
This Pull Request is a follow up of #14990. In
v20.0.0
we want to removemysqld
from thevitess/lite
image and remove the version-sepcific tags too.Related Issue
vitess/lite
image #15604