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

Dockerfile: do not build the legacy binary #2181

Merged

Conversation

jmontesi
Copy link
Contributor

@jmontesi jmontesi commented Jun 20, 2024

build-depends: redhatci/ansible-collection-redhatci-ocp#356

The container image is updated to avoid building the old binary and copying stuff that it is not required anymore.

The "certsuite" binary is installed in "/usr/local/bin" so that it can be directly called.

The GitHub workflows have been updated accordingly.

An update in DCI and QE will be necessary as well.

The container image is updated to avoid building the old binary and
copying stuff that it is not required anymore.

The "certsuite" binary is installed in "/usr/local/bin" so that it can
be directly called.

The GitHub workflows have been updated accordingly.

An update in DCI and QE will be necessary as well.
@dcibot
Copy link
Collaborator

dcibot commented Jun 20, 2024

@ramperher
Copy link
Collaborator

/dci-rerun

@ramperher
Copy link
Collaborator

Please do not merge this change until this change in the collections is validated: redhatci/ansible-collection-redhatci-ocp#356, else DCI automation will be broken

@dcibot
Copy link
Collaborator

dcibot commented Jun 20, 2024

Copy link
Collaborator

@ramperher ramperher left a comment

Choose a reason for hiding this comment

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

confirmed it's working, thanks!

@jmontesi jmontesi merged commit 4895509 into redhat-best-practices-for-k8s:main Jun 20, 2024
17 of 25 checks passed
@dcibot
Copy link
Collaborator

dcibot commented Jun 20, 2024

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.

5 participants