Skip to content

MGMT-22799: add Containerfile for pytest test image - #19

Merged
openshift-merge-bot[bot] merged 1 commit into
osac-project:mainfrom
omer-vishlitzky:MGMT-22799
Mar 23, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
osac-project:mainfrom
omer-vishlitzky:MGMT-22799

Conversation

@omer-vishlitzky

Copy link
Copy Markdown
Contributor

https://redhat.atlassian.net/browse/MGMT-22799

Self-contained container image for running the e2e test suite on bare metal machines via podman.

Contents:

  • Python 3.11 + pytest (via uv)
  • fulfillment-cli (from GitHub release)
  • oc/kubectl (from OpenShift mirror)
  • grpcurl (from GitHub release)

Depends on #18 (pyproject.toml and uv.lock must exist on main).

Self-contained image with all tools needed to run the
e2e test suite on a bare metal machine via podman.

Depends on osac-project#18 (pyproject.toml and uv.lock must exist).
@openshift-ci-robot

openshift-ci-robot commented Mar 23, 2026 •

Copy link
Copy Markdown

@omer-vishlitzky: This pull request references MGMT-22799 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

https://redhat.atlassian.net/browse/MGMT-22799

Self-contained container image for running the e2e test suite on bare metal machines via podman.

Contents:

  • Python 3.11 + pytest (via uv)
  • fulfillment-cli (from GitHub release)
  • oc/kubectl (from OpenShift mirror)
  • grpcurl (from GitHub release)

Depends on #18 (pyproject.toml and uv.lock must exist on main).

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-merge-bot
openshift-merge-bot Bot merged commit de7dfd6 into osac-project:main Mar 23, 2026
3 checks passed

@eranco74 eranco74 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Clean Containerfile that packages the test suite with all dependencies.

Minor follow-up suggestions (non-blocking, already merged):

  • Pin UBI base image to specific version for reproducibility
  • Add non-root USER directive
  • Consider checksum verification for downloaded binaries
  • Optimize layer caching by copying pyproject.toml/uv.lock before test code

@openshift-ci

openshift-ci Bot commented Mar 23, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adriengentil, eranco74, omer-vishlitzky

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

The pull request process is described here

Details 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

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