mesos-reviews mailing list archives: September 2019

Site index · List index
Message list1 · 2 · 3 · Next »Thread · Author · Date
Benjamin Bannier Review Request 71419: Fixed anchor name in upgrades guide. Mon, 02 Sep, 08:14
Jan Schlicht   Re: Review Request 71419: Fixed anchor name in upgrades guide. Mon, 02 Sep, 08:23
Benjamin Bannier Review Request 71420: Fixed typo in reservation documentation. Mon, 02 Sep, 08:14
Jan Schlicht   Re: Review Request 71420: Fixed typo in reservation documentation. Mon, 02 Sep, 08:24
Benjamin Bannier Review Request 71421: Switched verify-reviews from docker-build to prepared Docker image. Mon, 02 Sep, 09:35
Mesos Reviewbot   Re: Review Request 71421: Switched verify-reviews from docker-build to prepared Docker image. Mon, 02 Sep, 12:17
Benjamin Bannier Review Request 71422: Added Coverity support to mesos-build. Mon, 02 Sep, 09:53
Benjamin Bannier Review Request 71423: Removed legacy docker-build script. Mon, 02 Sep, 09:57
Mesos Reviewbot   Re: Review Request 71423: Removed legacy docker-build script. Mon, 02 Sep, 13:06
Re: Review Request 71390: Fixed the snapshot script to be non-interactive.
Benjamin Bannier   Re: Review Request 71390: Fixed the snapshot script to be non-interactive. Mon, 02 Sep, 11:19
Till Toenshoff via Review Board   Re: Review Request 71390: Fixed the snapshot script to be non-interactive. Thu, 05 Sep, 11:51
Re: Review Request 71391: Fixed sed arg to work on Linux and OSX.
Benjamin Bannier   Re: Review Request 71391: Fixed sed arg to work on Linux and OSX. Mon, 02 Sep, 12:41
Re: Review Request 71392: Improved snapshot script to deduce the Mesos version.
Benjamin Bannier   Re: Review Request 71392: Improved snapshot script to deduce the Mesos version. Mon, 02 Sep, 12:41
Benjamin Bannier     Re: Review Request 71392: Improved snapshot script to deduce the Mesos version. Thu, 05 Sep, 13:33
Till Toenshoff via Review Board   Re: Review Request 71392: Improved snapshot script to deduce the Mesos version. Thu, 05 Sep, 13:34
Re: Review Request 71393: Skiped GPG signing during `maven-install` in Makefile.
Benjamin Bannier   Re: Review Request 71393: Skiped GPG signing during `maven-install` in Makefile. Mon, 02 Sep, 12:44
Till Toenshoff via Review Board   Re: Review Request 71393: Skiped GPG signing during `maven-install` in Makefile. Thu, 05 Sep, 13:35
Re: Review Request 71394: Skipped GPG signing for snapshot builds.
Benjamin Bannier   Re: Review Request 71394: Skipped GPG signing for snapshot builds. Mon, 02 Sep, 12:45
Till Toenshoff via Review Board   Re: Review Request 71394: Skipped GPG signing for snapshot builds. Thu, 05 Sep, 13:35
Re: Review Request 71395: Implemented snapshot bot.
Benjamin Bannier   Re: Review Request 71395: Implemented snapshot bot. Mon, 02 Sep, 12:50
Till Toenshoff via Review Board   Re: Review Request 71395: Implemented snapshot bot. Thu, 05 Sep, 13:36
Till Toenshoff via Review Board   Re: Review Request 71395: Implemented snapshot bot. Thu, 05 Sep, 13:37
Andrei Budnik Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Mon, 02 Sep, 16:36
Mesos Reviewbot   Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Mon, 02 Sep, 19:02
James Peach   Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Wed, 04 Sep, 05:52
Andrei Budnik   Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Thu, 05 Sep, 14:00
Benno Evers     Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Thu, 05 Sep, 14:22
Mesos Reviewbot     Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Thu, 05 Sep, 17:47
James Peach     Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. Fri, 06 Sep, 06:23
Re: Review Request 71382: Removed unused and unimplemented method declaration.
Jan Schlicht   Re: Review Request 71382: Removed unused and unimplemented method declaration. Tue, 03 Sep, 12:50
Re: Review Request 71383: Added stringification for resource provider calls.
Jan Schlicht   Re: Review Request 71383: Added stringification for resource provider calls. Tue, 03 Sep, 12:58
Re: Review Request 71384: Passed down failures into resource provider fatal handler.
Jan Schlicht   Re: Review Request 71384: Passed down failures into resource provider fatal handler. Tue, 03 Sep, 13:02
Jan Schlicht     Re: Review Request 71384: Passed down failures into resource provider fatal handler. Tue, 03 Sep, 13:09
Re: Review Request 71240: Added a `dist` target to the cmake build.
Benno Evers   Re: Review Request 71240: Added a `dist` target to the cmake build. Tue, 03 Sep, 13:08
Benjamin Bannier     Re: Review Request 71240: Added a `dist` target to the cmake build. Tue, 03 Sep, 13:52
Benjamin Bannier   Re: Review Request 71240: Added a `dist` target to the cmake build. Tue, 03 Sep, 13:52
Benjamin Bannier     Re: Review Request 71240: Added a `dist` target to the cmake build. Tue, 03 Sep, 14:45
Till Toenshoff via Review Board     Re: Review Request 71240: Added a `dist` target to the cmake build. Thu, 05 Sep, 10:39
Till Toenshoff via Review Board     Re: Review Request 71240: Added a `dist` target to the cmake build. Thu, 05 Sep, 10:44
Till Toenshoff via Review Board     Re: Review Request 71240: Added a `dist` target to the cmake build. Thu, 05 Sep, 10:51
Benjamin Bannier     Re: Review Request 71240: Added a `dist` target to the cmake build. Fri, 06 Sep, 10:13
Re: Review Request 71241: Added a `distcheck` target to the cmake build.
Benno Evers   Re: Review Request 71241: Added a `distcheck` target to the cmake build. Tue, 03 Sep, 13:08
Benjamin Bannier     Re: Review Request 71241: Added a `distcheck` target to the cmake build. Tue, 03 Sep, 13:52
Benjamin Bannier   Re: Review Request 71241: Added a `distcheck` target to the cmake build. Tue, 03 Sep, 13:52
Benjamin Bannier     Re: Review Request 71241: Added a `distcheck` target to the cmake build. Tue, 03 Sep, 14:51
Mesos Reviewbot       Re: Review Request 71241: Added a `distcheck` target to the cmake build. Tue, 03 Sep, 19:33
Till Toenshoff via Review Board       Re: Review Request 71241: Added a `distcheck` target to the cmake build. Thu, 05 Sep, 11:47
Mesos Reviewbot       Re: Review Request 71241: Added a `distcheck` target to the cmake build. Fri, 06 Sep, 12:42
Re: Review Request 71398: Made `allocated` mean "allocated but not offered" in `allocator::Slave`.
Andrei Sekretenko   Re: Review Request 71398: Made `allocated` mean "allocated but not offered" in `allocator::Slave`. Tue, 03 Sep, 13:16
Meng Zhu     Re: Review Request 71398: Made `allocated` mean "allocated but not offered" in `allocator::Slave`. Tue, 03 Sep, 17:24
Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess.
Andrei Sekretenko   Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess. Tue, 03 Sep, 13:23
Meng Zhu     Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess. Tue, 03 Sep, 17:25
Andrei Sekretenko   Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess. Tue, 03 Sep, 13:23
Re: Review Request 71400: Renamed `toAllocate` into `toOffer` in `HierarchialAllocatorProcess`.
Andrei Sekretenko   Re: Review Request 71400: Renamed `toAllocate` into `toOffer` in `HierarchialAllocatorProcess`. Tue, 03 Sep, 13:24
Re: Review Request 71402: Renamed *Allocation into *Offering in __generateOffers().
Andrei Sekretenko   Re: Review Request 71402: Renamed *Allocation into *Offering in __generateOffers(). Tue, 03 Sep, 13:26
Mesos Reviewbot     Re: Review Request 71402: Renamed *Allocation into *Offering in __generateOffers(). Tue, 03 Sep, 15:42
Re: Review Request 71385: Added restart logic for failing resource providers.
Jan Schlicht   Re: Review Request 71385: Added restart logic for failing resource providers. Tue, 03 Sep, 13:28
Benjamin Bannier     Re: Review Request 71385: Added restart logic for failing resource providers. Wed, 04 Sep, 12:42
Benjamin Bannier   Re: Review Request 71385: Added restart logic for failing resource providers. Wed, 04 Sep, 12:42
Mesos Reviewbot     Re: Review Request 71385: Added restart logic for failing resource providers. Wed, 04 Sep, 21:01
Re: Review Request 68508: Added tracking of framework allocations in the allocator Slave class.
Meng Zhu   Re: Review Request 68508: Added tracking of framework allocations in the allocator Slave class. Tue, 03 Sep, 20:49
Andrei Sekretenko     Re: Review Request 68508: Added tracking of framework allocations in the allocator Slave class. Fri, 06 Sep, 17:06
Meng Zhu Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. Tue, 03 Sep, 20:50
Mesos Reviewbot   Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. Tue, 03 Sep, 23:37
Andrei Sekretenko   Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. Wed, 04 Sep, 13:23
Meng Zhu     Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. Thu, 05 Sep, 21:12
Andrei Sekretenko   Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. Fri, 06 Sep, 17:06
Review Request 71429: Precalculated the master per-framework metrics prefix.
James Peach   Review Request 71429: Precalculated the master per-framework metrics prefix. Wed, 04 Sep, 04:52
Meng Zhu   Re: Review Request 71429: Precalculated the master per-framework metrics prefix. Wed, 04 Sep, 22:02
Benjamin Bannier Review Request 71430: Included new Python CLI in distribution tarball. Wed, 04 Sep, 07:48
Mesos Reviewbot   Re: Review Request 71430: Included new Python CLI in distribution tarball. Wed, 04 Sep, 16:47
Till Toenshoff via Review Board   Re: Review Request 71430: Included new Python CLI in distribution tarball. Tue, 17 Sep, 15:18
Andrei Sekretenko Review Request 71433: Separated handling offer validation failure from handling success. Wed, 04 Sep, 17:52
Meng Zhu   Re: Review Request 71433: Separated handling offer validation failure from handling success. Wed, 04 Sep, 22:37
Andrei Sekretenko   Re: Review Request 71433: Separated handling offer validation failure from handling success. Fri, 06 Sep, 16:50
Meng Zhu     Re: Review Request 71433: Separated handling offer validation failure from handling success. Fri, 06 Sep, 19:33
Andrei Sekretenko Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. Wed, 04 Sep, 17:53
Meng Zhu   Re: Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. Thu, 05 Sep, 00:27
Andrei Sekretenko   Re: Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. Fri, 06 Sep, 16:55
Meng Zhu     Re: Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. Fri, 06 Sep, 19:40
Andrei Sekretenko Review Request 71435: Moved setting expectation for recoverResources() to a proper place. Wed, 04 Sep, 17:53
Meng Zhu   Re: Review Request 71435: Moved setting expectation for recoverResources() to a proper place. Thu, 05 Sep, 00:33
Andrei Sekretenko Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Wed, 04 Sep, 17:54
Meng Zhu   Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Wed, 04 Sep, 23:53
Andrei Sekretenko     Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Fri, 06 Sep, 17:03
Mesos Reviewbot   Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Thu, 05 Sep, 05:55
Andrei Sekretenko   Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Fri, 06 Sep, 16:56
Mesos Reviewbot     Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Fri, 06 Sep, 18:49
Meng Zhu     Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. Fri, 06 Sep, 21:13
Meng Zhu Review Request 71440: Improved allocator inverse offer test. Thu, 05 Sep, 21:12
Mesos Reviewbot   Re: Review Request 71440: Improved allocator inverse offer test. Thu, 05 Sep, 23:10
Andrei Sekretenko   Re: Review Request 71440: Improved allocator inverse offer test. Fri, 06 Sep, 13:37
Mesos Reviewbot   Re: Review Request 71440: Improved allocator inverse offer test. Thu, 12 Sep, 19:02
Benjamin Bannier   Re: Review Request 71440: Improved allocator inverse offer test. Mon, 30 Sep, 11:42
James Peach Review Request 71441: Fixed Docker registry malformed URI contruction. Fri, 06 Sep, 06:20
Mesos Reviewbot   Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Fri, 06 Sep, 09:24
Mesos Reviewbot   Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Mon, 09 Sep, 01:20
Andrei Budnik   Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Fri, 13 Sep, 15:39
James Peach     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Sun, 15 Sep, 00:10
Qian Zhang   Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Mon, 16 Sep, 05:24
James Peach     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Mon, 16 Sep, 05:51
Qian Zhang     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Mon, 16 Sep, 13:38
James Peach     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Tue, 17 Sep, 05:08
Qian Zhang     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Tue, 17 Sep, 07:45
James Peach     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Tue, 17 Sep, 22:17
Qian Zhang     Re: Review Request 71441: Fixed Docker registry malformed URI contruction. Wed, 18 Sep, 00:09
James Peach     Re: Review Request 71441: Fixed URI stringification. Thu, 19 Sep, 05:11
James Peach   Re: Review Request 71441: Fixed URI stringification. Thu, 19 Sep, 05:11
Mesos Reviewbot     Re: Review Request 71441: Fixed URI stringification. Thu, 19 Sep, 08:06
Qian Zhang     Re: Review Request 71441: Fixed URI stringification. Mon, 23 Sep, 06:41
Benjamin Bannier Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. Fri, 06 Sep, 12:39
Mesos Reviewbot   Re: Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. Fri, 06 Sep, 16:14
Benjamin Bannier   Re: Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. Thu, 12 Sep, 09:01
Jan Schlicht     Re: Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. Thu, 12 Sep, 09:07
Mesos Reviewbot     Re: Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. Thu, 12 Sep, 11:26
Benjamin Bannier Review Request 71450: Bumped Boost to 1.71.0. Mon, 09 Sep, 10:57
Benjamin Bannier Review Request 71451: Bumped Boost to 1.71.0. Mon, 09 Sep, 10:57
Benjamin Bannier Review Request 71452: Bumped Boost to 1.71.0. Mon, 09 Sep, 10:57
Mesos Reviewbot   Re: Review Request 71452: Bumped Boost to 1.71.0. Mon, 09 Sep, 11:18
Benjamin Bannier Review Request 71453: Punctually disabled pylint's `cyclic-import` warning. Mon, 09 Sep, 13:54
Jan Schlicht   Re: Review Request 71453: Punctually disabled pylint's `cyclic-import` warning. Mon, 09 Sep, 13:56
Re: Review Request 71300: Removed mesos-style transition script.
Mesos Reviewbot   Re: Review Request 71300: Removed mesos-style transition script. Mon, 09 Sep, 13:56
Mesos Reviewbot   Re: Review Request 71300: Removed mesos-style transition script. Tue, 10 Sep, 13:35
Re: Review Request 69411: Added new interface for constructing `cluster::Master`.
Benno Evers   Re: Review Request 69411: Added new interface for constructing `cluster::Master`. Mon, 09 Sep, 13:56
Mesos Reviewbot     Re: Review Request 69411: Added new interface for constructing `cluster::Master`. Mon, 09 Sep, 15:56
Mesos Reviewbot     Re: Review Request 69411: Added new interface for constructing `cluster::Master`. Mon, 09 Sep, 22:37
Andrei Budnik Review Request 71454: Added `futureTracker` to the `SlaveOptions` in tests. Mon, 09 Sep, 14:53
Qian Zhang   Re: Review Request 71454: Added `futureTracker` to the `SlaveOptions` in tests. Mon, 23 Sep, 02:28
Andrei Budnik     Re: Review Request 71454: Added `futureTracker` to the `SlaveOptions` in tests. Mon, 23 Sep, 12:01
Gilbert Song   Re: Review Request 71454: Added `futureTracker` to the `SlaveOptions` in tests. Mon, 23 Sep, 18:32
Message list1 · 2 · 3 · Next »Thread · Author · Date
Box list
Oct 201963
Sep 2019312
Aug 2019540
Jul 2019707
Jun 2019621
May 2019472
Apr 2019702
Mar 2019955
Feb 2019758
Jan 2019749
Dec 2018455
Nov 2018598
Oct 2018832
Sep 2018704
Aug 20181218
Jul 20181040
Jun 20181026
May 20181520
Apr 20181445
Mar 20181350
Feb 20181396
Jan 20181470
Dec 20171464
Nov 20171769
Oct 20171445
Sep 20171054
Aug 20171341
Jul 20171295
Jun 20171475
May 20171547
Apr 20171266
Mar 20172102
Feb 20171631
Jan 20171655
Dec 20161774
Nov 20162300
Oct 20162196
Sep 20162748
Aug 20162311
Jul 20163001
Jun 20163216
May 20162928
Apr 20163332
Mar 20164025
Feb 20162997
Jan 20163700
Dec 20152431
Nov 20152744
Oct 20152284
Sep 20152621
Aug 20152300
Jul 20152051
Jun 20152449
May 20151444
Apr 2015338