| Benjamin Bannier |
Re: Review Request 64151: Added a V1 API call to list resource providers. |
Fri, 08 Dec, 10:44 |
| Mesos Reviewbot Windows |
Re: Review Request 64439: Added `LocalResourceProviderDaemon` methods to modify configs. |
Fri, 08 Dec, 11:06 |
| Benjamin Bannier |
Re: Review Request 64422: Added explicit resource provider information to 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:32 |
| Benjamin Bannier |
Re: Review Request 64424: Removed 'total' from 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:32 |
| Benjamin Bannier |
Re: Review Request 64423: Explicitly passed resource-provider information in 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:32 |
| Benjamin Bannier |
Re: Review Request 64430: Only passed agent's resource version in top-level 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:32 |
| Benjamin Bannier |
Review Request 64445: Renamed resource categories in 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:32 |
| Benjamin Bannier |
Re: Review Request 64430: Only passed agent's resource version in top-level 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:33 |
| Benjamin Bannier |
Re: Review Request 64430: Only passed agent's resource version in top-level 'UpdateSlaveMessage'. |
Fri, 08 Dec, 11:33 |
| Mesos Reviewbot Windows |
Re: Review Request 63253: Fixed DefaultExecutorTest for proper URI handling on Windows. |
Fri, 08 Dec, 12:24 |
| Mesos Reviewbot Windows |
Re: Review Request 64430: Only passed agent's resource version in top-level 'UpdateSlaveMessage'. |
Fri, 08 Dec, 12:35 |
| Benjamin Bannier |
Review Request 64448: Added a missing include. |
Fri, 08 Dec, 12:52 |
| Jan Schlicht |
Re: Review Request 64448: Added a missing include. |
Fri, 08 Dec, 12:53 |
| Armand Grillet |
Re: Review Request 64379: Improved logs displayed after a slave failed recovery. |
Fri, 08 Dec, 13:23 |
| Jan Schlicht |
Re: Review Request 64381: Mesos: Used a namespace for UUID. |
Fri, 08 Dec, 13:43 |
| Jan Schlicht |
Re: Review Request 64168: Introduced a 'UUID' type. |
Fri, 08 Dec, 13:45 |
| Mesos Reviewbot Windows |
Re: Review Request 64432: Windows: Enabled remaining `FlagsTests`. |
Fri, 08 Dec, 14:16 |
| Jan Schlicht |
Re: Review Request 64380: Stout: Used a namespace for UUID. |
Fri, 08 Dec, 15:12 |
| Jan Schlicht |
Re: Review Request 64381: Mesos: Used a namespace for UUID. |
Fri, 08 Dec, 15:14 |
| Jan Schlicht |
Re: Review Request 64168: Introduced a 'UUID' type. |
Fri, 08 Dec, 15:15 |
| Benno Evers |
Review Request 64450: Update docs to mention new '--reconfiguration_policy' flag. |
Fri, 08 Dec, 15:19 |
| Mesos Reviewbot Windows |
Re: Review Request 64436: Toggled the authenticate_http_readwrite flag in MesosTest. |
Fri, 08 Dec, 15:39 |
| Michael Park |
Re: Review Request 63951: Added noexcept specifier to Option. |
Fri, 08 Dec, 16:09 |
| Alexander Rukletsov |
Review Request 64454: Fixed a typo in a gpu test. |
Fri, 08 Dec, 16:48 |
| Alexander Rukletsov |
Review Request 64455: Fixed a typo in a nested containerizer test. |
Fri, 08 Dec, 16:49 |
| Alexander Rukletsov |
Review Request 64456: Increased wait time for specific task updates. |
Fri, 08 Dec, 16:50 |
| Dmitry Zhuk |
Re: Review Request 63951: Added noexcept specifier to Option. |
Fri, 08 Dec, 16:51 |
| Benno Evers |
Re: Review Request 64454: Fixed a typo in a gpu test. |
Fri, 08 Dec, 16:56 |
| Benno Evers |
Re: Review Request 64455: Fixed a typo in a nested containerizer test. |
Fri, 08 Dec, 16:56 |
| Mesos Reviewbot Windows |
Re: Review Request 64445: Renamed resource categories in 'UpdateSlaveMessage'. |
Fri, 08 Dec, 17:04 |
| Benno Evers |
Re: Review Request 64454: Fixed a typo in a gpu test. |
Fri, 08 Dec, 17:13 |
| Benno Evers |
Re: Review Request 64454: Fixed a typo in a gpu test. |
Fri, 08 Dec, 17:13 |
| Benno Evers |
Re: Review Request 64456: Increased wait time for specific task updates. |
Fri, 08 Dec, 17:14 |
| Alexander Rukletsov |
Re: Review Request 64456: Increased wait time for specific task updates. |
Fri, 08 Dec, 17:40 |
| Alexander Rukletsov |
Re: Review Request 64379: Improved logs displayed after a slave failed recovery. |
Fri, 08 Dec, 17:44 |
| Alexander Rukletsov |
Re: Review Request 63953: Added logging based on container class. |
Fri, 08 Dec, 18:05 |
| Greg Mann |
Review Request 64457: Instructed frameworks to avoid setting the offer operation 'id' field. |
Fri, 08 Dec, 18:33 |
| Greg Mann |
Review Request 64458: Fixed a comment in the master. |
Fri, 08 Dec, 18:34 |
| Benjamin Bannier |
Re: Review Request 64458: Fixed a comment in the master. |
Fri, 08 Dec, 18:42 |
| Megha Sharma |
Re: Review Request 64098: Send status updates when agent re-registers. |
Fri, 08 Dec, 18:42 |
| Vinod Kone |
Re: Review Request 64457: Instructed frameworks to avoid setting the offer operation 'id' field. |
Fri, 08 Dec, 18:47 |
| Greg Mann |
Re: Review Request 64457: Instructed frameworks to avoid setting the offer operation 'id' field. |
Fri, 08 Dec, 18:53 |
| Jie Yu |
Re: Review Request 64457: Instructed frameworks to avoid setting the offer operation 'id' field. |
Fri, 08 Dec, 19:39 |
| Mesos Reviewbot Windows |
Re: Review Request 64379: Improved logs displayed after a slave failed recovery. |
Fri, 08 Dec, 20:30 |
| Andrew Schwartzmeyer |
Review Request 64461: Windows: Added legacy support for admin-only symlinks. |
Fri, 08 Dec, 21:18 |
| Greg Mann |
Review Request 64462: Edited some comments in the master for style and grammar. |
Fri, 08 Dec, 21:23 |
| Meng Zhu |
Re: Review Request 64304: Enforced quota limit in the presence of unallocated reservations. |
Fri, 08 Dec, 21:51 |
| Meng Zhu |
Review Request 64465: Added a test for non-quota role reservation allocation. |
Fri, 08 Dec, 21:53 |
| Meng Zhu |
Re: Review Request 64465: Added a test for non-quota role reservation allocation. |
Fri, 08 Dec, 21:54 |
| Meng Zhu |
Review Request 64466: Added a test to check shared resources accounting in quota enforcement. |
Fri, 08 Dec, 21:59 |
| Chun-Hung Hsiao |
Review Request 64468: SLRP reconciliation only preserves missing resources that are converted. |
Fri, 08 Dec, 21:59 |
| Chun-Hung Hsiao |
Re: Review Request 63901: Added operator API to update and remove resource provider configs. |
Fri, 08 Dec, 22:00 |
| Chun-Hung Hsiao |
Re: Review Request 64439: Added `LocalResourceProviderDaemon` methods to modify configs. |
Fri, 08 Dec, 22:00 |
| Chun-Hung Hsiao |
Review Request 64469: Added unit tests for resource provider config modification API. |
Fri, 08 Dec, 22:01 |
| Mesos Reviewbot Windows |
Re: Review Request 64168: Introduced a 'UUID' type. |
Fri, 08 Dec, 22:07 |
| Meng Zhu |
Review Request 64467: Rewrote the quota headroom enforcement logic in the allocator. |
Fri, 08 Dec, 22:13 |
| Meng Zhu |
Re: Review Request 64466: Added a test to check shared resources accounting in quota enforcement. |
Fri, 08 Dec, 22:15 |
| Meng Zhu |
Re: Review Request 64465: Added a test for non-quota role reservation allocation. |
Fri, 08 Dec, 22:16 |
| Meng Zhu |
Review Request 64470: Clarified comments in the allocator. |
Fri, 08 Dec, 22:50 |
| Mesos Reviewbot Windows |
Re: Review Request 64450: Update docs to mention new '--reconfiguration_policy' flag. |
Fri, 08 Dec, 23:34 |
| Greg Mann |
Review Request 64463: Added the OFFER_OPERATION_DROPPED state. |
Fri, 08 Dec, 23:40 |
| Greg Mann |
Review Request 64464: Made master reconcile known offer operations with agent. |
Fri, 08 Dec, 23:46 |
| Benjamin Mahler |
Re: Review Request 64470: Clarified comments in the allocator. |
Fri, 08 Dec, 23:57 |
| Benjamin Mahler |
Re: Review Request 64466: Added a test to check shared resources accounting in quota enforcement. |
Fri, 08 Dec, 23:59 |
| Chun-Hung Hsiao |
Review Request 64474: Changed `framework_id` to optional for offer oporations in RP API. |
Sat, 09 Dec, 00:06 |
| Chun-Hung Hsiao |
Review Request 64475: Initialized offer operation status update manager in SLRP. |
Sat, 09 Dec, 00:06 |
| Benjamin Mahler |
Re: Review Request 64303: Tracked resource reservations in the allocator. |
Sat, 09 Dec, 00:18 |
| Gaston Kleiman |
Review Request 64472: Improved the logging in `StatusUpdateManagerProcess`. |
Sat, 09 Dec, 00:25 |
| Gaston Kleiman |
Review Request 64473: Fixed naming of a variable in `status_update_manager_process.hpp`. |
Sat, 09 Dec, 00:25 |
| Andrew Schwartzmeyer |
Re: Review Request 60620: Modifed os::write to write binary files on Windows. |
Sat, 09 Dec, 00:25 |
| Andrew Schwartzmeyer |
Re: Review Request 60620: Modifed os::write to write binary files on Windows. |
Sat, 09 Dec, 00:26 |
| Andrew Schwartzmeyer |
Re: Review Request 60621: Added new stout capability: os::copyfile(source, dest). |
Sat, 09 Dec, 00:26 |
| Meng Zhu |
Re: Review Request 64467: Rewrote the quota headroom enforcement logic in the allocator. |
Sat, 09 Dec, 00:26 |
| Andrew Schwartzmeyer |
Re: Review Request 60622: Added new stout functions for path and URI conversions. |
Sat, 09 Dec, 00:28 |
| Andrew Schwartzmeyer |
Re: Review Request 60623: Converted "file://" URI handling to use new uri function. |
Sat, 09 Dec, 00:28 |
| Andrew Schwartzmeyer |
Re: Review Request 60624: Enabled HDFS compilation and associated tests. |
Sat, 09 Dec, 00:28 |
| Andrew Schwartzmeyer |
Re: Review Request 60626: Eliminated os::shell calls from HDFS for Windows compatibility. |
Sat, 09 Dec, 00:28 |
| Andrew Schwartzmeyer |
Re: Review Request 60628: Enabled fetcher_tests.cpp unit test module on Windows platform. |
Sat, 09 Dec, 00:30 |
| Mesos Reviewbot Windows |
Re: Review Request 64456: Increased wait time for specific task updates. |
Sat, 09 Dec, 00:42 |
| Akash Gupta |
Re: Review Request 64431: Updated Windows documentation. |
Sat, 09 Dec, 01:39 |
| Akash Gupta |
Re: Review Request 64432: Windows: Enabled remaining `FlagsTests`. |
Sat, 09 Dec, 01:44 |
| Akash Gupta |
Re: Review Request 64432: Windows: Enabled remaining `FlagsTests`. |
Sat, 09 Dec, 01:44 |
| Akash Gupta |
Re: Review Request 64461: Windows: Added legacy support for admin-only symlinks. |
Sat, 09 Dec, 01:47 |
| Gaston Kleiman |
Re: Review Request 64463: Added the OFFER_OPERATION_DROPPED state. |
Sat, 09 Dec, 01:53 |
| Gaston Kleiman |
Re: Review Request 64464: Made master reconcile known offer operations with agent. |
Sat, 09 Dec, 01:57 |
| Jie Yu |
Review Request 64477: Refactored agent to keep track of local resource providers. |
Sat, 09 Dec, 02:01 |
| Jie Yu |
Re: Review Request 64441: Removed a CHECK that should not be there in SLRP. |
Sat, 09 Dec, 02:04 |
| Jie Yu |
Re: Review Request 64437: Added missing filesystem isolation for SLRP tests. |
Sat, 09 Dec, 02:04 |
| Benjamin Mahler |
Re: Review Request 64304: Enforced quota limit in the presence of unallocated reservations. |
Sat, 09 Dec, 02:08 |
| Jie Yu |
Re: Review Request 64468: SLRP reconciliation only preserves missing resources that are converted. |
Sat, 09 Dec, 02:11 |
| Jie Yu |
Re: Review Request 64440: Added `MODIFY_RESOURCE_PROVIDER_CONFIG` authorization. |
Sat, 09 Dec, 02:14 |
| Joseph Wu |
Re: Review Request 64353: Added example VolumeProfile module. |
Sat, 09 Dec, 02:17 |
| Jie Yu |
Re: Review Request 64438: Implemented API for modifying resource provider configs. |
Sat, 09 Dec, 02:19 |
| Andrew Schwartzmeyer |
Re: Review Request 64461: Windows: Added legacy support for admin-only symlinks. |
Sat, 09 Dec, 03:02 |
| Andrew Schwartzmeyer |
Re: Review Request 64432: Windows: Enabled remaining `FlagsTests`. |
Sat, 09 Dec, 03:02 |
| Jie Yu |
Re: Review Request 64439: Added `LocalResourceProviderDaemon` methods to modify configs. |
Sat, 09 Dec, 03:30 |
| Chun-Hung Hsiao |
Review Request 64479: Added a test for `MODIFY_RESOURE_PROVIDER_CONFIG` authorization. |
Sat, 09 Dec, 15:28 |
| Jie Yu |
Re: Review Request 64477: Refactored agent to keep track of local resource providers. |
Sun, 10 Dec, 17:11 |
| Jie Yu |
Review Request 64487: Passed the message directly to the Master::registerSlave handler. |
Sun, 10 Dec, 17:12 |
| Jie Yu |
Re: Review Request 64487: Passed the message directly to the Master::registerSlave handler. |
Sun, 10 Dec, 17:12 |