OPRUN-4608: UPSTREAM: <carry>: Delete openshift/registry.Dockerfile

[joelanford]

Summary

• Delete openshift/registry.Dockerfile — this Dockerfile built the e2e-test-registry image which is no longer used by CI after Remove e2e-test-registry image build from operator-controller main config release#78581 removed it from the CI config.

Context

The dynamic per-scenario catalog system replaced the pre-built registry image. The CI config removal landed first (openshift/release#78581), so this Dockerfile is now dead code.

Test plan

• CI passes without the file
• No other files reference openshift/registry.Dockerfile

Jira: OPRUN-4608

Summary by CodeRabbit

• Chores
  • Removed legacy build configuration file.

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[openshift-ci-robot]

@joelanford: This pull request references OPRUN-4608 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 story to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

• Delete openshift/registry.Dockerfile — this Dockerfile built the e2e-test-registry image which is no longer used by CI after Remove e2e-test-registry image build from operator-controller main config release#78581 removed it from the CI config.

Context

The dynamic per-scenario catalog system replaced the pre-built registry image. The CI config removal landed first (openshift/release#78581), so this Dockerfile is now dead code.

Test plan

• CI passes without the file
• No other files reference openshift/registry.Dockerfile

Jira: OPRUN-4608

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.

[coderabbitai]

Walkthrough

The openshift/registry.Dockerfile file has been completely removed from the repository. This file previously contained a multi-stage Docker build that compiled a push utility binary and prepared container image artifacts for OpenShift deployment.

Changes

Cohort / File(s)	Summary
Dockerfile Deletion openshift/registry.Dockerfile	Complete removal of the OpenShift registry Dockerfile, including the multi-stage build configuration that compiled testdata/push/push.go and prepared container artifacts.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 12

✅ Passed checks (12 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names	✅ Passed	Custom check applies only to Ginkgo tests. This codebase uses standard Go testing, not Ginkgo. PR only removes a Dockerfile, no test names were modified.
Test Structure And Quality	✅ Passed	This PR removes openshift/registry.Dockerfile, a configuration file, not Ginkgo test code. The custom check targets Ginkgo test quality and is not applicable to Dockerfile deletions.
Microshift Test Compatibility	✅ Passed	New e2e tests use only standard Kubernetes APIs (Deployments, Pods, NetworkPolicies, RBAC) with no MicroShift-incompatible APIs detected.
Single Node Openshift (Sno) Test Compatibility	✅ Passed	SNO compatibility check applies only to new Ginkgo e2e tests. This PR removes a Dockerfile with no test code changes, so the check is not applicable.
Topology-Aware Scheduling Compatibility	✅ Passed	PR deletes a test Dockerfile, not deployment manifests or operator code. No Kubernetes scheduling constraints introduced or modified.
Ote Binary Stdout Contract	✅ Passed	PR only removes openshift/registry.Dockerfile, a Dockerfile (Docker config), not Go code. Custom check applies to stdout in OTE binaries' Go code, which is not affected by this change.
Ipv6 And Disconnected Network Test Compatibility	✅ Passed	This PR removes openshift/registry.Dockerfile (a container build file). The custom check applies only when new Ginkgo e2e tests are added. No test code is present.
Title check	✅ Passed	The title accurately describes the main change: deletion of openshift/registry.Dockerfile, which is the primary modification in this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: joelanford

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:

• DOWNSTREAM_OWNERS [joelanford]

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

[tmshort]

Needs rebase, and I'd like to get the sync PR in first...

[joelanford]

/hold

until #726 merges

[tmshort]

#726 merged

[joelanford]

/hold cancel

[openshift-ci]

@joelanford: 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.