+1 on disabling Appveyor (we have windows coverage on Jenkins and if we think
we need more, let's put it there)
+1 on re-enabling travis-ci. We absolutely need continuous regressions for OSX.
Am Freitag, 8. Dezember 2017, 16:32:23 MEZ hat Pedro Larroy
<pedro.larroy.li...@gmail.com> Folgendes geschrieben:
We can enable with just everything commented out and returning "true"
so we can fix it on successive PRs. Do we need an admin for that?
On Fri, Dec 8, 2017 at 12:20 PM, Marco de Abreu
<marco.g.ab...@googlemail.com> wrote:
> Who's going to maintain and fix problems which will arise with travis CI?
> Due to the protected master branch, this can cause some serious blocking
> issues. I think it should be tested and everything fixed before we
> re-enable travis CI.
>
> My proposal is to wait for stage 3 (new features) of the new CI launch
> until we start working on Travis CI.
>
> On Fri, Dec 8, 2017 at 12:07 PM, Pedro Larroy <pedro.larroy.li...@gmail.com>
> wrote:
>
>> Can somebody re-enable travis CI so we can do mac builds? currently
>> the unit tests don't build in Mac because Mac is not tested.
>>
>> On Fri, Dec 8, 2017 at 12:02 PM, Pedro Larroy
>> <pedro.larroy.li...@gmail.com> wrote:
>> > +1
>> >
>> >
>> >
>> > On Fri, Dec 8, 2017 at 1:02 AM, Chris Olivier <cjolivie...@gmail.com>
>> wrote:
>> >> +1
>> >>
>> >> On Thu, Dec 7, 2017 at 3:57 PM, kellen sunderland <
>> >> kellen.sunderl...@gmail.com> wrote:
>> >>
>> >>> It's using a fixed binary version of openblas and opencv, so the
>> versions
>> >>> there might be different than what we use. It's compiling with
>> profiler
>> >>> enabled. A few build flags. However all it's doing is building, it
>> >>> doesn't test.
>> >>>
>> >>> The best lines to compare would be this:
>> >>> https://github.com/apache/incubator-mxnet/blob/master/Jenkinsfile#L236
>> >>> to this:
>> >>> https://ci.appveyor.com/project/ApacheSoftwareFoundation/
>> >>> incubator-mxnet/build/1.0.4047#L96
>> >>> to get an idea of the (minor) differences.
>> >>>
>> >>>
>> >>> On Dec 8, 2017 12:51 AM, "Chris Olivier" <cjolivie...@gmail.com>
>> wrote:
>> >>>
>> >>> Does Appveyor do anything that Jenkins isn't doing? Anything at all?
>> >>>
>> >>> On Thu, Dec 7, 2017 at 3:41 PM, kellen sunderland <
>> >>> kellen.sunderl...@gmail.com> wrote:
>> >>>
>> >>> > @Chris nope, AppVeyor has only ever supported Windows.
>> >>> >
>> >>> > I think some people are thinking of Travis CI, which is a pretty
>> great
>> >>> > managed CI that does support MacOS/Linux. I'd be in favour of
>> setting up
>> >>> a
>> >>> > Mac build there. At a minimum we can at least start a new compiling
>> task
>> >>> > with clang in the new CI.
>> >>> >
>> >>> > On Dec 7, 2017 11:41 PM, "Indhu" <indhubhara...@gmail.com> wrote:
>> >>> >
>> >>> > > +1
>> >>> > >
>> >>> > > My build is waiting in queue for 13 hours now.
>> >>> > >
>> >>> > >
>> >>> > > On Thu, Dec 7, 2017, 11:26 AM kellen sunderland <
>> >>> > > kellen.sunderl...@gmail.com>
>> >>> > > wrote:
>> >>> > >
>> >>> > > > Background: Appveyor is a free-to-use (for opensource) CI system
>> >>> that
>> >>> > > > specializes in providing support for Windows environments.
>> Appveyor
>> >>> is
>> >>> > > > currently running a very simple windows build for MXNet.
>> >>> Specifically
>> >>> > we
>> >>> > > > are generating build files via cmake, and then building with
>> Visual
>> >>> > > Studio
>> >>> > > > 14 with fairly standard build options.
>> >>> > > >
>> >>> > > > Problem: AppVeyor frequently slows down our release process and
>> has
>> >>> > been
>> >>> > > > made redundant by the new CI system's windows builds.
>> >>> > > >
>> >>> > > > Action: Let's remove AppVeyor and rely on the new CI's windows
>> build
>> >>> > > > systems which has very similar functionality.
>> >>> > > >
>> >>> > > > Vote will close Monday Dec 11th and will rely on lazy consensus.
>> >>> > > >
>> >>> > > > -Kellen
>> >>> > > >
>> >>> > >
>> >>> >
>> >>>
>>
