Skip to content

docs(e2e): document overlay fullsend triage in the CI Medic guide - #5296

Open
subhashkhileri wants to merge 3 commits into
redhat-developer:mainfrom
subhashkhileri:docs/ci-medic-overlay-fullsend
Open

docs(e2e): document overlay fullsend triage in the CI Medic guide#5296
subhashkhileri wants to merge 3 commits into
redhat-developer:mainfrom
subhashkhileri:docs/ci-medic-overlay-fullsend

Conversation

@subhashkhileri

Copy link
Copy Markdown
Member

Summary

  • Document overlay nightly triage in the CI Medic guide: Slack alerts in #rhdh-e2e-alerts / #rhdh-e2e-alerts-1-10 include an E2E AI Triage Summary GitHub comment from fullsend.
  • Spell out the medic workflow by category (test_fix, product_bug, environment, infra_flake): validate issues/PRs, merge obvious test fixes, ping workspace CODEOWNERS otherwise, /fs-fix when the approach is wrong, file Jira and merge skip PRs for product bugs, and comment on GitHub issues when the analysis is off so the next run has the right context.
  • Make it explicit that the Core AI Test Triager (@Nightly Test Alerts) is not configured for overlay — fullsend is the only automated triage for those jobs.

Test plan

  • Skim docs/e2e-tests/CI-medic-guide.md Overview + Overlay Nightly Failures section as a CI medic would on Day 1
  • Confirm Core vs overlay split is clear (AI Test Triager is Core-only; overlay uses the E2E AI Triage Summary link)
  • Follow the linked overlay GitHub issues, CODEOWNERS, and fullsend docs from Useful Links

Made with Cursor

Give CI medics a workflow for overlay nightly failures: fullsend classifies
them, files GitHub issues, and opens PRs that medics validate and merge.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.68%. Comparing base (1f6ef14) to head (2263434).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5296      +/-   ##
==========================================
- Coverage   63.71%   63.68%   -0.04%     
==========================================
  Files         121      121              
  Lines        2285     2288       +3     
  Branches      525      503      -22     
==========================================
+ Hits         1456     1457       +1     
- Misses        827      830       +3     
+ Partials        2        1       -1     
Flag Coverage Δ
rhdh 63.68% <ø> (-0.04%) ⬇️
Components Coverage Δ
Backend plugins 100.00% <ø> (ø)
Backend app 69.69% <ø> (+0.18%) ⬆️
Frontend app 58.82% <ø> (-0.07%) ⬇️
Plugin utils 100.00% <ø> (ø)
Dynamic plugins utils ∅ <ø> (∅)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f6ef14...2263434. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@subhashkhileri

subhashkhileri commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

@github-actions

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

- Check [Jira](https://redhat.atlassian.net/jira/dashboards/21388#v=1&d=21388&rf=acef7fac-ada0-4363-b3fb-9aad7ae021f0&static=f0579c09-f63e-45aa-87b9-05e042eee707&g=60993:view@0a7ec296-c2fd-4ddc-b7cb-64de0540e8ba) for existing issues with the **`ci-fail`** label.
- If it's a **new issue**, create a bug and assign it to the responsible team or person. The AI triager can also create Jira bugs directly.
- If the failure **blocks PRs**, mark the test as skipped (`test.fixme`) until it is fixed.
- **Overlay** (`redhat-developer/rhdh-plugin-export-overlays`): Open the **E2E AI Triage Summary** link in the Slack alert. Fullsend has already classified each workspace failure, reused or created GitHub issues, and opened fix PRs where it can. Your job is to validate that work -- see [Overlay Nightly Failures (Fullsend)](#overlay-nightly-failures-fullsend).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess you'll add the automated sending of the comments later? I see the issue for today's nightly (redhat-developer/rhdh-plugin-export-overlays#3382), but don't see it automatically sent to https://redhat-internal.slack.com/archives/C07D05S8U82/p1787637084192769

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the change was done yesterday, so it should appear today.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread docs/e2e-tests/CI-medic-guide.md Outdated
|---|---|---|
| **Prow job name** | `periodic-ci-redhat-developer-rhdh-{BRANCH}-e2e-...` | `periodic-ci-redhat-developer-rhdh-plugin-export-overlays-{BRANCH}-e2e-...` |
| **Automated triage** | **AI Test Triager** (`@Nightly Test Alerts`) -- inline Slack analysis | **Fullsend only** -- the **E2E AI Triage Summary** GitHub comment in the alert. The AI Test Triager is not configured for overlay. |
| **Tracking** | Jira with the **`ci-fail`** label | GitHub issues titled `[fullsend] E2E: ...` |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be Automated tracking instead? We still want to create ci-fail issues for bugs that aren't automatically fixed by Fullsend, right?

Comment thread docs/e2e-tests/CI-medic-guide.md Outdated

PR check jobs use the `pull-ci-` prefix instead of `periodic-ci-`.

**Overlay jobs** insert `plugin-export-overlays` after the org/repo segment:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can rework this part and explain that the part after periodic-ci- is the org/repository. Or even add a link to https://docs.ci.openshift.org/how-tos/naming-your-ci-jobs/

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant that we can rewrite the whole Job Naming Convention section, with the explanation of the name breakdown. But this is enough, too.

Keep Jira ci-fail in the medic path for overlay bugs fullsend does not
close, and describe job names as org/repo after the periodic-ci- prefix.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@zdrapela

Copy link
Copy Markdown
Member

/lgtm
/hold

Explain OpenShift CI names as job-type, org, and repo first so Core and
overlay jobs share one breakdown instead of a Core-only pattern.

Co-authored-by: Cursor <cursoragent@cursor.com>
@openshift-ci openshift-ci Bot removed the lgtm label Aug 26, 2026
@sonarqubecloud

Copy link
Copy Markdown

@zdrapela

Copy link
Copy Markdown
Member

/lgtm
/unhold

@github-actions

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants