Skip to content

Conversation

@leahecole
Copy link
Collaborator

Ping me for internal bug reference

@leahecole leahecole requested a review from tswast October 10, 2019 22:36
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 10, 2019
@leahecole leahecole added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 10, 2019
@leahecole
Copy link
Collaborator Author

Oof I'll figure out this failure sometime next week maybe if I have time. Low priority, @tswast

@tswast
Copy link
Contributor

tswast commented Oct 11, 2019

Oof. Test failure is a strange one. Sounds like a problem with installing apache-airflow[gcp_api]==1.10.3 https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/composer/workflows/requirements.txt

@tswast
Copy link
Contributor

tswast commented Oct 11, 2019

Sounds like we need to pin the Flask or maybe just update to latest patch release of Airflow. puckel/docker-airflow#395

@leahecole leahecole removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 31, 2019
@leahecole leahecole merged commit 94b3f55 into master Oct 31, 2019
@leahecole leahecole deleted the kpo_docker_fix branch October 31, 2019 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants