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
docker: drop jmx and tools-java from installation #18566
docker: drop jmx and tools-java from installation #18566
Conversation
@@ -69,9 +69,6 @@ packages=( | |||
"build/dist/$config/debian/$product-conf_$version-$release-1_$arch.deb" | |||
"build/dist/$config/debian/$product-kernel-conf_$version-$release-1_$arch.deb" |
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.
why do we need kernel configuration inside the docker image? What does it do?
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.
It's one of Scylla-server dependency
dist/docker/debian/build_docker.sh
Outdated
@@ -98,7 +95,7 @@ run apt-get -y upgrade | |||
run apt-get -y --no-install-suggests install dialog apt-utils | |||
run bash -ec "echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections" | |||
run bash -ec "rm -rf /etc/rsyslog.conf" | |||
run apt-get -y --no-install-suggests install hostname supervisor openjdk-11-jre-headless python2 python3 python3-yaml curl rsyslog sudo | |||
run apt-get -y --no-install-suggests install hostname supervisor python2 python3 python3-yaml curl rsyslog sudo |
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.
who's still using python2? (unrelated to this issue of course)
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.
Based on 27e3266 it's related, I can remove it now
Following the work done in dd07796, removing the scylla-jmx and scylla-tools-java from our docker image
feb5208
to
01a4f1d
Compare
🟢 CI State: SUCCESS✅ - Build Build Details:
|
@scylladb/scylla-maint can you please merge this? |
Following the work done in dd07796, removing the Scylla-jmx and Scylla-tools-java from our docker image