Skip to content

chore(clo): remove e2e job in favor of one that runs using bundle - #83126

Merged
openshift-merge-bot[bot] merged 2 commits into
openshift:mainfrom
jcantrill:remove_e2e_job
Aug 10, 2026
Merged

chore(clo): remove e2e job in favor of one that runs using bundle#83126
openshift-merge-bot[bot] merged 2 commits into
openshift:mainfrom
jcantrill:remove_e2e_job

Conversation

@jcantrill

@jcantrill jcantrill commented Aug 7, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

This PR:

  • removes the test-e2e job and build of the old registry image
  • Places CLO e2e reliance upon the bundle built from CI

cc @Clee2691 @vparfonov

/hold

Summary by CodeRabbit

  • Updates OpenShift CI configuration for the Cluster Logging Operator.
  • Removes the legacy registry image build and e2e-target test.
  • Makes e2e-using-bundle required and uses the CI-built bundle for functional tests.
  • Removes obsolete CI configuration for ViaQ-vector-v0.50.0-rh.
  • Pins Cluster Logging Operator base images in the ViaQ-vector-v0.54.0-rh configuration to version 6.6.

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 7, 2026
@jcantrill

jcantrill commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

/pj-rehearse auto-ack

@coderabbitai

coderabbitai Bot commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

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: 09b87706-b8f2-41af-a49b-7779e2c9af42

📥 Commits

Reviewing files that changed from the base of the PR and between b136251 and cd4361f.

⛔ Files ignored due to path filters (4)
  • ci-operator/jobs/ViaQ/vector/ViaQ-vector-v0.50.0-rh-postsubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/ViaQ/vector/ViaQ-vector-v0.50.0-rh-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/cluster-logging-operator/openshift-cluster-logging-operator-master-postsubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/cluster-logging-operator/openshift-cluster-logging-operator-master-presubmits.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (3)
  • ci-operator/config/ViaQ/vector/ViaQ-vector-v0.50.0-rh.yaml
  • ci-operator/config/ViaQ/vector/ViaQ-vector-v0.54.0-rh.yaml
  • ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-master.yaml
💤 Files with no reviewable changes (2)
  • ci-operator/config/ViaQ/vector/ViaQ-vector-v0.50.0-rh.yaml
  • ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-master.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • ci-operator/config/ViaQ/vector/ViaQ-vector-v0.54.0-rh.yaml

Walkthrough

The CI configuration removes the retired Vector v0.50.0 configuration and obsolete Cluster Logging Operator registry and end-to-end test wiring. It pins three base images to 6.6 and makes e2e-using-bundle required.

Changes

Cluster Logging CI configuration

Layer / File(s) Summary
Pin Cluster Logging base images
ci-operator/config/ViaQ/vector/ViaQ-vector-v0.54.0-rh.yaml		 The three Cluster Logging Operator base images now use 6.6.
Remove obsolete CI wiring
ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-master.yaml		 The configuration removes the registry image build, the e2e-target test, and the registry image dependency. It makes e2e-using-bundle required.
Remove retired Vector configuration
ci-operator/config/ViaQ/vector/ViaQ-vector-v0.50.0-rh.yaml		 The Vector v0.50.0 CI configuration is removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: clee2691, cahartma

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: replacing the CLO e2e job with a bundle-based e2e job.
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 The pull request changes only CI YAML and generated job files; it adds, removes, or renames no Ginkgo test titles.
Test Structure And Quality ✅ Passed The complete PR diff changes only CI YAML/job configuration; it adds or modifies no Go or Ginkgo test code, so the stated test-quality requirements are not applicable.
Microshift Test Compatibility ✅ Passed The patch changes only CI YAML files, and the added-line scan found no new Ginkgo tests or MicroShift-incompatible API references.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The diff changes only CI configuration and job YAML; it adds no Ginkgo tests or multi-node assumptions requiring SNO review.
Topology-Aware Scheduling Compatibility ✅ Passed The PR changes only ci-operator config and job YAML; it adds no deployment manifests, operator code, controllers, or topology-based scheduling constraints.
Ote Binary Stdout Contract ✅ Passed The PR changes or deletes only six YAML CI configuration files; no OTE binary, suite setup, logging, or stdout-writing source code changed.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The commit adds no Ginkgo e2e tests or source files; its only additions are three YAML image-tag lines, and the remaining changes remove or edit CI declarations.
No-Weak-Crypto ✅ Passed The PR adds only three YAML image-tag changes; the complete diff contains no MD5, SHA1, DES, RC4, Blowfish, ECB, custom crypto, or secret-comparison usage.
Container-Privileges ✅ Passed The complete patch adds no privileged, hostPID, hostNetwork, hostIPC, SYS_ADMIN, allowPrivilegeEscalation, or root security settings.
No-Sensitive-Data-In-Logs ✅ Passed The commit only removes CI jobs/configuration and changes three image tags; it adds no logging statements or sensitive values.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@openshift-ci
openshift-ci Bot requested review from Clee2691 and cahartma August 7, 2026 19:12
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 7, 2026
@openshift-merge-bot

