You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/06/25 19:22:18 UTC

[GitHub] [airflow] potiuk commented on pull request #9468: Switched to Helm chart for Kubernetes tests

potiuk commented on pull request #9468:
URL: https://github.com/apache/airflow/pull/9468#issuecomment-649770813


   Hey @dimberman @ashb @kaxil  -> I managed to get all the kubernetes tests working using the helm chart :) (with lots of @dimberman's help) . I got rid of all the "templates" from the old tests. 
   
   The problems that prevented the tests to run turned to be simple: missing custom "resource + resource claim" for two of the tests and the fact that it takes a few seconds for the webserver to start and I had to add a loop to test if it is up when I setup port-forwarding. That seems to fix all the problems - and we can start adding more tests for kubernetes installation now.
   
   I also added more logs for the tests and breeze's "kind-cluster shell' command that drops you to shell where you can run kubernetes tests...
   
   Looking forward to merging it (I will cherry-pick to 1.10 right after)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org