Skip to content

Add to comment to clarify kubernetespodoperator - #2471

Merged
leahecole merged 2 commits into
masterfrom
kpo_docker_fix
Oct 31, 2019
Merged

Add to comment to clarify kubernetespodoperator#2471
leahecole merged 2 commits into
masterfrom
kpo_docker_fix

Conversation

@leahecole

Copy link
Copy Markdown
Collaborator

Ping me for internal bug reference

@leahecole
leahecole requested a review from tswastOctober 10, 2019 22:36
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 10, 2019
@leahecoleleahecole 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
Copy Markdown
CollaboratorAuthor

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

@tswast

Copy link
Copy Markdown
Contributor

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

@tswast

Copy link
Copy Markdown
Contributor

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

@tswasttswast mentioned this pull request Oct 16, 2019
@leahecoleleahecole 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 masterOct 31, 2019
@leahecole
leahecole deleted the kpo_docker_fix branch October 31, 2019 20:26
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@leahecole@tswast@googlebot