openshift-merge-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

@jcantrill: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@jcantrill

jcantrill commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

/pj-rehearse auto-ack

@openshift-merge-bot

openshift-merge-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

@jcantrill: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci

openshift-ci Bot commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

@jcantrill: 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/rehearse/openshift/cluster-logging-operator/master/functional-target a0b0c32 link unknown /pj-rehearse pull-ci-openshift-cluster-logging-operator-master-functional-target
ci/rehearse/openshift/cluster-logging-operator/master/e2e-using-bundle a0b0c32 link unknown /pj-rehearse pull-ci-openshift-cluster-logging-operator-master-e2e-using-bundle

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.

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

@jcantrill

jcantrill commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

/pj-rehearse auto-ack

@openshift-merge-bot

openshift-merge-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

@jcantrill: your /pj-rehearse request was not processed because the request waited in queue for longer than 5 minutes. Please retry in a few minutes.

@openshift-merge-bot

openshift-merge-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@jcantrill: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-openshift-cluster-logging-operator-master-e2e-using-bundle openshift/cluster-logging-operator presubmit Presubmit changed
pull-ci-openshift-cluster-logging-operator-master-ci-bundle-cluster-logging-operator-bundle openshift/cluster-logging-operator presubmit Ci-operator config changed
pull-ci-openshift-cluster-logging-operator-master-functional-target openshift/cluster-logging-operator presubmit Ci-operator config changed
pull-ci-openshift-cluster-logging-operator-master-images openshift/cluster-logging-operator presubmit Ci-operator config changed
pull-ci-openshift-cluster-logging-operator-master-lint openshift/cluster-logging-operator presubmit Ci-operator config changed
pull-ci-openshift-cluster-logging-operator-master-unit openshift/cluster-logging-operator presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-cargo-clippy-check ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-cargo-deny-check-bans ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-cargo-fmt-check ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-clo-functional ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-cluster-logging-operator-e2e ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-images ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-ldd-openssl ViaQ/vector presubmit Ci-operator config changed
pull-ci-ViaQ-vector-v0.54.0-rh-unit ViaQ/vector presubmit Ci-operator config changed
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@jcantrill

jcantrill commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

/hold cancel

@jcantrill

jcantrill commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

/pj-rehearse ack

@openshift-merge-bot

openshift-merge-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

@jcantrill: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-merge-bot openshift-merge-bot Bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Aug 10, 2026
@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 10, 2026
@vparfonov

vparfonov commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 10, 2026
@openshift-ci

openshift-ci Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcantrill, vparfonov

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-merge-bot
openshift-merge-bot Bot merged commit 7b9aa1f into openshift:main Aug 10, 2026
16 checks passed
@jcantrill
jcantrill deleted the remove_e2e_job branch August 10, 2026 20:14
jzding pushed a commit to jzding/release that referenced this pull request Aug 11, 2026
@jcantrill @jzding
chore(clo): remove e2e job in favor of one that runs using bundle (op…
c182852 
…enshift#83126)

* chore(clo): remove e2e job in favor of one that runs using bundle

* chore(vector): update v0.54 for logging 6.6.  Remove obs job
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cahartma cahartma Awaiting requested review from cahartma
@Clee2691 Clee2691 Awaiting requested review from Clee2691

Assignees

@vparfonov vparfonov

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. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jcantrill @vparfonov