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

Offen

#760 erstellt am 12.3.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Refactoring
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: kubernetes, rust
Bereich: backend, infrastructure

Rechercherichtung

In der Issue werden keine Quelldateien oder Tests genannt. Beginne damit, die Airflow-CR-Definition und alle Verweise auf clusterConfig.dagsGitSync zu finden; abgeschlossen ist die Aufgabe, wenn die Konfiguration eine git-sync-Ressource statt einer Liste akzeptiert und das bestehende Verhalten weiterhin durch die Checks des Projekts abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache: Rust
Sterne: 32
Forks: 6
Ø Merge: 1 T. 7 Std.
Gemergte PRs (30 T.): 13

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus stackabletech/airflow-operator

airflow scheduler default 0 - CrashLoopBackOff Offen

type/bug

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100

stackabletech/airflow-operator#855 · 1 Kommentar · Erstellt am 07.9.2026
Include HDFS provider in Airflow images Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 42/100

stackabletech/airflow-operator#697 · 1 Kommentar · Erstellt am 08.10.2025
Extend metrics collection with either Flower or OpenTelemetry Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

stackabletech/airflow-operator#638 · Erstellt am 02.6.2025
Integrate DAG editor in Airflow Offen

type/feature-new

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

stackabletech/airflow-operator#593 · Erstellt am 07.3.2025
Allow to install custom python libraries Offen

type/feature-new

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

stackabletech/airflow-operator#592 · 1 Kommentar · Erstellt am 07.3.2025

Alle Issues in stackabletech/airflow-operator