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

add openssl to ubi image #15929

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Conversation

kaisecheng
Copy link
Contributor

Logstash on ECK requires openssl command to build TLS keystore. logstash-ubi:8.12.0 has openssl command, but 8.12.1 and 8.12.2 don't.
This commit adds microdnf install -y openssl to ensure the command exists in ubi image

relates: elastic/cloud-on-k8s#7544

@kaisecheng kaisecheng requested a review from robbavey February 9, 2024 16:41
@kaisecheng
Copy link
Contributor Author

build ubi image locally

rake artifact:docker_ubi8

get openssl version

docker run --rm --entrypoint openssl docker.elastic.co/logstash/logstash-ubi8:8.13.0-SNAPSHOT version
OpenSSL 1.1.1k  FIPS 25 Mar 2021

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
0.0% 0.0% Duplication on New Code

See analysis details on SonarQube

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Copy link
Member

@robbavey robbavey left a comment

Choose a reason for hiding this comment

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

Works as advertised

➜  logstash git:(kaisecheng-ubi_image_add_openssl) ✗ docker run --rm --entrypoint openssl docker.elastic.co/logstash/logstash-ubi8:8.13.0-SNAPSHOT version

OpenSSL 1.1.1k  FIPS 25 Mar 2021

LGTM

@kaisecheng kaisecheng merged commit 4e6815b into elastic:main Feb 9, 2024
5 checks passed
@kaisecheng
Copy link
Contributor Author

@logstashmachine backport 8.12

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

Successfully merging this pull request may close these issues.

4 participants