kube-airflow icon indicating copy to clipboard operation
kube-airflow copied to clipboard

Provide a way to deploy new dags from github

Open Sedward opened this issue 7 years ago • 7 comments

It had previously been discussed on slack that we could use the the git-sync sidecar for updating dags from github. The environment variables have already been added. We could add the git-sync sidecar to the scheduler, worker and web deployments. If this is done, do we make it an option in the values.yaml that has to be enabled?

Sedward avatar Nov 09 '17 22:11 Sedward

it is already done, what is the difference with the current git-sync script and settings in values.yaml?

gsemet avatar Nov 09 '17 23:11 gsemet

How does it currently work? I haven't used git-sync yet but I was expecting to see a sidecar in the deployments that use it like the example in their repo.

https://github.com/kubernetes/git-sync/blob/master/demo/config/pod.yaml#L9

Sedward avatar Nov 10 '17 17:11 Sedward

Read the read me. It is a gitsync launched in the container. Using a sidecar means sharing a folder. Which is also a valid use case

gsemet avatar Nov 10 '17 17:11 gsemet

OK I see it now. Is there a benefit to using this script inside the container vs the sidecar container from the kubernetes org? I thought it was somewhat of an anti-pattern to run more than one process in a container.

Sedward avatar Nov 10 '17 19:11 Sedward

On we need to ensure a share path. If you adapt the conf with an optional sidecar gitsync it can be good to have.

gsemet avatar Nov 11 '17 08:11 gsemet

I see the value git_sync_enabled but i can't see where it is used in the charts. which makes me think this chart only support embedded dags via docker image?

Jonathan34 avatar Mar 27 '18 20:03 Jonathan34

yes, it does not work anymore, I did not took the time to restore it, and it has many issues that need to be fixed

gsemet avatar Mar 27 '18 20:03 gsemet