Skip to content

Add Containerfile.art, art.yaml, and image-references for ART builds#448

Open
shruti-rh wants to merge 5 commits into
openshift:cert-manager-1.19from
shruti-rh:cert-manager-1.19
Open

Add Containerfile.art, art.yaml, and image-references for ART builds#448
shruti-rh wants to merge 5 commits into
openshift:cert-manager-1.19from
shruti-rh:cert-manager-1.19

Conversation

@shruti-rh

@shruti-rh shruti-rh commented Jun 26, 2026

Copy link
Copy Markdown

Problem Statement

ART needs to build cert-manager-operator images through its Konflux-based layered products pipeline. ART builds require a dedicated Containerfile.art with ART's builder images, base images, and LABEL conventions.

Related Issue

Part of ART-14816 (Onboard OAP operators to ART).

Proposed Changes

Add Containerfile.art adapted from cert-manager-operator-release release-1.19 for ART build pipeline (COPY . . instead of submodule, ART builder images and labels).

shruti-rh added 5 commits June 9, 2026 12:56
Adapted from cert-manager-operator-release Containerfile.cert-manager-operator
for ART build pipeline

Signed-off-by: Shruti Anekar <sanekar@redhat.com>
- ART tooling reqs

Signed-off-by: Shruti Anekar <sanekar@redhat.com>
Signed-off-by: Shruti Anekar <sanekar@redhat.com>
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: ca20e826-e5e7-4890-b1de-8ea502f067a6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@openshift-ci openshift-ci Bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jun 26, 2026
@openshift-ci

openshift-ci Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Hi @shruti-rh. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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 kubernetes-sigs/prow repository.

@bharath-b-rh bharath-b-rh 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

Comment thread bundle/image-references
from:
kind: DockerImage
name: openshift.io/cert-manager-operator:latest
- name: jetstack-cert-manager-rhel9

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.

@shruti-rh Could you please let us know, how the ImageStreams will be used? Because all below upstream images in the bundle will use the same downstream image jetstack-cert-manager-rhel9.

  • quay.io/jetstack/cert-manager-webhook:v1.20.2
  • quay.io/jetstack/cert-manager-cainjector:v1.20.2
  • quay.io/jetstack/cert-manager-controller:v1.20.2

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 30, 2026
@bharath-b-rh

Copy link
Copy Markdown
Contributor

/ok-to-test

@openshift-ci

openshift-ci Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bharath-b-rh, shruti-rh

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

@openshift-ci openshift-ci Bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Jun 30, 2026
@openshift-ci

openshift-ci Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

@shruti-rh: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/images e6571ea link true /test images
ci/prow/e2e-operator-tech-preview e6571ea link false /test e2e-operator-tech-preview
ci/prow/e2e-operator e6571ea link true /test e2e-operator

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants