Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Storage path prefix follow-up: deploy tidies and refuses review/./.. segments, per-app lifecycle rule and its age, pre-alpha.37 review apps (template #168, #169)

Aperta
  • Qualcosa non va?
#255 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
45/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions, php
Ambito
cloud, devops, documentation

Direzione di ricerca

Read the linked template issues #168 and #169 and docs#247, then check the template branch fix/storage-prefix-168-169 and its CHANGELOG.md for the final commits. Verify the documented GCLOUD_PATH_PREFIX rules, lifecycle prefix and age guidance, pre-alpha.37 migration steps, and stop warning path against that change; done when the docs accurately cover each behavior and the upgrade steps.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Follow-up to docs#247 (closed, documented in de3025f), from template components-web-app#168 and #169. Fix on the template branch fix/storage-prefix-168-169, landing in the next template release after 2.0.0-alpha.37; the template's CHANGELOG.md will link the final commits. Found while working on #170.

GCLOUD_PATH_PREFIX rules (variables "Storage path prefix")

  • The deploy now tidies the prefix exactly as php does (trims whitespace, squeezes doubled slashes, strips leading/trailing /, adds one trailing /), so site-a// stores review apps under site-a/review/<slug>/.
  • The deploy fails before touching the cluster on a . or .. segment, or on a whole review segment (review/, site/review): review/ is where review apps store. reviews/ or my-review/ are fine. Upgrade note: rename such a prefix first, moving its objects with it.

Lifecycle rule (ci-cd "Review app storage")

  • One rule per prefixed app: matchesPrefix: ["<GCLOUD_PATH_PREFIX>review/"] (["review/"] only with no prefix).
  • age counts from each object's creation, not from the review app's stop, so pick an age longer than review apps live, or the rule deletes a running review app's uploads.

Review apps deployed before 2.0.0-alpha.37

  • Their uploads sit at the bucket root. After their next deploy (which adds the review/<slug>/ prefix) those uploads 404 and the orphan scan no longer sees them.
  • Fix: redeploy with fresh fixtures (CWA_CI_FIXTURES_PURGE=true on the review fixture job), or stop and recreate the review app, then delete the old root objects by hand.
  • Stop review's warning (when the guarded delete fails) now prints the app's real path, e.g. site-a/review/<slug>/, and the lifecycle prefix to rely on.
Lingua principale
TypeScript
Stelle
0
Fork
0
Merge medio
1h 20m
PR unite (30g)
1

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di components-web-app/docs

Tutte le issue di components-web-app/docs

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.