You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Chengwei Yang <ch...@gmail.com> on 2014/03/28 07:35:48 UTC

Review Request 19777: Fix make dist

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19777/
-----------------------------------------------------------

Review request for mesos, Jie Yu, Niklas Nielsen, and Vinod Kone.


Bugs: MESOS-1157
    https://issues.apache.org/jira/browse/MESOS-1157


Repository: mesos-git


Description
-------

Fix make dist


Diffs
-----

  3rdparty/libprocess/3rdparty/Makefile.am 73c9c2ed42348f7c2f31501e77e3b4447366f265 

Diff: https://reviews.apache.org/r/19777/diff/


Testing
-------

done


Thanks,

Chengwei Yang


Re: Review Request 19777: Fix make dist

Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19777/#review39176
-----------------------------------------------------------

Ship it!


The Review Bot failure is because of a flaky test. Not related to your change. I'll get this committed. Thanks.

- Vinod Kone


On March 28, 2014, 6:35 a.m., Chengwei Yang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19777/
> -----------------------------------------------------------
> 
> (Updated March 28, 2014, 6:35 a.m.)
> 
> 
> Review request for mesos, Jie Yu, Niklas Nielsen, and Vinod Kone.
> 
> 
> Bugs: MESOS-1157
>     https://issues.apache.org/jira/browse/MESOS-1157
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Fix make dist
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/Makefile.am 73c9c2ed42348f7c2f31501e77e3b4447366f265 
> 
> Diff: https://reviews.apache.org/r/19777/diff/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Chengwei Yang
> 
>


Re: Review Request 19777: Fix make dist

Posted by Chengwei Yang <ch...@gmail.com>.

