Skip to content

ci: retire dead deploy-cli-docs Netlify job - #1847

Merged
lokesh-vadlamudi merged 2 commits into
mainfrom
lvadlamudi/retire-netlify-cli-docs
Jul 17, 2026
Merged

ci: retire dead deploy-cli-docs Netlify job#1847
lokesh-vadlamudi merged 2 commits into
mainfrom
lvadlamudi/retire-netlify-cli-docs

Conversation

@lokesh-vadlamudi

@lokesh-vadlamudi lokesh-vadlamudi commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

What

Removes the deploy-cli-docs job from the Update CLI Docs workflow.

Why

That job published the CLI docs to the legacy Netlify site behind docs.lacework.net, which since the FortiCNAPP migration only 301-redirects to docs.fortinet.com/product/lacework-forticnapp. The Netlify site itself was decommissioned, so the job failed on every run:

GET https://api.netlify.com/api/v1/sites/$NETLIFY_SITE_ID/deploys
--> {"code":404,"message":"Not Found"}
jq: error (at <stdin>:1): Cannot index number with string "commit_ref"
Process completed with exit code 5

The 404 (site no longer resolves for the configured NETLIFY_SITE_ID) leaves the response as a {code,message} object, which the deploy script then fails to jq-parse. Example failing run: https://github.com/lacework/go-sdk/actions/runs/29549635816

CLI docs are now maintained on Fortinet's docs platform, so this deploy step has no valid target.

Scope

  • Only the deploy-cli-docs job is removed (+ an explanatory comment left in its place).
  • The update-cli-docs job is unchanged — it still generates the docs and commits them to lacework/docs_lw, which continues to feed the reader-redirect map (fortinet_redirects.py).

The deploy-cli-docs job published CLI docs to the legacy Netlify site behind
docs.lacework.net, which now 301-redirects to
docs.fortinet.com/product/lacework-forticnapp. The Netlify site was
decommissioned during the FortiCNAPP docs migration, so the job failed on every
run with a Netlify API 404 (then a jq parse error, exit 5). CLI docs are now
published via Fortinet's docs pipeline.
@lokesh-vadlamudi lokesh-vadlamudi self-assigned this Jul 17, 2026
@lokesh-vadlamudi
lokesh-vadlamudi requested a review from wsli-1 July 17, 2026 19:45
@lokesh-vadlamudi
lokesh-vadlamudi marked this pull request as ready for review July 17, 2026 19:45
@lokesh-vadlamudi
lokesh-vadlamudi requested a review from a team as a code owner July 17, 2026 19:45
@lokesh-vadlamudi
lokesh-vadlamudi enabled auto-merge (squash) July 17, 2026 19:45
@lokesh-vadlamudi
lokesh-vadlamudi merged commit e9b5111 into main Jul 17, 2026
19 of 20 checks passed
@lokesh-vadlamudi
lokesh-vadlamudi deleted the lvadlamudi/retire-netlify-cli-docs branch July 17, 2026 21:18
@lacework-releng lacework-releng mentioned this pull request Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants