Repository navigation
docs(networking): drop HTTPCache and hold back the Cloudflare Tunnel page for 1.7 - #755
myasnikovdaniil wants to merge 2 commits into
Conversation
The HTTPCache application was removed from the catalog in Cozystack 1.7 (cozystack/cozystack#4200), so packages/apps/http-cache does not exist at v1.7.0: it is gone from release-1.7 and from v1.7.0-rc.1. The NETWORKING generator list still named it, and hack/update_apps.sh gets a 404 fetching its README and exits 1. At promotion that aborts make update-all right after release-next, so no docs PR is opened. The page and its front-matter stub leave next/ for the same reason: release-next copies next/ wholesale into v1.7/. The other mentions in next/ go with them. The ApplicationDefinition naming examples switch to VMInstance, which still shows an acronym kind with a hyphenated name. The NGINX and IP2Location cards only described what HTTPCache bundled, so they go, and HAProxy keeps only TCP Balancer. Released version trees keep their HTTPCache pages, as those releases ship it. Assisted-By: LLM Signed-off-by: Myasnikov Daniil <[email protected]>
The Cloudflare Tunnel Gateway API package (cozystack/cozystack#3858) is not merged and does not ship in Cozystack 1.7. At promotion, release-next copies next/ wholesale into v1.7/, so the page would be published as v1.7 documentation for a package that release lacks. This takes out the page and its licenses card, which came in together in 9b9965a, and the one cross-link to the page in gateway-api.md, where the edge certificate mode named the tunnel as its worked example. Revert this commit once the package ships, to restore the page, the card and the link as they were. Assisted-By: LLM Signed-off-by: Myasnikov Daniil <[email protected]>
✅ Deploy Preview for cozystack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
IvanHunters
left a comment
There was a problem hiding this comment.
LGTM.
Verified the removal is complete and build-safe on the next/ tree: no ref links to the deleted pages remain (the leftover http-cache/cloudflare-tunnel references are all in the frozen v0/v1.0–v1.2 snapshots, where the pages still ship), the deleted http-cache.md alias paths are not linked from next/, the cozystack-api CRD table is consistent after dropping httpcaches, and the application-definitions.md HTTPCache→VMInstance example migration is accurate (ToLower("VMInstance") == "vminstance" matches singular but not the vm-instance CRD name — same divergence the old example showed).
One non-blocking note:
content/en/docs/next/networking/gateway-api.md:469still reads "That is the hole the Cloudflare Tunnel package closes with its own ownership layer...". Thereflink to the page was removed from this file, but this prose mention stayed. For 1.7, where the package is intentionally held back, this asserts a mitigation that does not ship. Folding the clause removal into the second commit would keep the page accurate and keep thegit revert-when-it-ships story whole. Not a blocker.
make release-next RELEASE_TAG=v1.7.0copiesnext/intov1.7/as is, andmake update-allthen regenerates reference pages from 1.7 README files. Two things innext/break that.HTTPCache was removed from the catalog in cozystack/cozystack#4200, so
packages/apps/http-cachedoesn't exist at 1.7. TheNETWORKINGlist still names it,hack/update_apps.shgets 404 on its README and exits 1, soupdate-allstops right afterrelease-nextand no docs PR is opened. First commit drops it from the list, deletes the page and its stub and cleans remaining mentions innext/(ApplicationDefinition examples now use VMInstance, NGINX and IP2Location cards are gone because they described only HTTPCache). Older version directories keep their HTTPCache pages, those releases ship it.content/en/_index.mdstill lists HTTP caching services, I left it for a separate change.Cloudflare Tunnel page documents the package from cozystack/cozystack#3858, still a draft and not in 1.7. Second commit removes the page, its card in licenses and the one link to it from the Gateway API page. It is one commit, so
git revertof it brings everything back when the package ships.I replayed the promotion sequence on a copy of the tree (
release-next, thenupdate-allwithFETCH_REF=v1.7.0-rc.1). Onmainit fails with 404 for the http-cache README, with this branch it completes, and Hugo 0.164.0 production build of the promotedv1.7/is clean, no links to the removed pages. Preview build and i18n checks pass too.#728 edits the line right below
NETWORKING, so it conflicts with this PR in any order. Whichever lands second needs one-line fix: keep its block and drophttp-cachefromNETWORKING.