> On March 28, 2014, 3:10 p.m., Mesos ReviewBot wrote:
> > Bad patch!
> > 
> > Reviews applied: [19777]
> > 
> > Failed command: make check
> > 
> > Error:
> >  Making check in .
> > make[1]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot'
> > make[1]: Nothing to be done for `check-am'.
> > make[1]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot'
> > Making check in 3rdparty
> > make[1]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
> > make  check-recursive
> > make[2]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
> > Making check in libprocess
> > make[3]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess'
> > Making check in 3rdparty
> > make[4]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make  check-recursive
> > make[5]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > Making check in stout
> > make[6]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty/stout'
> > make[6]: Nothing to be done for `check'.
> > make[6]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty/stout'
> > make[6]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make  libgmock.la stout-tests
> > make[7]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make[7]: `libgmock.la' is up to date.
> > make[7]: `stout-tests' is up to date.
> > make[7]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make  check-local
> > make[7]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > ./stout-tests
> > [==========] Running 117 tests from 24 test cases.
> > [----------] Global test environment set-up.
> > [----------] 3 tests from Stout
> > [ RUN      ] Stout.Bytes
> > [       OK ] Stout.Bytes (0 ms)
> > [ RUN      ] Stout.Set
> > [       OK ] Stout.Set (0 ms)
> > [ RUN      ] Stout.Some
> > [       OK ] Stout.Some (0 ms)
> > [----------] 3 tests from Stout (0 ms total)
> > 
> > [----------] 4 tests from Cache
> > [ RUN      ] Cache.Insert
> > [       OK ] Cache.Insert (0 ms)
> > [ RUN      ] Cache.Update
> > [       OK ] Cache.Update (0 ms)
> > [ RUN      ] Cache.Erase
> > [       OK ] Cache.Erase (0 ms)
> > [ RUN      ] Cache.LRUEviction
> > [       OK ] Cache.LRUEviction (0 ms)
> > [----------] 4 tests from Cache (0 ms total)
> > 
> > [----------] 4 tests from DurationTest
> > [ RUN      ] DurationTest.Comparison
> > [       OK ] DurationTest.Comparison (0 ms)
> > [ RUN      ] DurationTest.ParseAndTry
> > [       OK ] DurationTest.ParseAndTry (0 ms)
> > [ RUN      ] DurationTest.Arithmetic
> > [       OK ] DurationTest.Arithmetic (0 ms)
> > [ RUN      ] DurationTest.OutputFormat
> > [       OK ] DurationTest.OutputFormat (0 ms)
> > [----------] 4 tests from DurationTest (0 ms total)
> > 
> > [----------] 1 test from ErrorTest
> > [ RUN      ] ErrorTest.Test
> > [       OK ] ErrorTest.Test (0 ms)
> > [----------] 1 test from ErrorTest (0 ms total)
> > 
> > [----------] 12 tests from FlagsTest
> > [ RUN      ] FlagsTest.Load
> > [       OK ] FlagsTest.Load (0 ms)
> > [ RUN      ] FlagsTest.Add
> > [       OK ] FlagsTest.Add (0 ms)
> > [ RUN      ] FlagsTest.Flags
> > [       OK ] FlagsTest.Flags (0 ms)
> > [ RUN      ] FlagsTest.LoadFromEnvironment
> > [       OK ] FlagsTest.LoadFromEnvironment (0 ms)
> > [ RUN      ] FlagsTest.LoadFromCommandLine
> > [       OK ] FlagsTest.LoadFromCommandLine (0 ms)
> > [ RUN      ] FlagsTest.LoadFromCommandLineWithNonFlags
> > [       OK ] FlagsTest.LoadFromCommandLineWithNonFlags (0 ms)
> > [ RUN      ] FlagsTest.Stringification
> > [       OK ] FlagsTest.Stringification (0 ms)
> > [ RUN      ] FlagsTest.DuplicatesFromEnvironment
> > [       OK ] FlagsTest.DuplicatesFromEnvironment (0 ms)
> > [ RUN      ] FlagsTest.DuplicatesFromCommandLine
> > [       OK ] FlagsTest.DuplicatesFromCommandLine (0 ms)
> > [ RUN      ] FlagsTest.Errors
> > [       OK ] FlagsTest.Errors (0 ms)
> > [ RUN      ] FlagsTest.Usage
> > [       OK ] FlagsTest.Usage (0 ms)
> > [ RUN      ] FlagsTest.Duration
> > [       OK ] FlagsTest.Duration (0 ms)
> > [----------] 12 tests from FlagsTest (0 ms total)
> > 
> > [----------] 1 test from GzipTest
> > [ RUN      ] GzipTest.CompressDecompressString
> > [       OK ] GzipTest.CompressDecompressString (83 ms)
> > [----------] 1 test from GzipTest (83 ms total)
> > 
> > [----------] 2 tests from HashMapTest
> > [ RUN      ] HashMapTest.Insert
> > [       OK ] HashMapTest.Insert (0 ms)
> > [ RUN      ] HashMapTest.Contains
> > [       OK ] HashMapTest.Contains (0 ms)
> > [----------] 2 tests from HashMapTest (0 ms total)
> > 
> > [----------] 2 tests from HashsetTest
> > [ RUN      ] HashsetTest.Insert
> > [       OK ] HashsetTest.Insert (0 ms)
> > [ RUN      ] HashsetTest.Union
> > [       OK ] HashsetTest.Union (0 ms)
> > [----------] 2 tests from HashsetTest (0 ms total)
> > 
> > [----------] 8 tests from IntervalTest
> > [ RUN      ] IntervalTest.Interval
> > [       OK ] IntervalTest.Interval (0 ms)
> > [ RUN      ] IntervalTest.EmptyInterval
> > [       OK ] IntervalTest.EmptyInterval (0 ms)
> > [ RUN      ] IntervalTest.Constructor
> > [       OK ] IntervalTest.Constructor (0 ms)
> > [ RUN      ] IntervalTest.Addition
> > [       OK ] IntervalTest.Addition (0 ms)
> > [ RUN      ] IntervalTest.Subtraction
> > [       OK ] IntervalTest.Subtraction (0 ms)
> > [ RUN      ] IntervalTest.Intersection
> > [       OK ] IntervalTest.Intersection (0 ms)
> > [ RUN      ] IntervalTest.LargeInterval
> > [       OK ] IntervalTest.LargeInterval (0 ms)
> > [ RUN      ] IntervalTest.IntervalIteration
> > [       OK ] IntervalTest.IntervalIteration (0 ms)
> > [----------] 8 tests from IntervalTest (0 ms total)
> > 
> > [----------] 8 tests from JsonTest
> > [ RUN      ] JsonTest.DefaultValueIsNull
> > [       OK ] JsonTest.DefaultValueIsNull (0 ms)
> > [ RUN      ] JsonTest.BinaryData
> > [       OK ] JsonTest.BinaryData (0 ms)
> > [ RUN      ] JsonTest.NumberFormat
> > [       OK ] JsonTest.NumberFormat (0 ms)
> > [ RUN      ] JsonTest.BooleanFormat
> > [       OK ] JsonTest.BooleanFormat (0 ms)
> > [ RUN      ] JsonTest.BooleanAssignement
> > [       OK ] JsonTest.BooleanAssignement (0 ms)
> > [ RUN      ] JsonTest.CStringAssignment
> > [       OK ] JsonTest.CStringAssignment (0 ms)
> > [ RUN      ] JsonTest.NumericAssignment
> > [       OK ] JsonTest.NumericAssignment (0 ms)
> > [ RUN      ] JsonTest.parse
> > [       OK ] JsonTest.parse (0 ms)
> > [----------] 8 tests from JsonTest (0 ms total)
> > 
> > [----------] 5 tests from LinkedHashmapTest
> > [ RUN      ] LinkedHashmapTest.Put
> > [       OK ] LinkedHashmapTest.Put (0 ms)
> > [ RUN      ] LinkedHashmapTest.Contains
> > [       OK ] LinkedHashmapTest.Contains (0 ms)
> > [ RUN      ] LinkedHashmapTest.Erase
> > [       OK ] LinkedHashmapTest.Erase (0 ms)
> > [ RUN      ] LinkedHashmapTest.Keys
> > [       OK ] LinkedHashmapTest.Keys (0 ms)
> > [ RUN      ] LinkedHashmapTest.Values
> > [       OK ] LinkedHashmapTest.Values (0 ms)
> > [----------] 5 tests from LinkedHashmapTest (0 ms total)
> > 
> > [----------] 6 tests from MultimapTest/0, where TypeParam = Multimap<std::string, unsigned short>
> > [ RUN      ] MultimapTest/0.Put
> > [       OK ] MultimapTest/0.Put (0 ms)
> > [ RUN      ] MultimapTest/0.Remove
> > [       OK ] MultimapTest/0.Remove (0 ms)
> > [ RUN      ] MultimapTest/0.Size
> > [       OK ] MultimapTest/0.Size (0 ms)
> > [ RUN      ] MultimapTest/0.Keys
> > [       OK ] MultimapTest/0.Keys (0 ms)
> > [ RUN      ] MultimapTest/0.Iterator
> > [       OK ] MultimapTest/0.Iterator (0 ms)
> > [ RUN      ] MultimapTest/0.Foreach
> > [       OK ] MultimapTest/0.Foreach (0 ms)
> > [----------] 6 tests from MultimapTest/0 (1 ms total)
> > 
> > [----------] 6 tests from MultimapTest/1, where TypeParam = multihashmap<std::string, unsigned short>
> > [ RUN      ] MultimapTest/1.Put
> > [       OK ] MultimapTest/1.Put (0 ms)
> > [ RUN      ] MultimapTest/1.Remove
> > [       OK ] MultimapTest/1.Remove (0 ms)
> > [ RUN      ] MultimapTest/1.Size
> > [       OK ] MultimapTest/1.Size (0 ms)
> > [ RUN      ] MultimapTest/1.Keys
> > [       OK ] MultimapTest/1.Keys (0 ms)
> > [ RUN      ] MultimapTest/1.Iterator
> > [       OK ] MultimapTest/1.Iterator (0 ms)
> > [ RUN      ] MultimapTest/1.Foreach
> > [       OK ] MultimapTest/1.Foreach (0 ms)
> > [----------] 6 tests from MultimapTest/1 (0 ms total)
> > 
> > [----------] 2 tests from NetTest
> > [ RUN      ] NetTest.mac
> > [       OK ] NetTest.mac (0 ms)
> > [ RUN      ] NetTest.ip
> > [       OK ] NetTest.ip (1 ms)
> > [----------] 2 tests from NetTest (1 ms total)
> > 
> > [----------] 1 test from NoneTest
> > [ RUN      ] NoneTest.Test
> > [       OK ] NoneTest.Test (0 ms)
> > [----------] 1 test from NoneTest (0 ms total)
> > 
> > [----------] 3 tests from OptionTest
> > [ RUN      ] OptionTest.Min
> > [       OK ] OptionTest.Min (0 ms)
> > [ RUN      ] OptionTest.Max
> > [       OK ] OptionTest.Max (0 ms)
> > [ RUN      ] OptionTest.Comparison
> > [       OK ] OptionTest.Comparison (0 ms)
> > [----------] 3 tests from OptionTest (0 ms total)
> > 
> > [----------] 16 tests from OsTest
> > [ RUN      ] OsTest.rmdir
> > [       OK ] OsTest.rmdir (0 ms)
> > [ RUN      ] OsTest.nonblock
> > [       OK ] OsTest.nonblock (0 ms)
> > [ RUN      ] OsTest.touch
> > [       OK ] OsTest.touch (1 ms)
> > [ RUN      ] OsTest.readWriteString
> > [       OK ] OsTest.readWriteString (1 ms)
> > [ RUN      ] OsTest.find
> > [       OK ] OsTest.find (0 ms)
> > [ RUN      ] OsTest.bootId
> > [       OK ] OsTest.bootId (0 ms)
> > [ RUN      ] OsTest.uname
> > [       OK ] OsTest.uname (1 ms)
> > [ RUN      ] OsTest.sysname
> > [       OK ] OsTest.sysname (0 ms)
> > [ RUN      ] OsTest.release
> > [       OK ] OsTest.release (0 ms)
> > [ RUN      ] OsTest.sleep
> > [       OK ] OsTest.sleep (10 ms)
> > [ RUN      ] OsTest.pids
> > [       OK ] OsTest.pids (32 ms)
> > [ RUN      ] OsTest.children
> > [       OK ] OsTest.children (50 ms)
> > [ RUN      ] OsTest.process
> > [       OK ] OsTest.process (0 ms)
> > [ RUN      ] OsTest.processes
> > [       OK ] OsTest.processes (12 ms)
> > [ RUN      ] OsTest.killtree
> > [       OK ] OsTest.killtree (103 ms)
> > [ RUN      ] OsTest.pstree
> > [       OK ] OsTest.pstree (31 ms)
> > [----------] 16 tests from OsTest (241 ms total)
> > 
> > [----------] 1 test from ProtobufTest
> > [ RUN      ] ProtobufTest.JSON
> > [       OK ] ProtobufTest.JSON (0 ms)
> > [----------] 1 test from ProtobufTest (0 ms total)
> > 
> > [----------] 1 test from OsSendfileTest
> > [ RUN      ] OsSendfileTest.sendfile
> > [       OK ] OsSendfileTest.sendfile (0 ms)
> > [----------] 1 test from OsSendfileTest (0 ms total)
> > 
> > [----------] 1 test from OsSignalsTest
> > [ RUN      ] OsSignalsTest.suppress
> > [       OK ] OsSignalsTest.suppress (1 ms)
> > [----------] 1 test from OsSignalsTest (1 ms total)
> > 
> > [----------] 22 tests from StringsTest
> > [ RUN      ] StringsTest.Format
> > [       OK ] StringsTest.Format (0 ms)
> > [ RUN      ] StringsTest.Remove
> > [       OK ] StringsTest.Remove (0 ms)
> > [ RUN      ] StringsTest.Replace
> > [       OK ] StringsTest.Replace (0 ms)
> > [ RUN      ] StringsTest.Trim
> > [       OK ] StringsTest.Trim (0 ms)
> > [ RUN      ] StringsTest.Tokenize
> > [       OK ] StringsTest.Tokenize (0 ms)
> > [ RUN      ] StringsTest.TokenizeStringWithDelimsAtStart
> > [       OK ] StringsTest.TokenizeStringWithDelimsAtStart (0 ms)
> > [ RUN      ] StringsTest.TokenizeStringWithDelimsAtEnd
> > [       OK ] StringsTest.TokenizeStringWithDelimsAtEnd (0 ms)
> > [ RUN      ] StringsTest.TokenizeStringWithDelimsAtStartAndEnd
> > [       OK ] StringsTest.TokenizeStringWithDelimsAtStartAndEnd (0 ms)
> > [ RUN      ] StringsTest.TokenizeWithMultipleDelims
> > [       OK ] StringsTest.TokenizeWithMultipleDelims (0 ms)
> > [ RUN      ] StringsTest.TokenizeEmptyString
> > [       OK ] StringsTest.TokenizeEmptyString (0 ms)
> > [ RUN      ] StringsTest.TokenizeDelimOnlyString
> > [       OK ] StringsTest.TokenizeDelimOnlyString (0 ms)
> > [ RUN      ] StringsTest.TokenizeNullByteDelim
> > [       OK ] StringsTest.TokenizeNullByteDelim (0 ms)
> > [ RUN      ] StringsTest.SplitEmptyString
> > [       OK ] StringsTest.SplitEmptyString (0 ms)
> > [ RUN      ] StringsTest.SplitDelimOnlyString
> > [       OK ] StringsTest.SplitDelimOnlyString (0 ms)
> > [ RUN      ] StringsTest.Split
> > [       OK ] StringsTest.Split (0 ms)
> > [ RUN      ] StringsTest.SplitStringWithDelimsAtStart
> > [       OK ] StringsTest.SplitStringWithDelimsAtStart (0 ms)
> > [ RUN      ] StringsTest.SplitStringWithDelimsAtEnd
> > [       OK ] StringsTest.SplitStringWithDelimsAtEnd (0 ms)
> > [ RUN      ] StringsTest.SplitStringWithDelimsAtStartAndEnd
> > [       OK ] StringsTest.SplitStringWithDelimsAtStartAndEnd (0 ms)
> > [ RUN      ] StringsTest.SplitWithMultipleDelims
> > [       OK ] StringsTest.SplitWithMultipleDelims (0 ms)
> > [ RUN      ] StringsTest.Pairs
> > [       OK ] StringsTest.Pairs (0 ms)
> > [ RUN      ] StringsTest.StartsWith
> > [       OK ] StringsTest.StartsWith (0 ms)
> > [ RUN      ] StringsTest.Contains
> > [       OK ] StringsTest.Contains (0 ms)
> > [----------] 22 tests from StringsTest (0 ms total)
> > 
> > [----------] 1 test from Thread
> > [ RUN      ] Thread.local
> > [       OK ] Thread.local (0 ms)
> > [----------] 1 test from Thread (0 ms total)
> > 
> > [----------] 1 test from UUIDTest
> > [ RUN      ] UUIDTest.test
> > [       OK ] UUIDTest.test (0 ms)
> > [----------] 1 test from UUIDTest (0 ms total)
> > 
> > [----------] 6 tests from ProcTest
> > [ RUN      ] ProcTest.pids
> > [       OK ] ProcTest.pids (1 ms)
> > [ RUN      ] ProcTest.cpus
> > [       OK ] ProcTest.cpus (1 ms)
> > [ RUN      ] ProcTest.SystemStatus
> > [       OK ] ProcTest.SystemStatus (0 ms)
> > [ RUN      ] ProcTest.ProcessStatus
> > [       OK ] ProcTest.ProcessStatus (0 ms)
> > [ RUN      ] ProcTest.SingleThread
> > [       OK ] ProcTest.SingleThread (1 ms)
> > [ RUN      ] ProcTest.MultipleThreads
> > stout/tests/proc_tests.cpp:181: Failure
> > Value of: (procThreads).get()
> >   Actual: { 19479 }
> > Expected: childThreads
> > Which is: { 152 }
> > [  FAILED  ] ProcTest.MultipleThreads (154 ms)
> > [----------] 6 tests from ProcTest (157 ms total)
> > 
> > [----------] Global test environment tear-down
> > [==========] 117 tests from 24 test cases ran. (485 ms total)
> > [  PASSED  ] 116 tests.
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] ProcTest.MultipleThreads
> > 
> >  1 FAILED TEST
> > make[7]: *** [check-local] Error 1
> > make[7]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make[6]: *** [check-am] Error 2
> > make[6]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make[5]: *** [check-recursive] Error 1
> > make[5]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make[4]: *** [check] Error 2
> > make[4]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
> > make[3]: *** [check-recursive] Error 1
> > make[3]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess'
> > make[2]: *** [check-recursive] Error 1
> > make[2]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
> > make[1]: *** [check] Error 2
> > make[1]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
> > make: *** [check-recursive] Error 1
> >

This is quite weird, "make check" passed in my local test, my patch applied on top of 14039c5 Added a const qualifier to Stopwatch::elapsed.

I think there are new changes now which introduced this failure.


- Chengwei


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19777/#review38884
-----------------------------------------------------------


On March 28, 2014, 2:35 p.m., Chengwei Yang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19777/
> -----------------------------------------------------------
> 
> (Updated March 28, 2014, 2:35 p.m.)
> 
> 
> Review request for mesos, Jie Yu, Niklas Nielsen, and Vinod Kone.
> 
> 
> Bugs: MESOS-1157
>     https://issues.apache.org/jira/browse/MESOS-1157
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Fix make dist
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/Makefile.am 73c9c2ed42348f7c2f31501e77e3b4447366f265 
> 
> Diff: https://reviews.apache.org/r/19777/diff/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Chengwei Yang
> 
>


Re: Review Request 19777: Fix make dist

Posted by Mesos ReviewBot <de...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19777/#review38884
-----------------------------------------------------------


Bad patch!

Reviews applied: [19777]

Failed command: make check

Error:
 Making check in .
make[1]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot'
make[1]: Nothing to be done for `check-am'.
make[1]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot'
Making check in 3rdparty
make[1]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
make  check-recursive
make[2]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
Making check in libprocess
make[3]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess'
Making check in 3rdparty
make[4]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make  check-recursive
make[5]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
Making check in stout
make[6]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty/stout'
make[6]: Nothing to be done for `check'.
make[6]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty/stout'
make[6]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make  libgmock.la stout-tests
make[7]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[7]: `libgmock.la' is up to date.
make[7]: `stout-tests' is up to date.
make[7]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make  check-local
make[7]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
./stout-tests
[==========] Running 117 tests from 24 test cases.
[----------] Global test environment set-up.
[----------] 3 tests from Stout
[ RUN      ] Stout.Bytes
[       OK ] Stout.Bytes (0 ms)
[ RUN      ] Stout.Set
[       OK ] Stout.Set (0 ms)
[ RUN      ] Stout.Some
[       OK ] Stout.Some (0 ms)
[----------] 3 tests from Stout (0 ms total)

