-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fix Airflow-Ext Dependency #15
Conversation
Ran into this in #16, and I tried your fix but it didnt seem to make much of a difference
|
Thanks @keefertaylor! I can't test this right now, but if you (or someone else, cc @johntdyer) can get this to work with a specific tag on the git+https://github.com/meltano/airflow-ext repo I'll gladly merge it! |
@DouweM @keefertaylor @johntdyer confirmed just now that the fix works, at least when building locally, as does any commit of airflow-ext from ~mid-2023. FWIW, I was able to install Docker Compose and clone the Pixbyt repo on an AWS Cloud9 instance and run everything completely from the cloud. |
@bboatwright Looks like we were both looking into this at the same time! I just published #18 which fixes this issue by using the latest version of Airflow! |
The airflow-ext package is specified to be
main
, which is a branch that has changed sinceplugins.meltano.yml
was authored. When I build today, I get constraints errors:To Reproduce:
To fix, I've simply pinned
plugins.meltano.yml
to the commit onairflow-ext
'smain
branch at the timeplugins.meltano.yml
was last modified.I note that the
airflow-ext
project has started to version their library with tags and releases, and using one of them is probably more appropriate. I'm unfortunately unable to figure out the best way to do this since I'm not very familiar with any of these packages, or python/pip in general.