Skip to content

fix(ci): ignore prow.ci.openshift.org in markdown link check - #5409

Merged
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:mainfrom
jonkoops:fix/markdown-link-check-prow
Sep 15, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
redhat-developer:mainfrom
jonkoops:fix/markdown-link-check-prow

Conversation

@jonkoops

Copy link
Copy Markdown
Contributor

Prow now returns 403 to unauthenticated HTTP clients (OAuth proxy in front of prow.ci.openshift.org), so markdown-link-check fails on nine links in docs/e2e-tests/CI-medic-guide.md and docs/e2e-tests/CI.md. The URLs are still valid for humans with access; they are not dead links.

This adds prow.ci.openshift.org to ignorePatterns in .github/workflows/link-checker.config.json, same approach as #5383 used for rate-limited Red Hat doc hosts.

Unblocks PRs that touch any markdown under docs/, including #5405.

Prow now returns 403 to unauthenticated HTTP clients, so the link checker
reports nine valid docs links as dead. Add the host to ignorePatterns, same
as docs.redhat.com in redhat-developer#5383.
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.69%. Comparing base (822539e) to head (66ccc28).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5409      +/-   ##
==========================================
- Coverage   63.68%   59.69%   -3.99%     
==========================================
  Files         121      109      -12     
  Lines        2288     2062     -226     
  Branches      527      501      -26     
==========================================
- Hits         1457     1231     -226     
  Misses        830      830              
  Partials        1        1              
Flag Coverage Δ
rhdh 59.69% <ø> (-3.99%) ⬇️
Components Coverage Δ
Backend plugins ∅ <ø> (∅)
Backend app 69.69% <ø> (ø)
Frontend app 58.82% <ø> (ø)
Plugin 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 822539e...66ccc28. 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.

@openshift-ci openshift-ci Bot added the lgtm label Sep 15, 2026
@jonkoops jonkoops changed the title fix(ci): ignore prow.ci.openshift.org in markdown link check fix(ci): ignore prow.ci.openshift.org in markdown link check Sep 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

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

@openshift-merge-bot
openshift-merge-bot Bot merged commit 93a56d3 into redhat-developer:main Sep 15, 2026
24 checks passed
jonkoops added a commit to jonkoops/rhdh that referenced this pull request Sep 15, 2026
…developer#5409)

Prow now returns 403 to unauthenticated HTTP clients, so the link checker
reports nine valid docs links as dead. Add the host to ignorePatterns, same
as docs.redhat.com in redhat-developer#5383.
jonkoops pushed a commit that referenced this pull request Sep 16, 2026
)

* Revert "fix(ci): ignore prow.ci.openshift.org in markdown link check (#5409)"

This reverts commit 93a56d3.

* docs: clarify Prow URL instructions

Assisted-by: OpenCode

* ci: ignore private Google Doc in link check

Assisted-by: OpenCode

* ci: ignore Google Docs in link check

Assisted-by: OpenCode
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