Skip to content
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

[PKG-288]: Updates xtradb cluster Dockerfiles to use ubi9-minimal as … #1080

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

maqeel75
Copy link
Contributor

@maqeel75 maqeel75 commented Dec 12, 2024

PKG-288 Powered by Pull Request Badge

…base image.

EvgeniyPatlan
EvgeniyPatlan previously approved these changes Dec 12, 2024
@EvgeniyPatlan EvgeniyPatlan requested a review from hors December 12, 2024 14:05
curl -Lf -o /tmp/jq.rpm https://mirror.stream.centos.org/9-stream/AppStream/x86_64/os/Packages/jq-1.6-14.el9.x86_64.rpm; \
curl -Lf -o /tmp/oniguruma.rpm https://downloads.percona.com/downloads/packaging/oniguruma-6.9.6-1.el9.6.x86_64.rpm; \
curl -Lf -o /tmp/pv.rpm http://download.fedoraproject.org/pub/epel/9/Everything/x86_64/Packages/p/pv-1.6.20-1.el9.x86_64.rpm; \
curl -Lf -o /tmp/socat.rpm https://downloads.percona.com/downloads/packaging/socat-1.7.4.1-6.el9.x86_64.rpm; \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have any reason to install socat from Percona repo? As In can see UBI9 has the same version socat;1.7.4.1-6.el9;x86_64;ubi-9-appstream-rpms the same about oniguruma.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hors Thanks for pointing this out. Fixed the issue.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hors Can you please check now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants