Skip to content

WIP: [DO NOT MERGE] fix docs site publishing #737

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jameslamb wants to merge 4 commits into from
Closed

WIP: [DO NOT MERGE] fix docs site publishing #737

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d7afdbe
fix docs site publishing
jameslamb Jan 16, 2026
a190c06
skip S3 stuff for now
jameslamb Jan 16, 2026
2301332
empty commit to re-trigger CI
jameslamb Jan 16, 2026
5e9f9f9
another empty commit (test new token)
jameslamb Jan 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 8 additions & 9 deletions .github/workflows/deploy.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ on:
push:
branches:
- main
- fix-publishing
Copy link
Member Author

@jameslamb jameslamb Jan 16, 2026

Choose a reason for hiding this comment

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

Publishing from this branch should be fine. I'm not actually changing the docs, and the site is broken anyway!


concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
Expand Down Expand Up @@ -42,11 +43,11 @@ jobs:
aws-region: ${{ vars.AWS_REGION }}
role-duration-seconds: 7200 # 2h

- name: Fetch doc files from S3
run: ci/download_from_s3.sh
# - name: Fetch doc files from S3
# run: ci/download_from_s3.sh

- name: Post-process docs
run: ci/post-process.sh
# - name: Post-process docs
# run: ci/post-process.sh

- name: Deploy site
env:
Expand All @@ -57,9 +58,7 @@ jobs:
run: |
npm install --global --force @aschmidt8/netlify-cli

ARGS=""
if [ "$GITHUB_REF_NAME" = "main" ]; then
ARGS="--prod"
fi
netlify deploy "$ARGS" \
netlify deploy \
Copy link
Member Author

@jameslamb jameslamb Jan 16, 2026

Choose a reason for hiding this comment

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

Looked for the available flags like this:

docker run \
    --rm \
    -it node:lts-alpine3.23 \
    sh

npm install --global --force @aschmidt8/netlify-cli

netlify deploy --help

Hopefully getting more logs will help us here.

Copy link
Member Author

@jameslamb jameslamb Jan 16, 2026

Choose a reason for hiding this comment

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

Seeing some interesting things in the debug logs (logs link).

siteId: ***
 ›   Error: Failed retrieving addons for site ***: Not Found. Double-check your login status with 'netlify status' or contact support with details of your error.
 ›   Warning: Failed to resolve config, falling back to offline resolution

Copy link
Member Author

@jameslamb jameslamb Jan 16, 2026

Choose a reason for hiding this comment

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

Next I looked into the configuration for the site:

docs/.github/workflows/deploy.yaml

Lines 53 to 54 in 31497b6

NETLIFY_AUTH_TOKEN: ${{ secrets.NETLIFY_API_TOKEN }}
NETLIFY_SITE_ID: ${{ secrets.NETLIFY_DOCS_SITE_ID }}

Updated NETLIFY_SITE_ID to the value I see in the project configuration (https://app.netlify.com/projects/docs-rapids-ai/configuration/general), that did not fix anything.

Then updated NETLIFY_AUTH_TOKEN to a newly-generated token and that worked!

https://github.com/rapidsai/docs/actions/runs/21077092892/job/60621594737

Confirmed in this new build on main: https://github.com/rapidsai/docs/actions/runs/21077222946

--prod \
--debug \
--dir=_site