[----------] 4 tests from Cache
[ RUN      ] Cache.Insert
[       OK ] Cache.Insert (0 ms)
[ RUN      ] Cache.Update
[       OK ] Cache.Update (0 ms)
[ RUN      ] Cache.Erase
[       OK ] Cache.Erase (0 ms)
[ RUN      ] Cache.LRUEviction
[       OK ] Cache.LRUEviction (0 ms)
[----------] 4 tests from Cache (0 ms total)

[----------] 4 tests from DurationTest
[ RUN      ] DurationTest.Comparison
[       OK ] DurationTest.Comparison (0 ms)
[ RUN      ] DurationTest.ParseAndTry
[       OK ] DurationTest.ParseAndTry (0 ms)
[ RUN      ] DurationTest.Arithmetic
[       OK ] DurationTest.Arithmetic (0 ms)
[ RUN      ] DurationTest.OutputFormat
[       OK ] DurationTest.OutputFormat (0 ms)
[----------] 4 tests from DurationTest (0 ms total)

[----------] 1 test from ErrorTest
[ RUN      ] ErrorTest.Test
[       OK ] ErrorTest.Test (0 ms)
[----------] 1 test from ErrorTest (0 ms total)

[----------] 12 tests from FlagsTest
[ RUN      ] FlagsTest.Load
[       OK ] FlagsTest.Load (0 ms)
[ RUN      ] FlagsTest.Add
[       OK ] FlagsTest.Add (0 ms)
[ RUN      ] FlagsTest.Flags
[       OK ] FlagsTest.Flags (0 ms)
[ RUN      ] FlagsTest.LoadFromEnvironment
[       OK ] FlagsTest.LoadFromEnvironment (0 ms)
[ RUN      ] FlagsTest.LoadFromCommandLine
[       OK ] FlagsTest.LoadFromCommandLine (0 ms)
[ RUN      ] FlagsTest.LoadFromCommandLineWithNonFlags
[       OK ] FlagsTest.LoadFromCommandLineWithNonFlags (0 ms)
[ RUN      ] FlagsTest.Stringification
[       OK ] FlagsTest.Stringification (0 ms)
[ RUN      ] FlagsTest.DuplicatesFromEnvironment
[       OK ] FlagsTest.DuplicatesFromEnvironment (0 ms)
[ RUN      ] FlagsTest.DuplicatesFromCommandLine
[       OK ] FlagsTest.DuplicatesFromCommandLine (0 ms)
[ RUN      ] FlagsTest.Errors
[       OK ] FlagsTest.Errors (0 ms)
[ RUN      ] FlagsTest.Usage
[       OK ] FlagsTest.Usage (0 ms)
[ RUN      ] FlagsTest.Duration
[       OK ] FlagsTest.Duration (0 ms)
[----------] 12 tests from FlagsTest (0 ms total)

