Skip to content

ci: remove sync-docs-to-site workflow (superseded by build-time docs sync) - #7658

Closed
Ma77Ball wants to merge 1 commit into
apache:mainfrom
Ma77Ball:remove-sync-docs-to-site
Closed

ci: remove sync-docs-to-site workflow (superseded by build-time docs sync)#7658
Ma77Ball wants to merge 1 commit into
apache:mainfrom
Ma77Ball:remove-sync-docs-to-site

Conversation

@Ma77Ball

Copy link
Copy Markdown
Contributor

Removes the sync-docs-to-site.yml workflow, which pushed committed copies of docs/ into apache/incubator-texera-site (content/docs/latest/) on every push to main.

Why

The website is moving to a build-time docs sync: apache/incubator-texera-site now pulls docs directly from this repo's release/vX.Y branches while building the Hugo site (see the site's scripts/pull-release-docs.sh and updated publish-site.yml). Under that model content/docs/latest/ is generated at build time and git-ignored, never committed.

Keeping this workflow would fight the new setup: it would keep committing docs into content/docs/latest/ on the site repo, which the build-time sync expects to be ignored, build-generated output. Removing it makes the build-time pull the single source of truth.

Rollout note

This should land together with the site's build-time sync going live (and with the release CI firing the docs-updated repository_dispatch the new workflow listens for), since this workflow is currently the only thing keeping the site's docs up to date. Once it's removed, the SITE_SYNC_TOKEN secret it used can be retired.

@github-actions github-actions Bot added the ci changes related to CI label Aug 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

  • No candidates found from git blame history.

@codecov-commenter

codecov-commenter commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.68%. Comparing base (4013ca7) to head (475b282).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #7658   +/-   ##
=========================================
  Coverage     89.68%   89.68%           
  Complexity     4399     4399           
=========================================
  Files          1175     1175           
  Lines         46979    46979           
  Branches       5265     5265           
=========================================
  Hits          42133    42133           
  Misses         3082     3082           
  Partials       1764     1764           
Flag Coverage Δ *Carryforward flag
access-control-service 70.00% <ø> (ø)
agent-service 98.62% <ø> (ø)
amber 86.38% <ø> (ø) Carriedforward from 4013ca7
computing-unit-managing-service 72.46% <ø> (ø)
config-service 77.31% <ø> (ø)
file-service 68.90% <ø> (ø)
frontend 90.70% <ø> (ø)
notebook-migration-service 78.89% <ø> (ø)
pyamber 97.57% <ø> (ø)
workflow-compiling-service 57.89% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Benchmark changes need a look

🟢 0 better · 🔴 2 worse · ⚪ 13 noise (<±5%) · 0 without baseline

Compared against main 4013ca7 benchmarked on this same runner, so the delta is largely free of cross-runner hardware noise. The "7d avg" column still reflects the gh-pages dashboard. Treat <±5% as noise unless repeated.

Dashboard · Run

config throughput MB/s latency max Δ latest / 7d
🔴 bs=10 sw=10 sl=64 402 0.245 23,003/32,911/32,911 us 🔴 +5.7% / 🔴 +101.3%
bs=100 sw=10 sl=64 802 0.49 123,062/145,769/145,769 us ⚪ within ±5% / 🔴 +35.5%
bs=1000 sw=10 sl=64 913 0.557 1,097,402/1,158,713/1,158,713 us ⚪ within ±5% / 🔴 +11.8%
Baseline details

Latest main 4013ca7 from same runner

config metric PR latest main 7d avg Δ latest Δ 7d
bs=10 sw=10 sl=64 throughput 402 tuples/sec 420 tuples/sec 774.89 tuples/sec -4.3% -48.1%
bs=10 sw=10 sl=64 MB/s 0.245 MB/s 0.257 MB/s 0.473 MB/s -4.7% -48.2%
bs=10 sw=10 sl=64 p50 23,003 us 23,288 us 12,738 us -1.2% +80.6%
bs=10 sw=10 sl=64 p95 32,911 us 31,136 us 16,348 us +5.7% +101.3%
bs=10 sw=10 sl=64 p99 32,911 us 31,136 us 18,848 us +5.7% +74.6%
bs=100 sw=10 sl=64 throughput 802 tuples/sec 817 tuples/sec 1,005 tuples/sec -1.8% -20.2%
bs=100 sw=10 sl=64 MB/s 0.49 MB/s 0.499 MB/s 0.613 MB/s -1.8% -20.1%
bs=100 sw=10 sl=64 p50 123,062 us 120,760 us 100,970 us +1.9% +21.9%
bs=100 sw=10 sl=64 p95 145,769 us 148,494 us 107,605 us -1.8% +35.5%
bs=100 sw=10 sl=64 p99 145,769 us 148,494 us 116,429 us -1.8% +25.2%
bs=1000 sw=10 sl=64 throughput 913 tuples/sec 910 tuples/sec 1,030 tuples/sec +0.3% -11.4%
bs=1000 sw=10 sl=64 MB/s 0.557 MB/s 0.555 MB/s 0.629 MB/s +0.4% -11.4%
bs=1000 sw=10 sl=64 p50 1,097,402 us 1,095,284 us 991,433 us +0.2% +10.7%
bs=1000 sw=10 sl=64 p95 1,158,713 us 1,135,714 us 1,036,668 us +2.0% +11.8%
bs=1000 sw=10 sl=64 p99 1,158,713 us 1,135,714 us 1,070,470 us +2.0% +8.2%
Raw CSV
config_idx,batch_size,schema_width,string_len,num_batches,total_ms,total_tuples,total_bytes,tuples_per_sec,mb_per_sec,lat_p50_us,lat_p95_us,lat_p99_us
0,10,10,64,20,497.73,200,128000,402,0.245,23003.37,32910.56,32910.56
1,100,10,64,20,2492.27,2000,1280000,802,0.490,123061.68,145768.65,145768.65
2,1000,10,64,20,21899.92,20000,12800000,913,0.557,1097402.46,1158712.84,1158712.84

@Ma77Ball

Copy link
Copy Markdown
Contributor Author

Closing as a duplicate: #6948 already removes sync-docs-to-site.yml and additionally wires the docs-updated repository_dispatch into create-release-candidate.yml. Superseded by #6948.

@Ma77Ball Ma77Ball closed this Aug 14, 2026
@Ma77Ball
Ma77Ball deleted the remove-sync-docs-to-site branch August 14, 2026 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci changes related to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants