| Benjamin Bannier |
Review Request 71419: Fixed anchor name in upgrades guide. |
Mon, 02 Sep, 08:14 |
| Benjamin Bannier |
Review Request 71420: Fixed typo in reservation documentation. |
Mon, 02 Sep, 08:14 |
| Jan Schlicht |
Re: Review Request 71419: Fixed anchor name in upgrades guide. |
Mon, 02 Sep, 08:23 |
| 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 |
| 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 |
| Benjamin Bannier |
Re: Review Request 71390: Fixed the snapshot script to be non-interactive. |
Mon, 02 Sep, 11:19 |
| Mesos Reviewbot |
Re: Review Request 71421: Switched verify-reviews from docker-build to prepared Docker image. |
Mon, 02 Sep, 12:17 |
| Benjamin Bannier |
Re: Review Request 71391: Fixed sed arg to work on Linux and OSX. |
Mon, 02 Sep, 12:41 |
| Benjamin Bannier |
Re: Review Request 71392: Improved snapshot script to deduce the Mesos version. |
Mon, 02 Sep, 12:41 |
| Benjamin Bannier |
Re: Review Request 71393: Skiped GPG signing during `maven-install` in Makefile. |
Mon, 02 Sep, 12:44 |
| Benjamin Bannier |
Re: Review Request 71394: Skipped GPG signing for snapshot builds. |
Mon, 02 Sep, 12:45 |
| Benjamin Bannier |
Re: Review Request 71395: Implemented snapshot bot. |
Mon, 02 Sep, 12:50 |
| Mesos Reviewbot |
Re: Review Request 71423: Removed legacy docker-build script. |
Mon, 02 Sep, 13:06 |
| 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 |
| Jan Schlicht |
Re: Review Request 71382: Removed unused and unimplemented method declaration. |
Tue, 03 Sep, 12:50 |
| Jan Schlicht |
Re: Review Request 71383: Added stringification for resource provider calls. |
Tue, 03 Sep, 12:58 |
| Jan Schlicht |
Re: Review Request 71384: Passed down failures into resource provider fatal handler. |
Tue, 03 Sep, 13:02 |
| Benno Evers |
Re: Review Request 71240: Added a `dist` target to the cmake build. |
Tue, 03 Sep, 13:08 |
| Benno Evers |
Re: Review Request 71241: Added a `distcheck` target to the cmake build. |
Tue, 03 Sep, 13:08 |
| Jan Schlicht |
Re: Review Request 71384: Passed down failures into resource provider fatal handler. |
Tue, 03 Sep, 13:09 |
| Andrei Sekretenko |
Re: Review Request 71398: Made `allocated` mean "allocated but not offered" in `allocator::Slave`. |
Tue, 03 Sep, 13:16 |
| Andrei Sekretenko |
Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess. |
Tue, 03 Sep, 13:23 |
| Andrei Sekretenko |
Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess. |
Tue, 03 Sep, 13:23 |
| Andrei Sekretenko |
Re: Review Request 71400: Renamed `toAllocate` into `toOffer` in `HierarchialAllocatorProcess`. |
Tue, 03 Sep, 13:24 |
| Andrei Sekretenko |
Re: Review Request 71402: Renamed *Allocation into *Offering in __generateOffers(). |
Tue, 03 Sep, 13:26 |
| Jan Schlicht |
Re: Review Request 71385: Added restart logic for failing resource providers. |
Tue, 03 Sep, 13:28 |
| 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 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 71240: Added a `dist` target to the cmake build. |
Tue, 03 Sep, 14:45 |
| Benjamin Bannier |
Re: Review Request 71241: Added a `distcheck` target to the cmake build. |
Tue, 03 Sep, 14:51 |
| Mesos Reviewbot |
Re: Review Request 71402: Renamed *Allocation into *Offering in __generateOffers(). |
Tue, 03 Sep, 15:42 |
| Meng Zhu |
Re: Review Request 71398: Made `allocated` mean "allocated but not offered" in `allocator::Slave`. |
Tue, 03 Sep, 17:24 |
| Meng Zhu |
Re: Review Request 71399: Renamed `allocate` and its derivatives in HierarchialAllocatorProcess. |
Tue, 03 Sep, 17:25 |
| Mesos Reviewbot |
Re: Review Request 71241: Added a `distcheck` target to the cmake build. |
Tue, 03 Sep, 19:33 |
| Meng Zhu |
Re: Review Request 68508: Added tracking of framework allocations in the allocator Slave class. |
Tue, 03 Sep, 20:49 |
| 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 |
| James Peach |
Review Request 71429: Precalculated the master per-framework metrics prefix. |
Wed, 04 Sep, 04:52 |
| James Peach |
Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. |
Wed, 04 Sep, 05:52 |
| Benjamin Bannier |
Review Request 71430: Included new Python CLI in distribution tarball. |
Wed, 04 Sep, 07:48 |
| 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 |
| Andrei Sekretenko |
Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. |
Wed, 04 Sep, 13:23 |
| Mesos Reviewbot |
Re: Review Request 71430: Included new Python CLI in distribution tarball. |
Wed, 04 Sep, 16:47 |
| Andrei Sekretenko |
Review Request 71433: Separated handling offer validation failure from handling success. |
Wed, 04 Sep, 17:52 |
| Andrei Sekretenko |
Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. |
Wed, 04 Sep, 17:53 |
| Andrei Sekretenko |
Review Request 71435: Moved setting expectation for recoverResources() to a proper place. |
Wed, 04 Sep, 17:53 |
| Andrei Sekretenko |
Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Wed, 04 Sep, 17:54 |
| Mesos Reviewbot |
Re: Review Request 71385: Added restart logic for failing resource providers. |
Wed, 04 Sep, 21:01 |
| Meng Zhu |
Re: Review Request 71429: Precalculated the master per-framework metrics prefix. |
Wed, 04 Sep, 22:02 |
| Meng Zhu |
Re: Review Request 71433: Separated handling offer validation failure from handling success. |
Wed, 04 Sep, 22:37 |
| Meng Zhu |
Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Wed, 04 Sep, 23:53 |
| Meng Zhu |
Re: Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. |
Thu, 05 Sep, 00:27 |
| Meng Zhu |
Re: Review Request 71435: Moved setting expectation for recoverResources() to a proper place. |
Thu, 05 Sep, 00:33 |
| Mesos Reviewbot |
Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Thu, 05 Sep, 05:55 |
| 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 |
| Till Toenshoff via Review Board |
Re: Review Request 71241: Added a `distcheck` target to the cmake build. |
Thu, 05 Sep, 11:47 |
| Till Toenshoff via Review Board |
Re: Review Request 71390: Fixed the snapshot script to be non-interactive. |
Thu, 05 Sep, 11:51 |
| 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 |
| Till Toenshoff via Review Board |
Re: Review Request 71393: Skiped GPG signing during `maven-install` in Makefile. |
Thu, 05 Sep, 13:35 |
| Till Toenshoff via Review Board |
Re: Review Request 71394: Skipped GPG signing for snapshot builds. |
Thu, 05 Sep, 13:35 |
| 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 |
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 |
| Meng Zhu |
Review Request 71440: Improved allocator inverse offer test. |
Thu, 05 Sep, 21:12 |
| Meng Zhu |
Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. |
Thu, 05 Sep, 21:12 |
| Mesos Reviewbot |
Re: Review Request 71440: Improved allocator inverse offer test. |
Thu, 05 Sep, 23:10 |
| James Peach |
Review Request 71441: Fixed Docker registry malformed URI contruction. |
Fri, 06 Sep, 06:20 |
| James Peach |
Re: Review Request 71424: Added `SlaveOptions` for wrapping all parameters of `StartSlave`. |
Fri, 06 Sep, 06:23 |
| Mesos Reviewbot |
Re: Review Request 71441: Fixed Docker registry malformed URI contruction. |
Fri, 06 Sep, 09:24 |
| Benjamin Bannier |
Re: Review Request 71240: Added a `dist` target to the cmake build. |
Fri, 06 Sep, 10:13 |
| Benjamin Bannier |
Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. |
Fri, 06 Sep, 12:39 |
| Mesos Reviewbot |
Re: Review Request 71241: Added a `distcheck` target to the cmake build. |
Fri, 06 Sep, 12:42 |
| Andrei Sekretenko |
Re: Review Request 71440: Improved allocator inverse offer test. |
Fri, 06 Sep, 13:37 |
| Mesos Reviewbot |
Re: Review Request 71445: Removed race in `StorageLocalResourceProviderTest.Update`. |
Fri, 06 Sep, 16:14 |
| Andrei Sekretenko |
Re: Review Request 71433: Separated handling offer validation failure from handling success. |
Fri, 06 Sep, 16:50 |
| Andrei Sekretenko |
Re: Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. |
Fri, 06 Sep, 16:55 |
| Andrei Sekretenko |
Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Fri, 06 Sep, 16:56 |
| Andrei Sekretenko |
Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Fri, 06 Sep, 17:03 |
| Andrei Sekretenko |
Re: Review Request 68508: Added tracking of framework allocations in the allocator Slave class. |
Fri, 06 Sep, 17:06 |
| Andrei Sekretenko |
Re: Review Request 71428: Removed `Sorter::allocation(const SlaveID& slaveId)`. |
Fri, 06 Sep, 17:06 |
| Mesos Reviewbot |
Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Fri, 06 Sep, 18:49 |
| Meng Zhu |
Re: Review Request 71433: Separated handling offer validation failure from handling success. |
Fri, 06 Sep, 19:33 |
| Meng Zhu |
Re: Review Request 71434: Moved `removeOffers()` from `Master::accept()` into `Master::_accept()`. |
Fri, 06 Sep, 19:40 |
| Meng Zhu |
Re: Review Request 71436: Replaced removeOffer + recoverResources pairs with specialized helpers. |
Fri, 06 Sep, 21:13 |
| Mesos Reviewbot |
Re: Review Request 71441: Fixed Docker registry malformed URI contruction. |
Mon, 09 Sep, 01:20 |
| Benjamin Bannier |
Review Request 71451: Bumped Boost to 1.71.0. |
Mon, 09 Sep, 10:57 |
| Benjamin Bannier |
Review Request 71450: 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 |