[----------] 1 test from GzipTest
[ RUN      ] GzipTest.CompressDecompressString
[       OK ] GzipTest.CompressDecompressString (83 ms)
[----------] 1 test from GzipTest (83 ms total)

[----------] 2 tests from HashMapTest
[ RUN      ] HashMapTest.Insert
[       OK ] HashMapTest.Insert (0 ms)
[ RUN      ] HashMapTest.Contains
[       OK ] HashMapTest.Contains (0 ms)
[----------] 2 tests from HashMapTest (0 ms total)

[----------] 2 tests from HashsetTest
[ RUN      ] HashsetTest.Insert
[       OK ] HashsetTest.Insert (0 ms)
[ RUN      ] HashsetTest.Union
[       OK ] HashsetTest.Union (0 ms)
[----------] 2 tests from HashsetTest (0 ms total)

[----------] 8 tests from IntervalTest
[ RUN      ] IntervalTest.Interval
[       OK ] IntervalTest.Interval (0 ms)
[ RUN      ] IntervalTest.EmptyInterval
[       OK ] IntervalTest.EmptyInterval (0 ms)
[ RUN      ] IntervalTest.Constructor
[       OK ] IntervalTest.Constructor (0 ms)
[ RUN      ] IntervalTest.Addition
[       OK ] IntervalTest.Addition (0 ms)
[ RUN      ] IntervalTest.Subtraction
[       OK ] IntervalTest.Subtraction (0 ms)
[ RUN      ] IntervalTest.Intersection
[       OK ] IntervalTest.Intersection (0 ms)
[ RUN      ] IntervalTest.LargeInterval
[       OK ] IntervalTest.LargeInterval (0 ms)
[ RUN      ] IntervalTest.IntervalIteration
[       OK ] IntervalTest.IntervalIteration (0 ms)
[----------] 8 tests from IntervalTest (0 ms total)

[----------] 8 tests from JsonTest
[ RUN      ] JsonTest.DefaultValueIsNull
[       OK ] JsonTest.DefaultValueIsNull (0 ms)
[ RUN      ] JsonTest.BinaryData
[       OK ] JsonTest.BinaryData (0 ms)
[ RUN      ] JsonTest.NumberFormat
[       OK ] JsonTest.NumberFormat (0 ms)
[ RUN      ] JsonTest.BooleanFormat
[       OK ] JsonTest.BooleanFormat (0 ms)
[ RUN      ] JsonTest.BooleanAssignement
[       OK ] JsonTest.BooleanAssignement (0 ms)
[ RUN      ] JsonTest.CStringAssignment
[       OK ] JsonTest.CStringAssignment (0 ms)
[ RUN      ] JsonTest.NumericAssignment
[       OK ] JsonTest.NumericAssignment (0 ms)
[ RUN      ] JsonTest.parse
[       OK ] JsonTest.parse (0 ms)
[----------] 8 tests from JsonTest (0 ms total)

[----------] 5 tests from LinkedHashmapTest
[ RUN      ] LinkedHashmapTest.Put
[       OK ] LinkedHashmapTest.Put (0 ms)
[ RUN      ] LinkedHashmapTest.Contains
[       OK ] LinkedHashmapTest.Contains (0 ms)
[ RUN      ] LinkedHashmapTest.Erase
[       OK ] LinkedHashmapTest.Erase (0 ms)
[ RUN      ] LinkedHashmapTest.Keys
[       OK ] LinkedHashmapTest.Keys (0 ms)
[ RUN      ] LinkedHashmapTest.Values
[       OK ] LinkedHashmapTest.Values (0 ms)
[----------] 5 tests from LinkedHashmapTest (0 ms total)

