airflow-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kaxil Naik <kaxiln...@gmail.com>
Subject Re: Travis CI reporting incorrectly
Date Sat, 05 Jan 2019 15:52:56 GMT
It is fixed now. I will merge that PR (
https://github.com/apache/airflow/pull/4441) once the CI build finishes.

It is now reporting error correctly. Can someone approve the PR?



On Sat, Jan 5, 2019 at 3:38 PM Naik Kaxil <k.naik@reply.com> wrote:

> I have that in my PR but it still failed. I made another change and
> triggered the CI, let's see
>
>
> Kaxil Naik
>
> Data Reply
> Nova South
> 160 Victoria Street, Westminster
> London SW1E 5LB - UK
> phone: +44 (0)20 7730 6000
> k.naik@reply.com
> www.reply.com
>
> [image: Data Reply]
> ------------------------------
> *From:* Deng Xiaodong <xd.deng.r@gmail.com>
> *Sent:* Saturday, January 5, 2019 3:20:57 PM
> *To:* dev@airflow.apache.org
> *Cc:* Tao Feng; Driesprong, Fokko
> *Subject:* Re: Travis CI reporting incorrectly
>
> Hi Kaxil,
>
> Not sure if you guys have found the root-cause. If not, I suspect it’s
> because of this line “sudo ln -s ${ROOTDIR}
> /home/travis/build/apache/incubator-airflow” in
> https://github.com/apache/airflow/blob/master/scripts/ci/5-run-tests.sh#L97
> <https://github.com/apache/airflow/blob/master/scripts/ci/5-run-tests.sh>.
> It was not updated accordingly in PR 4419.
>
> Please check.
>
> Thanks.
>
>
> XD
>
> > On 5 Jan 2019, at 11:06 PM, Kaxil Naik <kaxilnaik@gmail.com> wrote:
> >
> > An Example is the latest build:
> > https://travis-ci.org/apache/airflow/jobs/475704671#L7363
> >
> > It had "8 FAILED, 2 errors, 40 skipped (2072 tests passed)"
> >
> > But still the test show as successful.
> >
> > On Sat, Jan 5, 2019 at 1:18 PM Kaxil Naik <kaxilnaik@gmail.com> wrote:
> >
> >> Sorry - I meant I think it *is* because of the renaming of the repo.
> >>
> >> On Sat, Jan 5, 2019 at 1:14 PM Kaxil Naik <kaxilnaik@gmail.com> wrote:
> >>
> >>> Hi guys,
> >>>
> >>> Travis CI is reporting the build status incorrectly. Few of the tests
> >>> have failed but it still shows the overall test as passed.
> >>>
> >>> Request to all the committers to not merge any PR before going to
> Travis
> >>> and checking.
> >>>
> >>> PS: I don't think it has anything to do with renaming for
> >>> incubator-airflow to airflow
> >>>
> >>> Regards,
> >>>
> >>> *Kaxil Naik*
> >>>
> >>
> >>
> >> --
> >> *Kaxil Naik*
> >> *Big Data Consultant *@ *Data Reply UK*
> >> *Certified *Google Cloud Data Engineer | *Certified* Apache Spark &
> Neo4j
> >> Developer
> >> *Phone: *+44 (0) 74820 88992
> >> *LinkedIn*: https://www.linkedin.com/in/kaxil
> >>
> >
> >
> > --
> > *Kaxil Naik*
> > *Big Data Consultant *@ *Data Reply UK*
> > *Certified *Google Cloud Data Engineer | *Certified* Apache Spark & Neo4j
> > Developer
> > *Phone: *+44 (0) 74820 88992
> > *LinkedIn*: https://www.linkedin.com/in/kaxil
>
>

-- 
*Kaxil Naik*
*Big Data Consultant *@ *Data Reply UK*
*Certified *Google Cloud Data Engineer | *Certified* Apache Spark & Neo4j
Developer
*Phone: *+44 (0) 74820 88992
*LinkedIn*: https://www.linkedin.com/in/kaxil

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message