Fix: Remove the possibility to add multiple git-sync resources as DAG backends
Los mantenedores suelen responder en 1 día
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
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- kubernetes, rust
- Área
- backend, infrastructure
Línea de trabajo
En el issue no se nombran archivos fuente ni pruebas. Empieza por localizar la definición de CR de Airflow y todas las referencias a clusterConfig.dagsGitSync; se considera terminado cuando la configuración acepta un recurso git-sync en lugar de una lista y el comportamiento existente sigue estando cubierto por las comprobaciones del proyecto.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Context
We have tried to implement multiple git-sync resources into one DAGS_CORE_FOLDER which was successful.
However when the feature was completed we noticed problems with this approach like the local dag path bleeds into DAG logic. After investigating we found reasons to rather not do it: https://github.com/stackabletech/airflow-operator/issues/721#issuecomment-4038282638
This now leads to the decision to not expose a possibility to add multiple dags in the Airflow CR.
What needs to happen
We want to remove the possibility to
clusterConfig:
credentialsSecret: test-airflow-credentials
dagsGitSync:
- repo: https://github.com/a-pretty-repo
branch: feat/beautiful-branch
gitFolder: "a-passably-folder"
- repo: https://github.com/a-grave-repo
credentialsSecret: git-credentials
gitSyncConf:
# supply some config to check that safe.directory is correctly set
--git-config: http.sslVerify:false
gitFolder: "a-well-formed-folder"
and rather have something like:
clusterConfig:
credentialsSecret: test-airflow-credentials
dagsGitSync:
repo: https://github.com/the-master-repo
branch: feat/mighty-branch
gitFolder: "all-my-magic"
as leaving this option open suggest a incomplete feature and might raise expectation on us to implement it.
- Lenguaje dominante
- Rust
- Estrellas
- 32
- Forks
- 6
- Merge medio
- 21 h 38 min
- PR fusionados (30 d)
- 13
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin 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 stackabletech/airflow-operator
-
type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
stackabletech/airflow-operator#855 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 42/100
stackabletech/airflow-operator#697 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
stackabletech/airflow-operator#638 ·
Los mantenedores suelen responder en 1 día
-
Integrate DAG editor in AirflowAbiertotype/feature-new
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
stackabletech/airflow-operator#593 ·
Los mantenedores suelen responder en 1 día
-
type/feature-new
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
stackabletech/airflow-operator#592 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de stackabletech/airflow-operator
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
dani-garcia/vaultwarden#7801 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
boxlite-ai/boxlite#1814 ·
Los mantenedores suelen responder en 1 día
-
tech-debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
xingkongliang/skills-manager#516 ·
Los mantenedores suelen responder en 5 días