Skip to content

remove konflux config in favor of ART builds#220

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
weshayutin:artoverflux
Mar 26, 2026
Merged

remove konflux config in favor of ART builds#220
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
weshayutin:artoverflux

Conversation

@weshayutin
Copy link
Copy Markdown
Contributor

  • OADP comnmits to building $next version via art
    immediately after $version release
  • konflux built containers will be built on-going for
    rosa and art teams as discussed with Toni.

Need review and ack from @celebdor and @rayfordj

* OADP comnmits to building $next version via art
immediately after $version  release
* konflux built containers will be built on-going for
rosa and art teams as discussed with Toni.

Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e1e4942a-3fa3-4c87-9a74-c19edcc64d02

📥 Commits

Reviewing files that changed from the base of the PR and between 601cc2a and dcaf969.

📒 Files selected for processing (2)
  • .tekton/oadp-hypershift-oadp-plugin-main-pull-request.yaml
  • .tekton/oadp-hypershift-oadp-plugin-main-push.yaml
💤 Files with no reviewable changes (2)
  • .tekton/oadp-hypershift-oadp-plugin-main-push.yaml
  • .tekton/oadp-hypershift-oadp-plugin-main-pull-request.yaml

Walkthrough

Two Tekton PipelineRun manifests defining multi-architecture OCI image build and security scanning workflows for OADP HyperShift plugin—one for pull-request events and one for push events—were entirely deleted from the repository.

Changes

Cohort / File(s) Summary
Tekton Pipeline Manifests
.tekton/oadp-hypershift-oadp-plugin-main-pull-request.yaml, .tekton/oadp-hypershift-oadp-plugin-main-push.yaml
Complete removal of two parameterized PipelineRun manifests (~1,297 lines total) that orchestrated multi-stage CI/CD workflows including repository cloning, per-platform image builds, security scanning (Clair, SAST, ClamAV, RPM signature validation), compliance checks, and artifact management for containerized builds.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

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

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

@openshift-ci openshift-ci bot requested review from bryan-cox and devguyio March 23, 2026 17:43
@rayfordj
Copy link
Copy Markdown
Contributor

💭 Deleting the component in Konflux and letting it create the removal PR may be the better route, but I'm not objecting to any course that ultimately ends in its removal from the tenant on the stone-prd-rh01 cluster for all branches.

@weshayutin , I defer to repo owners for necessary acks & such to make this a reality. Thanks!

@weshayutin
Copy link
Copy Markdown
Contributor Author

@jparrill wdyt?

@jparrill
Copy link
Copy Markdown
Contributor

/cc @rayfordj

@openshift-ci openshift-ci bot requested a review from rayfordj March 24, 2026 08:46
@jparrill
Copy link
Copy Markdown
Contributor

/cc @celebdor

@openshift-ci openshift-ci bot requested a review from celebdor March 24, 2026 08:46
@jparrill
Copy link
Copy Markdown
Contributor

/assign @jparrill

@celebdor
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 25, 2026
@jparrill
Copy link
Copy Markdown
Contributor

/test verify

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 25, 2026

@weshayutin: all tests passed!

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.

@kaovilai
Copy link
Copy Markdown
Member

cherrypick wont work.. these are -main- files

@jparrill
Copy link
Copy Markdown
Contributor

/approve

@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 26, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jparrill, weshayutin

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 the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 26, 2026
@openshift-merge-bot openshift-merge-bot bot merged commit d62f95e into openshift:main Mar 26, 2026
6 checks passed
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants