Skip to content

(ansible): update ansible base images to use multi-stage builds#6333

Merged
everettraven merged 2 commits into
operator-framework:masterfrom
everettraven:bugfix/gh6302
Feb 23, 2023
Merged

(ansible): update ansible base images to use multi-stage builds#6333
everettraven merged 2 commits into
operator-framework:masterfrom
everettraven:bugfix/gh6302

Conversation

@everettraven

Copy link
Copy Markdown
Contributor

Description of the change:
Updates the ansible operator images base.Dockerfile to multi-stage builds so that we can ensure all packages are always included in the images we deliver by installing/building all packages in a build stage and copying them over to the final build stage.

Motivation for the change:
fixes #6302

Checklist

If the pull request includes user-facing changes, extra documentation is required:

so that we can ensure all packages are always included in the images we deliver
by installing/building all packages in a build stage and copying them over
to the final build stage.

fixes operator-framework#6302

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 19:57 — with GitHub Actions Inactive
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@oceanc80

Copy link
Copy Markdown
Collaborator

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Feb 23, 2023
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:22 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:22 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:23 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:23 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:23 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:23 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy February 23, 2023 20:26 — with GitHub Actions Inactive
@grokspawn

Copy link
Copy Markdown
Contributor

/approve

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

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package 'cryptography' is missing from non-amd64 images in ansible-operator images > v1.26.0

3 participants