[----------] 6 tests from MultimapTest/0, where TypeParam = Multimap<std::string, unsigned short>
[ RUN      ] MultimapTest/0.Put
[       OK ] MultimapTest/0.Put (0 ms)
[ RUN      ] MultimapTest/0.Remove
[       OK ] MultimapTest/0.Remove (0 ms)
[ RUN      ] MultimapTest/0.Size
[       OK ] MultimapTest/0.Size (0 ms)
[ RUN      ] MultimapTest/0.Keys
[       OK ] MultimapTest/0.Keys (0 ms)
[ RUN      ] MultimapTest/0.Iterator
[       OK ] MultimapTest/0.Iterator (0 ms)
[ RUN      ] MultimapTest/0.Foreach
[       OK ] MultimapTest/0.Foreach (0 ms)
[----------] 6 tests from MultimapTest/0 (1 ms total)

[----------] 6 tests from MultimapTest/1, where TypeParam = multihashmap<std::string, unsigned short>
[ RUN      ] MultimapTest/1.Put
[       OK ] MultimapTest/1.Put (0 ms)
[ RUN      ] MultimapTest/1.Remove
[       OK ] MultimapTest/1.Remove (0 ms)
[ RUN      ] MultimapTest/1.Size
[       OK ] MultimapTest/1.Size (0 ms)
[ RUN      ] MultimapTest/1.Keys
[       OK ] MultimapTest/1.Keys (0 ms)
[ RUN      ] MultimapTest/1.Iterator
[       OK ] MultimapTest/1.Iterator (0 ms)
[ RUN      ] MultimapTest/1.Foreach
[       OK ] MultimapTest/1.Foreach (0 ms)
[----------] 6 tests from MultimapTest/1 (0 ms total)

[----------] 2 tests from NetTest
[ RUN      ] NetTest.mac
[       OK ] NetTest.mac (0 ms)
[ RUN      ] NetTest.ip
[       OK ] NetTest.ip (1 ms)
[----------] 2 tests from NetTest (1 ms total)

[----------] 1 test from NoneTest
[ RUN      ] NoneTest.Test
[       OK ] NoneTest.Test (0 ms)
[----------] 1 test from NoneTest (0 ms total)

[----------] 3 tests from OptionTest
[ RUN      ] OptionTest.Min
[       OK ] OptionTest.Min (0 ms)
[ RUN      ] OptionTest.Max
[       OK ] OptionTest.Max (0 ms)
[ RUN      ] OptionTest.Comparison
[       OK ] OptionTest.Comparison (0 ms)
[----------] 3 tests from OptionTest (0 ms total)

[----------] 16 tests from OsTest
[ RUN      ] OsTest.rmdir
[       OK ] OsTest.rmdir (0 ms)
[ RUN      ] OsTest.nonblock
[       OK ] OsTest.nonblock (0 ms)
[ RUN      ] OsTest.touch
[       OK ] OsTest.touch (1 ms)
[ RUN      ] OsTest.readWriteString
[       OK ] OsTest.readWriteString (1 ms)
[ RUN      ] OsTest.find
[       OK ] OsTest.find (0 ms)
[ RUN      ] OsTest.bootId
[       OK ] OsTest.bootId (0 ms)
[ RUN      ] OsTest.uname
[       OK ] OsTest.uname (1 ms)
[ RUN      ] OsTest.sysname
[       OK ] OsTest.sysname (0 ms)
[ RUN      ] OsTest.release
[       OK ] OsTest.release (0 ms)
[ RUN      ] OsTest.sleep
[       OK ] OsTest.sleep (10 ms)
[ RUN      ] OsTest.pids
[       OK ] OsTest.pids (32 ms)
[ RUN      ] OsTest.children
[       OK ] OsTest.children (50 ms)
[ RUN      ] OsTest.process
[       OK ] OsTest.process (0 ms)
[ RUN      ] OsTest.processes
[       OK ] OsTest.processes (12 ms)
[ RUN      ] OsTest.killtree
[       OK ] OsTest.killtree (103 ms)
[ RUN      ] OsTest.pstree
[       OK ] OsTest.pstree (31 ms)
[----------] 16 tests from OsTest (241 ms total)

[----------] 1 test from ProtobufTest
[ RUN      ] ProtobufTest.JSON
[       OK ] ProtobufTest.JSON (0 ms)
[----------] 1 test from ProtobufTest (0 ms total)

[----------] 1 test from OsSendfileTest
[ RUN      ] OsSendfileTest.sendfile
[       OK ] OsSendfileTest.sendfile (0 ms)
[----------] 1 test from OsSendfileTest (0 ms total)

[----------] 1 test from OsSignalsTest
[ RUN      ] OsSignalsTest.suppress
[       OK ] OsSignalsTest.suppress (1 ms)
[----------] 1 test from OsSignalsTest (1 ms total)

