Skip to content

Add standalone website build workflow#7343

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

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

Conversation

@friedrichg
Copy link
Copy Markdown
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>
Comment thread .github/workflows/web.yml
uses: actions/setup-go@d35c59abb061a4a6fb18e82ac0862c26744d6ab5 # v5.5.0
with:
go-version-file: go.mod
cache: false
Copy link
Copy Markdown
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
Copy Markdown
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
Shvejan pushed a commit to Shvejan/cortex that referenced this pull request Mar 25, 2026
* Add standalone website build workflow (cortexproject#7342)

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>

* Install needed tool

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

* Testing forget to update docs

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

* Revert "Testing forget to update docs"

This reverts commit 10aa9c4.

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

---------

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
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