Skip to content

ci(docs): exclude flaky istio.io from lychee link check - #483

Merged
Destynova2 merged 1 commit into
mainfrom
fix/lychee-exclude-istio
Jul 28, 2026
Merged

ci(docs): exclude flaky istio.io from lychee link check#483
Destynova2 merged 1 commit into
mainfrom
fix/lychee-exclude-istio

Conversation

@Destynova2

Copy link
Copy Markdown
Contributor

What

The lychee (broken-link) gate failed on the v0.36.84 release PR (#482):

### Errors in docs/decisions/0022-declarative-endpoints-policies-schema.md
* [ERROR] https://istio.io/latest/docs/reference/config/security/authorization-policy/
  Connection failed. Check network connectivity and firewall settings

The link is not broken — it returns 200 from a normal client (verified:
0.6s, HTTP 200). istio.io intermittently refuses connections from GitHub runner
IPs; the same URL timed out earlier this release cycle too.

Fix

Added the host to .lycheeignore, matching the existing precedent for
public.cyber.mil (excluded for the identical "flaky from runners, not broken"
reason). The link stays in the doc; only the runner-side flakiness stops blocking
merges.

It was non-blocking for v0.36.84 (lychee isn't a Required check, so the release
went through), but it turns the docs-lint workflow red on every run that
re-checks that URL.

🤖 Generated with Claude Code

`https://istio.io/latest/docs/reference/config/security/authorization-policy/`
(cited in ADR-0022) intermittently refuses connections from GitHub runners —
it returns 200 from a normal client (verified: 0.6s, HTTP 200). It failed the
lychee gate on the v0.36.84 release PR (#482) and timed out earlier this cycle.

Excluded the host, matching the existing precedent for public.cyber.mil. The
link is valid; this only stops a live external host's runner-side flakiness from
blocking merges.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Destynova2
Destynova2 enabled auto-merge (squash) July 28, 2026 06:31
@Destynova2
Destynova2 merged commit 5ee0cc4 into main Jul 28, 2026
28 checks passed
@Destynova2
Destynova2 deleted the fix/lychee-exclude-istio branch July 28, 2026 06:35
Destynova2 added a commit that referenced this pull request Jul 28, 2026
## 🤖 New release

* `grob`: 0.36.84 -> 0.36.85

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.36.85](v0.36.84...v0.36.85)
- 2026-07-28

### Fixed

- *(openai)* preserve tool-result images on the Responses (Codex) path
([#488](#488))

### Other

- *(docs)* exclude flaky istio.io from lychee link check
([#483](#483))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).
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