[----------] 22 tests from StringsTest
[ RUN      ] StringsTest.Format
[       OK ] StringsTest.Format (0 ms)
[ RUN      ] StringsTest.Remove
[       OK ] StringsTest.Remove (0 ms)
[ RUN      ] StringsTest.Replace
[       OK ] StringsTest.Replace (0 ms)
[ RUN      ] StringsTest.Trim
[       OK ] StringsTest.Trim (0 ms)
[ RUN      ] StringsTest.Tokenize
[       OK ] StringsTest.Tokenize (0 ms)
[ RUN      ] StringsTest.TokenizeStringWithDelimsAtStart
[       OK ] StringsTest.TokenizeStringWithDelimsAtStart (0 ms)
[ RUN      ] StringsTest.TokenizeStringWithDelimsAtEnd
[       OK ] StringsTest.TokenizeStringWithDelimsAtEnd (0 ms)
[ RUN      ] StringsTest.TokenizeStringWithDelimsAtStartAndEnd
[       OK ] StringsTest.TokenizeStringWithDelimsAtStartAndEnd (0 ms)
[ RUN      ] StringsTest.TokenizeWithMultipleDelims
[       OK ] StringsTest.TokenizeWithMultipleDelims (0 ms)
[ RUN      ] StringsTest.TokenizeEmptyString
[       OK ] StringsTest.TokenizeEmptyString (0 ms)
[ RUN      ] StringsTest.TokenizeDelimOnlyString
[       OK ] StringsTest.TokenizeDelimOnlyString (0 ms)
[ RUN      ] StringsTest.TokenizeNullByteDelim
[       OK ] StringsTest.TokenizeNullByteDelim (0 ms)
[ RUN      ] StringsTest.SplitEmptyString
[       OK ] StringsTest.SplitEmptyString (0 ms)
[ RUN      ] StringsTest.SplitDelimOnlyString
[       OK ] StringsTest.SplitDelimOnlyString (0 ms)
[ RUN      ] StringsTest.Split
[       OK ] StringsTest.Split (0 ms)
[ RUN      ] StringsTest.SplitStringWithDelimsAtStart
[       OK ] StringsTest.SplitStringWithDelimsAtStart (0 ms)
[ RUN      ] StringsTest.SplitStringWithDelimsAtEnd
[       OK ] StringsTest.SplitStringWithDelimsAtEnd (0 ms)
[ RUN      ] StringsTest.SplitStringWithDelimsAtStartAndEnd
[       OK ] StringsTest.SplitStringWithDelimsAtStartAndEnd (0 ms)
[ RUN      ] StringsTest.SplitWithMultipleDelims
[       OK ] StringsTest.SplitWithMultipleDelims (0 ms)
[ RUN      ] StringsTest.Pairs
[       OK ] StringsTest.Pairs (0 ms)
[ RUN      ] StringsTest.StartsWith
[       OK ] StringsTest.StartsWith (0 ms)
[ RUN      ] StringsTest.Contains
[       OK ] StringsTest.Contains (0 ms)
[----------] 22 tests from StringsTest (0 ms total)

[----------] 1 test from Thread
[ RUN      ] Thread.local
[       OK ] Thread.local (0 ms)
[----------] 1 test from Thread (0 ms total)

[----------] 1 test from UUIDTest
[ RUN      ] UUIDTest.test
[       OK ] UUIDTest.test (0 ms)
[----------] 1 test from UUIDTest (0 ms total)

[----------] 6 tests from ProcTest
[ RUN      ] ProcTest.pids
[       OK ] ProcTest.pids (1 ms)
[ RUN      ] ProcTest.cpus
[       OK ] ProcTest.cpus (1 ms)
[ RUN      ] ProcTest.SystemStatus
[       OK ] ProcTest.SystemStatus (0 ms)
[ RUN      ] ProcTest.ProcessStatus
[       OK ] ProcTest.ProcessStatus (0 ms)
[ RUN      ] ProcTest.SingleThread
[       OK ] ProcTest.SingleThread (1 ms)
[ RUN      ] ProcTest.MultipleThreads
stout/tests/proc_tests.cpp:181: Failure
Value of: (procThreads).get()
  Actual: { 19479 }
Expected: childThreads
Which is: { 152 }
[  FAILED  ] ProcTest.MultipleThreads (154 ms)
[----------] 6 tests from ProcTest (157 ms total)

[----------] Global test environment tear-down
[==========] 117 tests from 24 test cases ran. (485 ms total)
[  PASSED  ] 116 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ProcTest.MultipleThreads

 1 FAILED TEST
make[7]: *** [check-local] Error 1
make[7]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[6]: *** [check-am] Error 2
make[6]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[5]: *** [check-recursive] Error 1
make[5]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[4]: *** [check] Error 2
make[4]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[3]: *** [check-recursive] Error 1
make[3]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess'
make[2]: *** [check-recursive] Error 1
make[2]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
make[1]: *** [check] Error 2
make[1]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
make: *** [check-recursive] Error 1


- Mesos ReviewBot


On March 28, 2014, 6:35 a.m., Chengwei Yang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19777/
> -----------------------------------------------------------
> 
> (Updated March 28, 2014, 6:35 a.m.)
> 
> 
> Review request for mesos, Jie Yu, Niklas Nielsen, and Vinod Kone.
> 
> 
> Bugs: MESOS-1157
>     https://issues.apache.org/jira/browse/MESOS-1157
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Fix make dist
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/Makefile.am 73c9c2ed42348f7c2f31501e77e3b4447366f265 
> 
> Diff: https://reviews.apache.org/r/19777/diff/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Chengwei Yang
> 
>