Skip to content

Add standalone website build workflow#7343

Merged
friedrichg merged 4 commits intomasterfrom
decouple-website-build
Mar 10, 2026
Merged

Add standalone website build workflow#7343
friedrichg merged 4 commits intomasterfrom
decouple-website-build

Conversation

@friedrichg
Copy link
Member

What this PR does:

Phase 1 of #7342: Adds a standalone GitHub Actions workflow (.github/workflows/web.yml) that builds the documentation website without depending on the build-image container.

Why:

The build-image takes ~80 minutes to rebuild (#7339). Website tooling (Hugo, Node.js, PostCSS) is bundled in the image, so any website dependency change requires a full rebuild. Additionally, website/package-lock.json was not used by any build process — the build image installs npm packages globally.

What changed:

  • .github/workflows/web.yml — new workflow that installs Go, Node.js, and Hugo via GitHub Actions, runs npm ci with the lockfile, then builds the website
  • website/package-lock.json — regenerated so npm ci works
  • The deploy job exists but is disabled (if: false && ...) to avoid racing with the existing deploy in test-build-deploy.yml
  • The existing website build in test-build-deploy.yml is unchanged — both run in parallel for comparison during Phase 1

Next steps (Phase 2):

  1. Verify the new workflow produces correct output
  2. Enable the deploy job in web.yml, remove website build + deploy from test-build-deploy.yml
  3. Remove Node.js/Hugo from build-image/Dockerfile (Phase 3)

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated

@dosubot dosubot bot added the ci/cd label Mar 10, 2026
@friedrichg friedrichg force-pushed the decouple-website-build branch from 4fb0c0f to 440cee5 Compare March 10, 2026 01:35
Extract website build into a separate GitHub Actions workflow that
does not depend on the build-image container. This is Phase 1:
both the new workflow and the existing build job produce website
artifacts in parallel for comparison.

The deploy job is disabled (false condition) until Phase 2 when
we remove the website build from test-build-deploy.yml.

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg force-pushed the decouple-website-build branch from 440cee5 to f9c53b4 Compare March 10, 2026 02:04
@pull-request-size pull-request-size bot added size/L and removed size/M labels Mar 10, 2026
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg force-pushed the decouple-website-build branch from dd922cd to 6ad4323 Compare March 10, 2026 02:11
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
This reverts commit 10aa9c4.

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
uses: actions/setup-go@d35c59abb061a4a6fb18e82ac0862c26744d6ab5 # v5.5.0
with:
go-version-file: go.mod
cache: false
Copy link
Member

Choose a reason for hiding this comment

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

Should we keep disabling the cache?

Copy link
Member Author

Choose a reason for hiding this comment

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

it's fast for now. Let's keep it like this for now

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 10, 2026
@friedrichg friedrichg merged commit 982bd7a into master Mar 10, 2026
64 of 67 checks passed
@friedrichg friedrichg deleted the decouple-website-build branch March 10, 2026 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd lgtm This PR has been approved by a maintainer size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants