Block the `Remote` removal if it is referred by a `Distribution`
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 62/100
Línea de trabajo
Inspecciona pulpcore/app/models/publication.py alrededor de la línea 659, donde se define la relación entre Distribution y Remote, y revisa el comportamiento de models.PROTECT de Django. Confirma el comportamiento de los modelos relacionados Remote y Distribution y, a continuación, verifica que una Remote referenciada por una Distribution no se pueda eliminar, mientras que las remotes no referenciadas sigan pudiéndose eliminar.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
If we remove a remote that is linked to a distribution, trying to pull non-synced content may fail.
Describe the solution you'd like
We could define the on_delete of Distribution remote attribute as models.Protect
https://github.com/pulp/pulpcore/blob/1672b3f405170d0a45de9c4b8cc2b059a066a162/pulpcore/app/models/publication.py#L659
as a safeguard to ensure that there is no associated Distribution before removing the Remote.
Additional context
This is related to https://github.com/pulp/pulp_container/issues/1729
- Lenguaje dominante
- Python
- Estrellas
- 606
- Forks
- 174
- Merge medio
- 16 h 44 min
- PR fusionados (30 d)
- 107
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de pulp/pulpcore
-
Reclaim Space Server ErrorAbiertoIssue
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Content app answers 404 instead of 502 when the remote of on-demand content is unavailablePosiblemente ocupada @gigi206 la tomó hace 5 días. AbiertoTriage-Needed
Dificultad 3/5 Medio día Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
pulp/pulpcore#8214 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Performance Task Triage-Needed
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
pulp/pulpcore#8206 · 3 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Issue Triage-Needed
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 1 día
Todos los issues de pulp/pulpcore
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 68/100
FavioVazquez/showtime#101 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
rossant/awesome-math#199 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
nautechsystems/nautilus_trader#5365 ·
Los mantenedores suelen responder en 1 día
-
Examples say cnxAbiertodocumentation embedded
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día