-
Notifications
You must be signed in to change notification settings - Fork 542
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
Using Official Airflow Image apache/airflow:1.10.11 #576
base: master
Are you sure you want to change the base?
Conversation
I would recommend people use this going forward. |
The custom airflow.cfg doesnt seem to work, the only way i could get airflow configs changes to propagate was via doing it the Environment Variable. |
I agree, @swapniel99, do you feel like maintaining a new fork repo based on your work? |
@MeTaNoV Sure. I have also made some enhancements in a private repo to make it as compatible to puckel's format as possible. I can publish them. |
@swapniel99 that would be gr8, yes! Thanks for your contribution! |
@swapniel99 / @MeTaNoV I am happy to help contribute to your repo going forward. Can you let us know here when it is up to date and published? I currently have yours forked and am adding my own improvements well. |
I had not changed ownership of airflow home. probably that's why cfg didnt work. |
@swapniel99 I checked your repo. You're not using AIRFLOW_DEPS and PYTHON_DEPS anymore. Any reason ? |
Must have missed it since I was not using it in my vanilla image of private repo. Please feel free to contribute. |
everything worked great with the upgrade except the root user doesnt work as it did and is crucial to my solution. If you launch the containers with root user and - C_FORCE_ROOT=true as env (like it used to) containers fail, airflow is not found. Any ideas? |
Will that be merged anytime soon? |
No description provided.