Skip to content

ADR: Disable CI Testing at Stream EOL (008) - #24

Open
zdrapela wants to merge 8 commits into
redhat-developer:mainfrom
zdrapela:adr/006-disable-ci-testing-at-stream-eol
Open

ADR: Disable CI Testing at Stream EOL (008)#24
zdrapela wants to merge 8 commits into
redhat-developer:mainfrom
zdrapela:adr/006-disable-ci-testing-at-stream-eol

Conversation

@zdrapela

@zdrapela zdrapela commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Documents existing practice: stop CI E2E / nightly testing for an RHDH minor stream at published EOL
  • Clarifies that support-only support exceptions do not keep testing in scope (Support tickets only)
  • Leaves Konflux/productization and future delivery-bearing EUS out of scope
  • Outcome of discussion at the RHDH Architecture call

Test plan

  • Confirm ADR number 008 does not conflict with other open ADR PRs
  • Team review of Context / Decision / Alternatives / Consequences
  • Confirm wording matches Architecture call outcome
  • ADR Number Check CI passes

Captures existing practice that support-only exceptions do not keep E2E/nightly testing in scope.
Numbers 006 and 007 are already taken on redhat-developer/rhdh-adr.
@zdrapela
zdrapela force-pushed the adr/006-disable-ci-testing-at-stream-eol branch from 62f09a3 to fd8bda7 Compare August 4, 2026 16:26
@zdrapela zdrapela changed the title ADR: Disable CI Testing at Stream EOL ADR: Disable CI Testing at Stream EOL (008) Aug 4, 2026
Clarify E2E/nightly scope, no-restore rule, life-cycle cite, on-demand alternative, and template consequence style.
Align the documented stop-at-EOL practice with both continuous nightlies and PR checks living as CI jobs.
Replace bold-lead paragraphs and heavy inline markup that rendered as large underlined blocks with ### sections and a blockquote for exception wording.
@zdrapela
zdrapela marked this pull request as ready for review August 4, 2026 16:56
@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

ADR 008: Stop CI testing for minor streams at published EOL

📝 Documentation 🕐 10-20 Minutes

Grey Divider

AI Description

• Document policy: stop E2E nightlies and PR-check CI jobs when a stream reaches published EOL
• Clarify support-only exceptions do not extend CI coverage; Support handles tickets
• Record scope, alternatives, and consequences from Architecture call alignment
Diagram

graph TD
  A["Published life cycle"] --> B{"Stream at EOL?"} -->|"No"| C["Keep CI jobs"]
  A --> B -->|"Yes"| D["Stop E2E nightlies"] --> E["Stop PR-check jobs"] --> F{"Support-only exception?"} -->|"Yes"| G["Support tickets only"]
  A --> B -->|"Yes"| D --> E --> F -->|"No"| H["No support (EOL)"]
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Add an operational runbook/checklist alongside the ADR
  • ➕ Turns the policy into a repeatable EOL procedure (who disables what, when, and where)
  • ➕ Reduces ambiguity for new CI maintainers and release engineering
  • ➖ Adds process documentation that may need updates as CI tooling changes
  • ➖ Slightly increases scope beyond documenting the decision
2. Automate EOL-driven CI job disabling
  • ➕ Reduces manual steps and the chance of leaving EOL jobs running
  • ➕ Makes the stop condition consistently enforced from a single source of truth
  • ➖ Requires additional engineering/CI platform work and ownership
  • ➖ May be hard if job definitions are spread across multiple repos/systems

Recommendation: The ADR approach is appropriate for capturing the decision and aligning expectations. Consider a follow-up PR for a lightweight runbook (or automation if feasible) so the documented policy reliably translates into CI actions at each stream’s EOL.

Files changed (1) +91 / -0

Documentation (1) +91 / -0
008-disable-ci-testing-at-stream-eol.mdAdd ADR documenting CI stop-at-EOL policy for minor streams +91/-0

Add ADR documenting CI stop-at-EOL policy for minor streams

• Introduces ADR 008 defining that E2E nightlies and PR-check CI jobs for a minor stream stop at the stream’s published EOL. Clarifies support-only exceptions do not keep CI in scope, enumerates alternatives considered, and documents positive/negative consequences and out-of-scope items (Konflux/productization, future EUS).

decisions/008-disable-ci-testing-at-stream-eol.md

@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

To customize comments, go to the Qodo configuration screen, or learn more in the docs.

Qodo Logo

Policy applies at EOL for any RHDH release stream (major or minor).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant