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 2021/04/12 14:08:35 UTC

[GitHub] [airflow] SamWheating edited a comment on issue #15241: Testing Providers prepared 2021.04.07

SamWheating edited a comment on issue #15241:
URL: https://github.com/apache/airflow/issues/15241#issuecomment-817842664


   Hey there, I'm hoping to test my changes to the dataproc operators sometime today or tomorrow. The CNCF changes are harder to test as the issue in question only surfaces occasionally and only when running large workflows on a kubernetes cluster. In [the original PR](https://github.com/apache/airflow/pull/15137) I did a pretty thorough test, I hope this will suffice. 
   
   While setting up my local environment to test the google provider, the initial installation broke because of the introduction of `plyvel` which depends on a preexisting `leveldb` installation. 
   ```
     plyvel/_plyvel.cpp:632:10: fatal error: 'leveldb/db.h' file not found
     #include "leveldb/db.h"
              ^~~~~~~~~~~~~~
     1 error generated.
     error: command 'clang' failed with exit status 1
   ```
   
   It was easy to fix with a simple `brew install leveldb`, but I just wanted to check that you were aware of a new dependency which could break some people's builds. 


-- 
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