You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@airflow.apache.org by Michael Crawford <mi...@modernizingmedicine.com> on 2017/09/24 17:34:12 UTC

Build failing on master

Hi,

It looks like the following pull request didn’t actually get reverted:
https://github.com/apache/incubator-airflow/pull/2411 <https://github.com/apache/incubator-airflow/pull/2411>

All travis builds are failing now because of the docker operator change.

Thanks,
Mike



Re: Build failing on master

Posted by Michael Crawford <mi...@modernizingmedicine.com>.
Thanks,   I put in a pull request for fixing the readme and my previous pull is rerunning.   
(Had to modify my commit a little to get it to rerun travis)

Thanks,
Mike


> On Sep 24, 2017, at 1:40 PM, Bolke de Bruin <bd...@gmail.com> wrote:
> 
> Master, now it is tracking all branches and assumes the latest build is the one to report on. So yes we need to fix that.
> 
>> On 24 Sep 2017, at 19:39, Michael Crawford <mi...@modernizingmedicine.com> wrote:
>> 
>> Great thanks.
>> 
>> Also it looks like we may be linking the wrong thing in the readme.   
>> 
>> We are linking https://api.travis-ci.org/apache/incubator-airflow.svg <https://api.travis-ci.org/apache/incubator-airflow.svg> which shows passing
>> 
>> But https://api.travis-ci.org/apache/incubator-airflow.svg?branch=master <https://api.travis-ci.org/apache/incubator-airflow.svg?branch=master> shows failing?  
>> 
>> What is that build status in the readme supposed to represent? 
>> 
>> 
>>> On Sep 24, 2017, at 1:36 PM, Bolke de Bruin <bd...@gmail.com> wrote:
>>> 
>>> Fixed that.
>>> 
>>> Bolke
>>> 
>>>> On 24 Sep 2017, at 19:34, Michael Crawford <mi...@modernizingmedicine.com> wrote:
>>>> 
>>>> Hi,
>>>> 
>>>> It looks like the following pull request didn’t actually get reverted:
>>>> https://github.com/apache/incubator-airflow/pull/2411 <https://github.com/apache/incubator-airflow/pull/2411>
>>>> 
>>>> All travis builds are failing now because of the docker operator change.
>>>> 
>>>> Thanks,
>>>> Mike
>>>> 
>>>> 
>>> 
>> 
> 


Re: Build failing on master

Posted by Bolke de Bruin <bd...@gmail.com>.
Master, now it is tracking all branches and assumes the latest build is the one to report on. So yes we need to fix that.

> On 24 Sep 2017, at 19:39, Michael Crawford <mi...@modernizingmedicine.com> wrote:
> 
> Great thanks.
> 
> Also it looks like we may be linking the wrong thing in the readme.   
> 
> We are linking https://api.travis-ci.org/apache/incubator-airflow.svg <https://api.travis-ci.org/apache/incubator-airflow.svg> which shows passing
> 
> But https://api.travis-ci.org/apache/incubator-airflow.svg?branch=master <https://api.travis-ci.org/apache/incubator-airflow.svg?branch=master> shows failing?  
> 
> What is that build status in the readme supposed to represent? 
> 
> 
>> On Sep 24, 2017, at 1:36 PM, Bolke de Bruin <bd...@gmail.com> wrote:
>> 
>> Fixed that.
>> 
>> Bolke
>> 
>>> On 24 Sep 2017, at 19:34, Michael Crawford <mi...@modernizingmedicine.com> wrote:
>>> 
>>> Hi,
>>> 
>>> It looks like the following pull request didn’t actually get reverted:
>>> https://github.com/apache/incubator-airflow/pull/2411 <https://github.com/apache/incubator-airflow/pull/2411>
>>> 
>>> All travis builds are failing now because of the docker operator change.
>>> 
>>> Thanks,
>>> Mike
>>> 
>>> 
>> 
> 


Re: Build failing on master

Posted by Michael Crawford <mi...@modernizingmedicine.com>.
Great thanks.

Also it looks like we may be linking the wrong thing in the readme.   

We are linking https://api.travis-ci.org/apache/incubator-airflow.svg <https://api.travis-ci.org/apache/incubator-airflow.svg> which shows passing

But https://api.travis-ci.org/apache/incubator-airflow.svg?branch=master <https://api.travis-ci.org/apache/incubator-airflow.svg?branch=master> shows failing?  

What is that build status in the readme supposed to represent? 


> On Sep 24, 2017, at 1:36 PM, Bolke de Bruin <bd...@gmail.com> wrote:
> 
> Fixed that.
> 
> Bolke
> 
>> On 24 Sep 2017, at 19:34, Michael Crawford <mi...@modernizingmedicine.com> wrote:
>> 
>> Hi,
>> 
>> It looks like the following pull request didn’t actually get reverted:
>> https://github.com/apache/incubator-airflow/pull/2411 <https://github.com/apache/incubator-airflow/pull/2411>
>> 
>> All travis builds are failing now because of the docker operator change.
>> 
>> Thanks,
>> Mike
>> 
>> 
> 


Re: Build failing on master

Posted by Bolke de Bruin <bd...@gmail.com>.
Fixed that.

Bolke

> On 24 Sep 2017, at 19:34, Michael Crawford <mi...@modernizingmedicine.com> wrote:
> 
> Hi,
> 
> It looks like the following pull request didn’t actually get reverted:
> https://github.com/apache/incubator-airflow/pull/2411 <https://github.com/apache/incubator-airflow/pull/2411>
> 
> All travis builds are failing now because of the docker operator change.
> 
> Thanks,
> Mike
> 
>