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 podman script/alias support to build and push scripts #108

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

Jdubrick
Copy link
Contributor

What does this PR do / why we need it

Adds the necessary env script to the repository to enable the use of Podman or Docker as the container engine for the build and push scripts.

Which issue(s) does this PR fix

Fixes #?
resolves devfile/api#1355

PR acceptance criteria

  • Builds the images using Docker with USE_PODMAN=false and using Podman with USE_PODMAN=true

How to test changes / Special notes to the reviewer

Run the build and push scripts with the env set and unset to test the image building

Copy link
Member

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

/lgtm

Tried scripts and worked locally.

Copy link

openshift-ci bot commented Nov 18, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Jdubrick, michael-valdron

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Jdubrick Jdubrick merged commit 2223f1b into devfile:main Nov 20, 2023
3 checks passed
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.

Implement podman support for devfile-web
2 participants