Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de 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)

Abierto
  • ¿Algo no está bien?
#255 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
45/100
Tipo de issue
Documentación
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github-actions, php
Área
cloud, devops, documentation

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.
Lenguaje dominante
Vue
Estrellas
0
Forks
0
Merge medio
1 h 20 min
PR fusionados (30 d)
1

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de components-web-app/docs

Todos los issues de components-web-app/docs

Issues similares

Más issues de Cloud

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.