Skip to content
This repository was archived by the owner on Apr 22, 2020. It is now read-only.

Conversation

@gerardo
Copy link

@gerardo gerardo commented Oct 8, 2018

This is required by apache/airflow#3815 and apache/airflow#3816.

I decided to keep both Dockerfiles in the same branch, and build them both through Dockerhub instead. This is my current configuration for https://hub.docker.com/r/gerardo/incubator-airflow-ci/:

screen shot 2018-10-08 at 7 44 09 pm

@gerardo
Copy link
Author

gerardo commented Oct 8, 2018

@Fokko @tedmiston @kaxil please have a look.

@Fokko
Copy link
Contributor

Fokko commented Oct 12, 2018

@gerardo Awesome, this was exactly what I had in mind!

@Fokko
Copy link
Contributor

Fokko commented Oct 12, 2018

We can't merge this (yet): apache/airflow#3815 (comment)

@kaxil
Copy link
Member

kaxil commented Jan 5, 2019

Any status on this PR @gerardo @Fokko

@Fokko
Copy link
Contributor

Fokko commented Jan 7, 2019

This is something that we still want, but I don't have the time to look into this because this will probably will break the master build.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants