Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not working with latest Airflow Helm Chart #77

Open
VfFlash opened this issue Mar 21, 2024 · 3 comments
Open

Not working with latest Airflow Helm Chart #77

VfFlash opened this issue Mar 21, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@VfFlash
Copy link

VfFlash commented Mar 21, 2024

Hi,

First of all: thank you for this great idea and work!!! Unfortunately, I couldn't manage to use it when deployed to an AKS via the latest airflow helm chart, using airflow version 2.8.3: any dag-py file (that I create with this plugin) and store under /opt/airflow/dags doesn't either show up in the dag section - or it isn't added to DagBag when using airflow dags reserialize. The logs don't reveal much more information.

Is it possible that the plugin can't be used in the aks context?

Thanks for your help!!

@VfFlash VfFlash added the bug Something isn't working label Mar 21, 2024
@VfFlash
Copy link
Author

VfFlash commented Mar 22, 2024

Sry that I labeled my question as a bug - I didn't know how else to get in contact with you.

@josedev-union
Copy link

josedev-union commented May 25, 2024

I experienced the same issue and figured out what the problem is.
This code editor stores DAG files in airflow-webserver. Airflow helm charts for airflowVersion > 2.0, it deploys airflow-scheduler as a separate pod along side the airflow-webserver, and DAGs files should be stored inside the airflow-scheduler pod. https://github.com/apache/airflow/blob/cae96630e1ddc1d3aae867c41d137e1d1fd7d465/chart/templates/webserver/webserver-deployment.yaml#L271-L283
As long as webserver and schedulerr pods are not using a shared volume for DAGs files, this code editor will not work.

As a workaround, we can create a RWX volume and mount it on both webserver and scheduler pods, at dags file path.

@josedev-union
Copy link

Here is my custom values with verion 1.13.1 chart.

dags:
  persistence:
    enabled: true
    size: 1Gi
    storageClassName: azurefile-csi
    accessMode: ReadWriteMany
webserver:
  extraVolumes:
    - name: dags
      persistentVolumeClaim:
        claimName: airflow-staging-dags
  extraVolumeMounts:
    - name: dags
      mountPath: /opt/airflow/dags

You'll need to change the claimName based on your helm release name. My helm release name is airflow-staging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants