Skip to content

WIP: DO_NOT_MERGE: test e2e jobs - #621

Draft
mpatlasov wants to merge 1 commit into
openshift:mainfrom
mpatlasov:WIP-DO_NOT_MERGE-test-e2e-jobs2
Draft

WIP: DO_NOT_MERGE: test e2e jobs#621
mpatlasov wants to merge 1 commit into
openshift:mainfrom
mpatlasov:WIP-DO_NOT_MERGE-test-e2e-jobs2

Conversation

@mpatlasov

Copy link
Copy Markdown
Contributor

No description provided.

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 8, 2026
@openshift-ci

openshift-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

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

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 497942b4-9617-4540-9b61-e9625e5f1217

📥 Commits

Reviewing files that changed from the base of the PR and between f7573c1 and 1cf430b.

📒 Files selected for processing (1)
  • pkg/operator/starter.go

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


📝 Walkthrough

Walkthrough

The change adds a DUMMY CHANGE comment after the package declaration in pkg/operator/starter.go. It does not modify exported entities or functional behavior.

Suggested reviewers: romanbednar

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 1cf43

This update only adds a source comment and has no runtime or user-facing impact.

🚥 Pre-merge checks | ✅ 14 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The pull request has no description. The available context does not provide a clear author-written explanation of the change. Add a brief description that explains the end-to-end job testing objective and the purpose of the dummy change in pkg/operator/starter.go.
✅ Passed checks (14 passed)
Check name Status Explanation
Title check ✅ Passed The title identifies this pull request as a work-in-progress change for testing end-to-end jobs. It matches the stated pull request objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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 PASS. The pull request changes only pkg/operator/starter.go by adding the // DUMMY CHANGE comment. The diff contains no test files, Ginkgo test titles, or runtime values. Therefore, it introduces …
Test Structure And Quality ✅ Passed PASS. The parent-to-HEAD diff changes only pkg/operator/starter.go by adding the // DUMMY CHANGE comment and a blank line. It does not modify Ginkgo tests, test setup or cleanup, waits, cluster op…
Microshift Test Compatibility ✅ Passed PASS. The pull request changes only pkg/operator/starter.go by adding the // DUMMY CHANGE comment. The commit adds no Ginkgo e2e tests or test constructs such as It(), Describe(), Context(),…
Single Node Openshift (Sno) Test Compatibility ✅ Passed PASS: The pull request changes only pkg/operator/starter.go by adding the // DUMMY CHANGE comment. The parent-to-HEAD diff contains no new Ginkgo declarations or e2e tests. Therefore, the SNO comp…
Topology-Aware Scheduling Compatibility ✅ Passed PASS. The pull request changes only pkg/operator/starter.go by adding a comment and a blank line. The diff introduces no deployment manifest, controller behavior, replica logic, affinity, topology s…
Ote Binary Stdout Contract ✅ Passed PASS. The pull request changes only pkg/operator/starter.go by adding the comment // DUMMY CHANGE. The exact commit diff contains no stdout write, logging configuration, or executable code change.…
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PASS: The pull request changes only pkg/operator/starter.go by adding the // DUMMY CHANGE comment. It adds no Ginkgo e2e tests, IPv4 assumptions, or external connectivity requirements.
No-Weak-Crypto ✅ Passed PASS. The parent-to-HEAD diff changes only pkg/operator/starter.go by adding the comment // DUMMY CHANGE. The changed patch adds no weak-crypto primitive, crypto implementation, or secret comparis…
Container-Privileges ✅ Passed PASS: The pull request changes only pkg/operator/starter.go, adding the comment // DUMMY CHANGE. The diff contains no container or Kubernetes manifest changes and no privilege-related tokens. Ther…
No-Sensitive-Data-In-Logs ✅ Passed PASS: The pull request adds only the // DUMMY CHANGE comment and a blank line in pkg/operator/starter.go. The exact diff adds no logging call, log message, or sensitive data. Existing logging is u…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/pipeline required

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Scheduling required tests:
/test e2e-aws-csi
/test e2e-aws-ovn-upgrade
/test e2e-azure
/test e2e-azure-csi
/test e2e-azure-file-csi
/test e2e-azure-file-nfs-csi
/test e2e-azure-ovn-upgrade
/test e2e-gcp
/test e2e-gcp-csi
/test e2e-gcp-ovn-upgrade

Scheduling tests matching the pipeline_run_if_changed or not excluded by pipeline_skip_if_only_changed parameters:
/test e2e-aws-csi-extended
/test e2e-azure-csi-extended
/test e2e-azure-csi-hypershift
/test e2e-azure-file-csi-extended
/test e2e-azure-file-csi-hypershift
/test e2e-azure-file-nfs-csi-hypershift
/test e2e-azurestack-csi

@openshift-ci

openshift-ci Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mpatlasov

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 Sep 8, 2026
@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test all

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test hypershift-e2e-openstack-aws-csi-cinder

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test hypershift-e2e-openstack-aws-csi-manila

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test hypershift-aws-e2e-external

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test e2e-gcp

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test e2e-openstack-cinder-csi

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test e2e-openstack-manila-csi

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test hypershift-e2e-openstack-aws-csi-manila

@mpatlasov

Copy link
Copy Markdown
Contributor Author

/test e2e-gcp-csi

@openshift-ci

openshift-ci Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@mpatlasov: 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/e2e-azure-file-csi-hypershift 1cf430b link false /test e2e-azure-file-csi-hypershift
ci/prow/e2e-azure-file-nfs-csi-hypershift 1cf430b link false /test e2e-azure-file-nfs-csi-hypershift
ci/prow/hypershift-e2e-aks 1cf430b link false /test hypershift-e2e-aks
ci/prow/smb-win2022-operator-e2e 1cf430b link false /test smb-win2022-operator-e2e
ci/prow/e2e-aws-csi-extended 1cf430b link false /test e2e-aws-csi-extended
ci/prow/smb-win2019-operator-e2e 1cf430b link false /test smb-win2019-operator-e2e
ci/prow/e2e-azurestack-csi 1cf430b link false /test e2e-azurestack-csi
ci/prow/e2e-azure-csi-extended 1cf430b link false /test e2e-azure-csi-extended
ci/prow/e2e-azure-csi-hypershift 1cf430b link false /test e2e-azure-csi-hypershift
ci/prow/e2e-azure-file-csi-extended 1cf430b link false /test e2e-azure-file-csi-extended
ci/prow/aws-efs-single-zone-operator-e2e-extended 1cf430b link false /test aws-efs-single-zone-operator-e2e-extended
ci/prow/aws-efs-operator-e2e-extended 1cf430b link false /test aws-efs-operator-e2e-extended
ci/prow/hypershift-aws-e2e-external 1cf430b link true /test hypershift-aws-e2e-external
ci/prow/hypershift-e2e-openstack-aws-csi-cinder 1cf430b link true /test hypershift-e2e-openstack-aws-csi-cinder
ci/prow/e2e-gcp 1cf430b link true /test e2e-gcp
ci/prow/e2e-openstack-cinder-csi 1cf430b link true /test e2e-openstack-cinder-csi
ci/prow/e2e-openstack-manila-csi 1cf430b link true /test e2e-openstack-manila-csi
ci/prow/e2e-gcp-csi 1cf430b link true /test e2e-gcp-csi
ci/prow/hypershift-e2e-openstack-aws-csi-manila 1cf430b link true /test hypershift-e2e-openstack-aws-csi-manila

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. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant