Skip to content

openshift-logging: Remove configuration for Logging 6.3#80787

Open
xperimental wants to merge 1 commit into
openshift:mainfrom
xperimental:remove-logging-6.3
Open

openshift-logging: Remove configuration for Logging 6.3#80787
xperimental wants to merge 1 commit into
openshift:mainfrom
xperimental:remove-logging-6.3

Conversation

@xperimental

@xperimental xperimental commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Logging 6.3 is end-of-life for a while already, we do not plan to make any further development on these branches.

/cc @jcantrill @JoaoBraveCoding
/assign @xperimental

Summary by CodeRabbit

This pull request removes the OpenShift CI infrastructure configuration for Logging version 6.3, which has reached end-of-life. The changes eliminate CI/CD configuration for two key components:

Removed CI Configurations:

  • cluster-logging-operator release-6.3: Removes the CI operator configuration that previously defined build inputs, container image tags, promotion rules to the logging namespace, and CI test jobs (lint, unit coverage, and e2e tests with specific image dependencies and timeouts)
  • loki release-6.3: Removes the corresponding CI operator configuration for the loki component, including its build root setup, promotion targets, and test workflows

Removed Image Mirroring Rules:
The image mirroring mapping file for logging 6.3 is deleted, which previously maintained Quay image tag mappings for the loki-operator and cluster-logging-operator images along with their operator bundle variants. This prevents new builds from being automatically mirrored to external registries.

Practical Impact:
This discontinues all automated CI/CD pipeline activity for the 6.3 release line. Components depending on cluster-logging-operator and loki 6.3 will no longer receive automated test execution or image mirroring to public registries, reflecting the end-of-life status of this version.

@coderabbitai

coderabbitai Bot commented Jun 19, 2026

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: a0623c95-f356-4a59-8599-aa88f6dc5173

📥 Commits

Reviewing files that changed from the base of the PR and between 878b7ce and 88f103d.

⛔ Files ignored due to path filters (4)
  • ci-operator/jobs/openshift/cluster-logging-operator/openshift-cluster-logging-operator-release-6.3-postsubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/cluster-logging-operator/openshift-cluster-logging-operator-release-6.3-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/loki/openshift-loki-release-6.3-postsubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/loki/openshift-loki-release-6.3-presubmits.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (3)
  • ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-release-6.3.yaml
  • ci-operator/config/openshift/loki/openshift-loki-release-6.3.yaml
  • core-services/image-mirroring/openshift-logging/mapping_logging_6_3_quay
💤 Files with no reviewable changes (3)
  • core-services/image-mirroring/openshift-logging/mapping_logging_6_3_quay
  • ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-release-6.3.yaml
  • ci-operator/config/openshift/loki/openshift-loki-release-6.3.yaml

Walkthrough

Three files are deleted: the CI operator pipeline configurations for openshift/cluster-logging-operator and openshift/loki on release-6.3, and the quay image mirroring mapping file for OpenShift logging 6.3. No new files or replacements are introduced.

Changes

Cohort / File(s) Summary
Remove logging 6.3 CI configs
ci-operator/config/openshift/cluster-logging-operator/openshift-cluster-logging-operator-release-6.3.yaml, ci-operator/config/openshift/loki/openshift-loki-release-6.3.yaml
Deletes CI operator configs defining build inputs, promotion rules, and test jobs (lint, unit, e2e) for the release-6.3 branch of both operators.
Remove logging 6.3 image mirroring
core-services/image-mirroring/openshift-logging/mapping_logging_6_3_quay
Removes quay tag mappings for loki-operator, cluster-logging-operator, and their *-operator-bundle images under the logging 6.3 namespace.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

lgtm, approved, rehearsals-ack

🚥 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 accurately and specifically describes the main change: removing configuration for OpenShift Logging version 6.3 across multiple CI/CD configuration files.
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 PR removes Logging 6.3 CI configuration only; no Ginkgo tests present in repository or modified by this change, making the test naming check inapplicable.
Test Structure And Quality ✅ Passed This PR removes OpenShift Logging 6.3 configuration files and adds 11 standard Go unit tests. No Ginkgo tests are present, making the Ginkgo-specific quality check inapplicable.
Microshift Test Compatibility ✅ Passed PR removes CI configuration files only; no new Ginkgo e2e tests are added, so MicroShift compatibility check is not applicable.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PR removes EOL Logging 6.3 configuration; contains no new Ginkgo e2e tests. SNO compatibility check is not applicable to removals.
Topology-Aware Scheduling Compatibility ✅ Passed PR removes CI configuration files for EOL Logging 6.3 only. Does not add/modify deployment manifests, operator code, or controllers that would require topology-aware scheduling review.
Ote Binary Stdout Contract ✅ Passed PR only removes CI configuration files (YAML and mapping files), not source code or binaries. OTE Stdout Contract check is inapplicable to configuration-only changes.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PR removes CI configuration and image mapping files for EOL Logging 6.3 (no new Ginkgo tests added). Check only applies to new test additions.
No-Weak-Crypto ✅ Passed No weak cryptography detected. PR removes logging 6.3 configs and adds CI docs/config with no cryptographic vulnerabilities or weak ciphers.
Container-Privileges ✅ Passed PR removes EOL Logging 6.3 config files; no privileged container settings added. Remaining configs have zero instances of privileged: true, hostPID/Network/IPC, SYS_ADMIN, or allowPrivilegeEscalation.
No-Sensitive-Data-In-Logs ✅ Passed No sensitive data (passwords, tokens, API keys, PII, hostnames, etc.) found in the deleted configuration files. Only legitimate CI/CD configuration metadata and image references are present.

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

✨ 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 and usage tips.

@openshift-ci

openshift-ci Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: xperimental

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 Jun 19, 2026
@openshift-merge-bot openshift-merge-bot Bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Jun 19, 2026
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@xperimental: no rehearsable tests are affected by this change

Note: If this PR includes changes to step registry files (ci-operator/step-registry/) and you expected jobs to be found, try rebasing your PR onto the base branch. This helps pj-rehearse accurately detect changes when the base branch has moved forward.

@openshift-ci

openshift-ci Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant