Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Fix: Remove the possibility to add multiple git-sync resources as DAG backends

Ouverte
  • Quelque chose ne va pas ?
#760 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
45/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
kubernetes, rust
Domaine
backend, infrastructure

Piste de recherche

Aucun fichier source ni test n’est nommé dans l’issue. Commencez par localiser la définition de la CR Airflow et toutes les références à clusterConfig.dagsGitSync ; le travail est terminé lorsque la configuration accepte une ressource git-sync plutôt qu’une liste et que le comportement existant reste couvert par les vérifications du projet.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

good first issue type/feature-improvement

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.

Langage dominant
Rust
Étoiles
32
Forks
6
Merge moyen
2 j 1 h
PR mergées (30 j)
15

Préparer son environnement

  • Aucun Dockerfile ni fichier Docker Compose
  • Propose un modèle de pull request
  • Aucun guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de stackabletech/airflow-operator

Toutes les issues de stackabletech/airflow-operator

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.