This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris-website.git
The following commit(s) were added to refs/heads/master by this push:
new 2b52209fe42 [fix](ci) filter retired versions from build-check
onlyIncludeVersions (#3742)
2b52209fe42 is described below
commit 2b52209fe42e3ae26b95ff474d554349f12e9ecd
Author: Mingyu Chen (Rayner) <[email protected]>
AuthorDate: Fri May 22 10:53:59 2026 -0700
[fix](ci) filter retired versions from build-check onlyIncludeVersions
(#3742)
## Summary
- `build-check.yml` previously forwarded every version segment it found
in changed file paths straight to Docusaurus' `onlyIncludeVersions`.
Touching files under retired version dirs (e.g.
`versioned_docs/version-1.2/`, `version-2.0/`) made the build fail with
`unknown versions (1.2,2.0) found`, even when those files were just
being deleted.
- This change intersects the detected versions with the active set
(`current` + entries from `versions.json`) before building.
Retired/unknown versions are logged and skipped; if everything is
retired, we fall back to a minimal `4.x` build so the workflow still
validates the site.
## Repro
PR #3731 (deletes `query_detail` doc from every version dir including
1.2 and 2.0) failed with:
```
Invalid docs option "onlyIncludeVersions": unknown versions (1.2,2.0) found.
Available version names are: current, 4.x, 3.x, 2.1
```
Failed job:
https://github.com/apache/doris-website/actions/runs/26280229035/job/77354293705?pr=3731
## Test plan
- [ ] Local simulation with PR #3731's changed-file list now produces
`DOCS_VERSIONS=current,4.x,3.x,2.1` and logs `Ignoring changes to
retired/unknown versions: 2.0,1.2`.
- [ ] CI of this PR passes (only `.github/workflows/build-check.yml`
changes; this PR itself touches no doc versions, so the workflow takes
the minimal `4.x` fallback branch — verify by reading the `=== Build
plan ===` line in the job log).
- [ ] After merge, re-run CI on #3731 and confirm it goes green without
reverting the 1.2/2.0 file deletions.
Co-authored-by: Claude Opus 4.7 (1M context) <[email protected]>
---
.github/workflows/build-check.yml | 38 +++++++++++++++++++++++++++++++++++---
1 file changed, 35 insertions(+), 3 deletions(-)
diff --git a/.github/workflows/build-check.yml
b/.github/workflows/build-check.yml
index 63e43dfd4dd..592d12d78cf 100644
--- a/.github/workflows/build-check.yml
+++ b/.github/workflows/build-check.yml
@@ -156,15 +156,47 @@ jobs:
esac
done <<< "$CHANGED_FILES"
+ # Build the set of versions Docusaurus actually builds:
+ # `current` (the dev/unreleased version) plus everything in
versions.json.
+ # Older version dirs (e.g. version-1.2, version-2.0) may still
exist on
+ # disk but be absent from versions.json — touching them must
not poison
+ # onlyIncludeVersions, which would fail the build with
"unknown versions".
+ ACTIVE_VERSIONS=$(jq -r '.[]' versions.json | tr '\n' ',' |
sed 's/,$//')
+ ACTIVE_VERSIONS="current,${ACTIVE_VERSIONS}"
+ echo "Active versions (current + versions.json):
$ACTIVE_VERSIONS"
+
# Deduplicate versions
if [ "$NEED_FULL_BUILD" = "true" ]; then
# Structural changes: build all active versions
DOCS_VERSIONS=""
echo "Structural changes detected, will build ALL
versions."
elif [ -n "$VERSIONS" ]; then
- # Only doc content changes: build only affected versions
- DOCS_VERSIONS=$(echo "$VERSIONS" | tr ',' '\n' | sort -u |
grep -v '^$' | tr '\n' ',' | sed 's/,$//')
- echo "Doc-only changes detected for versions:
$DOCS_VERSIONS"
+ # Only doc content changes: build only affected versions.
+ # Intersect detected versions with the active set so that
edits to
+ # retired version dirs (e.g. version-1.2) don't break the
build.
+ RAW_VERSIONS=$(echo "$VERSIONS" | tr ',' '\n' | sort -u |
grep -v '^$')
+ DOCS_VERSIONS=""
+ SKIPPED_VERSIONS=""
+ for v in $RAW_VERSIONS; do
+ if echo ",$ACTIVE_VERSIONS," | grep -q ",$v,"; then
+ DOCS_VERSIONS="${v},$DOCS_VERSIONS"
+ else
+ SKIPPED_VERSIONS="${v},$SKIPPED_VERSIONS"
+ fi
+ done
+ DOCS_VERSIONS=$(echo "$DOCS_VERSIONS" | sed 's/,$//')
+ SKIPPED_VERSIONS=$(echo "$SKIPPED_VERSIONS" | sed 's/,$//')
+ if [ -n "$SKIPPED_VERSIONS" ]; then
+ echo "Ignoring changes to retired/unknown versions:
$SKIPPED_VERSIONS"
+ fi
+ if [ -z "$DOCS_VERSIONS" ]; then
+ # Everything we detected was retired/unknown; fall
back to a
+ # minimal build so the workflow still validates the
site.
+ DOCS_VERSIONS="4.x"
+ echo "All detected versions were retired/unknown,
doing minimal build with '4.x' only."
+ else
+ echo "Doc-only changes detected for versions:
$DOCS_VERSIONS"
+ fi
else
# No versioned doc changes (e.g., only blog, community, or
scripts).
# Blog and community plugins are always compiled by
Docusaurus
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
