-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
DRAFT :: Enhancement: solve issue #29255 - Change docker image from CentOS to Ubuntu20.04 #29374
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -42,27 +42,48 @@ ENV BEAT_SETUID_AS={{ .user }} | |
{{- if contains .from "ubi-minimal" }} | ||
RUN for iter in {1..10}; do microdnf update -y && microdnf install -y shadow-utils jq && microdnf clean all && exit_code=0 && break || exit_code=$? && echo "microdnf error: retry $iter in 10s" && sleep 10; done; (exit $exit_code) | ||
{{- else }} | ||
# Installing jq needs to be installed after epel-release and cannot be in the same yum install command. | ||
RUN case $(arch) in aarch64) YUM_FLAGS="-x bind-license";; esac; \ | ||
for iter in {1..10}; do \ | ||
yum update -y $YUM_FLAGS && \ | ||
yum install -y epel-release && \ | ||
yum update -y $YUM_FLAGS && \ | ||
yum install -y jq && \ | ||
yum clean all && \ | ||
exit_code=0 && break || exit_code=$? && echo "yum error: retry $iter in 10s" && sleep 10; \ | ||
|
||
RUN for iter in {1..10}; do \ | ||
apt update -y && \ | ||
apt install --no-install-recommends --yes jq && \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You need to add So it should be come:
|
||
apt clean all && \ | ||
exit_code=0 && break || exit_code=$? && echo "apt error: retry $iter in 10s" && sleep 10; \ | ||
done; \ | ||
(exit $exit_code) | ||
{{- end }} | ||
|
||
## Changes Notes: To migratre from CentOS to Ubuntu, the following changes are needed: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Those are all packages migrated from CentOs to Ubuntu related There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this temporarily for discussion or you plane to level this in here? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hello @ruflin, this is for temporary discussion, I'll try to find where I add them to a changelog for future notes. 😸 |
||
# atk -> libatk1.0-0 | ||
# gtk -> GTK3 is installed by default REMOVED | ||
# gdk -> libgdk-pixbuf2.0-dev | ||
# xrandr -> x11-xserver-utils | ||
# pango -> libpango-1.0-0 | ||
# libXcomposite -> libxcomposite-dev | ||
# libXcursor -> libxcursor-dev | ||
# libXdamage -> libxdamage-dev | ||
# libXext -> libxext-dev | ||
# libXi -> libxi-dev | ||
# libXtst -> libxtst-dev | ||
# libXScrnSaver -> libxss-dev | ||
# libXrandr -> libxrandr-dev | ||
# GConf2 -> libgconf2-4 | ||
# alsa-lib -> libasound2 | ||
# gtk3 -> GTK3 is installed by default REMOVED | ||
# ipa-gothic-fonts -> fonts-ipafont-gothic | ||
# xorg-x11-fonts-100dpi -> xfonts-100dpi | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @jsoriano @andrewkroh Do you remember what requires the xorg dependencies? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Blind guess: abrt-xorg.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That must be a dependency required by Synthetics. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah, I missed the part below that it is only included in the "complete" image. Good to know it is not in the standard image. @andrewvc FYI There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yep! |
||
# xorg-x11-fonts-75dpi -> xfonts-75dpi | ||
# xorg-x11-utils -> x11-utils | ||
# xorg-x11-fonts-cyrillic -> xfonts-cyrillic | ||
# xorg-x11-fonts-Type1 -> GET at xfonts-base | ||
# xorg-x11-fonts-misc -> xfonts-base | ||
{{- if (and (contains .image_name "-complete") (not (contains .from "ubi-minimal"))) }} | ||
RUN for iter in {1..10}; do \ | ||
yum -y install atk gtk gdk xrandr pango libXcomposite libXcursor libXdamage \ | ||
libXext libXi libXtst libXScrnSaver libXrandr GConf2 \ | ||
alsa-lib atk gtk3 ipa-gothic-fonts xorg-x11-fonts-100dpi xorg-x11-fonts-75dpi xorg-x11-utils \ | ||
xorg-x11-fonts-cyrillic xorg-x11-fonts-Type1 xorg-x11-fonts-misc \ | ||
yum clean all && \ | ||
exit_code=0 && break || exit_code=$? && echo "yum error: retry $iter in 10s" && sleep 10; \ | ||
RUN apt update -y && \ | ||
for iter in {1..10}; do \ | ||
apt install --no-install-recommends --yes libatk1.0-0 libgdk-pixbuf2.0-dev x11-xserver-utils libpango-1.0-0 libxcomposite-dev libxcursor-dev libxdamage-dev \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here add the |
||
libxext-dev libxi-dev libxtst-dev libxss-dev libxrandr-dev gconf2 \ | ||
libasound2 fonts-ipafont-gothic xfonts-100dpi xfonts-75dpi xfonts-cyrillic xfonts-scalable xfonts-base x11-utils &&\ | ||
apt clean all && \ | ||
exit_code=0 && break || exit_code=$? && echo "apt error: retry $iter in 10s" && sleep 10; \ | ||
done; \ | ||
(exit $exit_code) | ||
ENV NODE_PATH={{ $beatHome }}/.node | ||
|
@@ -213,3 +234,4 @@ RUN echo -e '#!/bin/sh\nexec /usr/local/bin/docker-entrypoint' > /app/apm.sh && | |
{{- else }} | ||
ENTRYPOINT ["/usr/bin/tini", "--", "/usr/local/bin/docker-entrypoint"] | ||
{{- end }} | ||
|
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 YUM_FLAGS was not requested when running ubuntu