Fix: Remove the possibility to add multiple git-sync resources as DAG backends
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- kubernetes, rust
調査の方向性
issue ではソースファイルやテストは指定されていません。まず Airflow の CR 定義と clusterConfig.dagsGitSync へのすべての参照を見つけてください。設定がリストではなく 1 つの git-sync リソースを受け入れ、既存の動作がプロジェクトのチェックで引き続きカバーされれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Rust
- スター
- 32
- フォーク
- 6
- 平均マージ
- 1日 7時間
- マージ済み PR（30日）
- 13
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
stackabletech/airflow-operator のほかの issue
-
type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
stackabletech/airflow-operator#855 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 42/100
stackabletech/airflow-operator#697 · コメント 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
type/feature-new
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
type/feature-new
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
stackabletech/airflow-operator#592 · コメント 1 件 ·
stackabletech/airflow-operator の issue をすべて見る
似ている issue
-
Browser (wasm) relay client cannot connect to relays whose URL has a trailing-dot FQDN hostname オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
n0-computer/iroh#4550 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
paritytech/zombienet-sdk#591 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
farion1231/cc-switch#7638 · コメント 1 件 ·
-
onnx-ir re-exports ModelProto and GraphProto but not NodeProto, AttributeProto and AttributeType オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100