-
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
bump airflow version to 1.10.12 #623
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
When this is getting merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is necessary.
Why don't they merge this?
I just switched to apache airflow official image. It seems like no one
maintain this anymore.
…On Sat, Apr 10, 2021, 1:14 AM Hyo-kyun Park ***@***.***> wrote:
***@***.**** approved this pull request.
I think this is necessary.
Why doesn't they merge this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#623 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC65XLTJLIH34XIULF3OSTTTH7NC5ANCNFSM4V5D3Y2Q>
.
|
@MJFND thx for your reply. 👍 The (apache airflow) official images seem to support only airflow version 2.0.1. Do you have any better solution for me? :D |
I am actually using 1.10.14, they do have the image, but they will not
update that anymore.
And it will work fine.
…On Sat, Apr 10, 2021, 1:44 AM Hyo-kyun Park ***@***.***> wrote:
I just switched to apache airflow official image. It seems like no one
maintain this anymore.
@MJFND <https://github.com/MJFND> thx for your reply. 👍
The (apache airflow) official images seem to support only airflow version
2.0.1.
But I need airflow version 1.10.12 because I want the image which is same
as AWS managed airflow
<https://aws.amazon.com/ko/blogs/korea/introducing-amazon-managed-workflows-for-apache-airflow-mwaa/>
.
Do you have any better solution for me? :D
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#623 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC65XLQCKQTRBJFRMCQ26B3TH7QR7ANCNFSM4V5D3Y2Q>
.
|
@MJFND You're right. I can find 1.10.x at dockerhub. How did you run 1.10.14 with CeleryExecutor (which depends on postgres, redis)? |
I used the compose config from here
https://airflow.apache.org/docs/apache-airflow/stable/start/docker.html.
It is for the latest image, with little changes it will work.
…On Sat, Apr 10, 2021, 4:22 AM Hyo-kyun Park ***@***.***> wrote:
@MJFND <https://github.com/MJFND> You're right. I can find 1.10.x at
dockerhub.
But there is no docker-compose.yml in the official website -
https://airflow.apache.org/docs/apache-airflow/stable/start/docker.html
Even I could not find Dockerfile of 1.10.x. (from
https://github.com/apache/airflow)
How did you run 1.10.14 with CeleryExecutor (which depends on postgres,
redis)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#623 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC65XLWJL5BQZDN67NH2XPLTIADEBANCNFSM4V5D3Y2Q>
.
|
please check and merge