| Jie Yu |
Re: Review Request 64226: Added a `ns::supported` convenience API. |
Fri, 01 Dec, 00:07 |
| Gaston Kleiman |
Re: Review Request 64095: Added a generic actor to be used by status update managers. |
Fri, 01 Dec, 00:10 |
| James Peach |
Re: Review Request 64226: Added a `ns::supported` convenience API. |
Fri, 01 Dec, 00:17 |
| Akash Gupta |
Re: Review Request 63276: Windows: Added `Cpu` and `Mem` isolators. |
Fri, 01 Dec, 00:25 |
| Gaston Kleiman |
Re: Review Request 64096: Implemented the `OfferOperationStatusUpdateManager`. |
Fri, 01 Dec, 00:27 |
| Michael Park |
Re: Review Request 64194: Stripped AllocationInfo from offers to non-MULTI_ROLE schedulers. |
Fri, 01 Dec, 00:54 |
| Benjamin Mahler |
Review Request 64236: Documented when ReregisterSlaveMessage.frameworks was introduced. |
Fri, 01 Dec, 01:03 |
| Benjamin Mahler |
Review Request 64237: Removed a stale TODO from before frameworks could modify their roles. |
Fri, 01 Dec, 01:04 |
| Benjamin Mahler |
Review Request 64238: Updated the allocator to track allocations via a single code path. |
Fri, 01 Dec, 01:05 |
| Jie Yu |
Re: Review Request 63276: Windows: Added `Cpu` and `Mem` isolators. |
Fri, 01 Dec, 01:13 |
| Jie Yu |
Re: Review Request 63277: Windows: Ported `Cpu` and `Mem` isolator tests. |
Fri, 01 Dec, 01:15 |
| Michael Park |
Re: Review Request 64195: Updated the tests to use MULTI_ROLE frameworks by default. |
Fri, 01 Dec, 01:16 |
| Jie Yu |
Re: Review Request 64236: Documented when ReregisterSlaveMessage.frameworks was introduced. |
Fri, 01 Dec, 01:25 |
| Chun-Hung Hsiao |
Re: Review Request 63823: Initialized and subscribed storage local resource provider. |
Fri, 01 Dec, 01:34 |
| Chun-Hung Hsiao |
Re: Review Request 63021: Added `getService()` function to launch CSI plugins. |
Fri, 01 Dec, 01:36 |
| Chun-Hung Hsiao |
Re: Review Request 64044: Recover controller and node services and clean up unused containers. |
Fri, 01 Dec, 01:38 |
| Gaston Kleiman |
Re: Review Request 64095: Added a generic actor to be used by status update managers. |
Fri, 01 Dec, 01:42 |
| Gaston Kleiman |
Re: Review Request 64095: Added a generic actor to be used by status update managers. |
Fri, 01 Dec, 01:42 |
| Chun-Hung Hsiao |
Re: Review Request 64075: Added the `ResourceProviderState` protobuf for resource providers. |
Fri, 01 Dec, 01:47 |
| Jie Yu |
Re: Review Request 63860: Windows: Mapped the Docker network info types. |
Fri, 01 Dec, 01:47 |
| Andrew Schwartzmeyer |
Re: Review Request 64102: Renamed curl target to libcurl, and staging of curl.exe on Windows. |
Fri, 01 Dec, 01:48 |
| Andrew Schwartzmeyer |
Re: Review Request 64102: Renamed curl target to libcurl, and staging of curl.exe on Windows. |
Fri, 01 Dec, 01:50 |
| Chun-Hung Hsiao |
Re: Review Request 63022: Imported and reconcile resources from CSI plugins. |
Fri, 01 Dec, 01:52 |
| Chun-Hung Hsiao |
Re: Review Request 63386: Added utility functions to print types in resource provider events. |
Fri, 01 Dec, 01:53 |
| Chun-Hung Hsiao |
Re: Review Request 63387: Added publish/unpublish in storage local resource provider. |
Fri, 01 Dec, 01:55 |
| Andrew Schwartzmeyer |
Re: Review Request 64103: Changed dependency of curl to libcurl for stout. |
Fri, 01 Dec, 02:01 |
| Andrew Schwartzmeyer |
Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests. |
Fri, 01 Dec, 02:04 |
| Andrew Schwartzmeyer |
Re: Review Request 63859: Windows: Fixed mock signal values in stout. |
Fri, 01 Dec, 02:14 |
| Andrew Schwartzmeyer |
Re: Review Request 63862: Windows: Ported docker_tests.cpp. |
Fri, 01 Dec, 02:26 |
| Andrew Schwartzmeyer |
Re: Review Request 63862: Windows: Ported docker_tests.cpp. |
Fri, 01 Dec, 02:33 |
| Jie Yu |
Re: Review Request 63823: Initialized and subscribed storage local resource provider. |
Fri, 01 Dec, 03:00 |
| Michael Park |
Re: Review Request 63628: Implemented index_sequence and related functionality. |
Fri, 01 Dec, 03:44 |
| Jan Schlicht |
Re: Review Request 64065: Allowed resubscription of resource providers. |
Fri, 01 Dec, 08:45 |
| Alexander Rojas |
Re: Review Request 60891: Added ACLs and AuthZ for standalone containers. |
Fri, 01 Dec, 09:30 |
| Benjamin Bannier |
Re: Review Request 64158: Used helper functions instead of switches for resource extraction. |
Fri, 01 Dec, 11:03 |
| Benjamin Bannier |
Re: Review Request 64065: Allowed resubscription of resource providers. |
Fri, 01 Dec, 11:03 |
| Joseph Wu |
Re: Review Request 60891: Added ACLs and AuthZ for standalone containers. |
Fri, 01 Dec, 11:20 |
| Mesos Reviewbot Windows |
Re: Review Request 63862: Windows: Ported docker_tests.cpp. |
Fri, 01 Dec, 11:25 |
| Mesos Reviewbot Windows |
Re: Review Request 64226: Added a `ns::supported` convenience API. |
Fri, 01 Dec, 11:25 |
| Mesos Reviewbot Windows |
Re: Review Request 64065: Allowed resubscription of resource providers. |
Fri, 01 Dec, 12:19 |
| Mesos Reviewbot Windows |
Re: Review Request 64096: Implemented the `OfferOperationStatusUpdateManager`. |
Fri, 01 Dec, 12:20 |
| Benjamin Bannier |
Re: Review Request 64010: Added additional parameters to Allocator::updateSlave(). |
Fri, 01 Dec, 12:24 |
| Mesos Reviewbot Windows |
Re: Review Request 64238: Updated the allocator to track allocations via a single code path. |
Fri, 01 Dec, 13:15 |
| Mesos Reviewbot Windows |
Re: Review Request 64195: Updated the tests to use MULTI_ROLE frameworks by default. |
Fri, 01 Dec, 13:16 |
| Mesos Reviewbot Windows |
Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests. |
Fri, 01 Dec, 13:17 |
| Alexander Rukletsov |
Re: Review Request 63830: Fixed 'NoOffersWithAllRolesSuppressed' test. |
Fri, 01 Dec, 14:28 |
| Alexander Rukletsov |
Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter. |
Fri, 01 Dec, 14:45 |
| Jan Schlicht |
Review Request 64247: Fixed a flaky test case in reservation tests. |
Fri, 01 Dec, 14:46 |
| Michael Park |
Review Request 64248: Added `cpp17::invoke` in <stout/cpp17.hpp>. |
Fri, 01 Dec, 15:36 |
| Mesos Reviewbot Windows |
Re: Review Request 64247: Fixed a flaky test case in reservation tests. |
Fri, 01 Dec, 15:44 |
| Mesos Reviewbot Windows |
Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter. |
Fri, 01 Dec, 15:44 |
| Alexander Rukletsov |
Re: Review Request 63741: Fixed a bug in devolving framework subscription with suppressed roles. |
Fri, 01 Dec, 15:58 |
| Megha Sharma |
Review Request 64250: Added new reasons task status update. |
Fri, 01 Dec, 16:20 |
| Alexander Rukletsov |
Re: Review Request 63741: Fixed a bug in devolving framework subscription with suppressed roles. |
Fri, 01 Dec, 16:21 |
| Alexander Rukletsov |
Re: Review Request 63830: Fixed 'NoOffersWithAllRolesSuppressed' test. |
Fri, 01 Dec, 16:21 |
| Alexander Rukletsov |
Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter. |
Fri, 01 Dec, 16:22 |
| Mesos Reviewbot Windows |
Re: Review Request 64248: Added `cpp17::invoke` in <stout/cpp17.hpp>. |
Fri, 01 Dec, 16:44 |
| Michael Park |
Re: Review Request 63630: Added support for callable once functors. |
Fri, 01 Dec, 17:03 |
| Michael Park |
Re: Review Request 63631: Separated event visiting and consumption. |
Fri, 01 Dec, 17:13 |
| Michael Park |
Re: Review Request 63632: Migrated to event consumer interface. |
Fri, 01 Dec, 17:25 |
| Mesos Reviewbot Windows |
Re: Review Request 64098: Send status updates when agent re-registers. |
Fri, 01 Dec, 17:44 |
| Andrew Schwartzmeyer |
Re: Review Request 63862: Windows: Ported docker_tests.cpp. |
Fri, 01 Dec, 17:52 |
| Michael Park |
Re: Review Request 63636: Added placeholder implementation. |
Fri, 01 Dec, 17:54 |
| Greg Mann |
Re: Review Request 64096: Implemented the `OfferOperationStatusUpdateManager`. |
Fri, 01 Dec, 18:04 |
| Greg Mann |
Re: Review Request 64096: Implemented the `OfferOperationStatusUpdateManager`. |
Fri, 01 Dec, 18:16 |
| Gilbert Song |
Re: Review Request 64232: Add a temporary filter for overlay backend related tests. |
Fri, 01 Dec, 18:26 |
| Benno Evers |
Review Request 64252: Convert resource format of messages entering master. |
Fri, 01 Dec, 18:30 |
| Benno Evers |
Re: Review Request 64010: Added additional parameters to Allocator::updateSlave(). |
Fri, 01 Dec, 18:31 |
| Benno Evers |
Re: Review Request 64011: Updated master behaviour to update agent state on reregistration. |
Fri, 01 Dec, 18:31 |
| Benno Evers |
Re: Review Request 64008: Activated AGENT_UPDATE master capability. |
Fri, 01 Dec, 18:32 |
| Benno Evers |
Re: Review Request 64012: Added new --reconfiguration_compatibility slave flag and implementation. |
Fri, 01 Dec, 18:32 |
| Benno Evers |
Re: Review Request 64009: Added new UpdateSlave registry operation. |
Fri, 01 Dec, 18:38 |
| Vinod Kone |
Re: Review Request 64252: Convert resource format of messages entering master. |
Fri, 01 Dec, 18:57 |
| Vinod Kone |
Re: Review Request 64009: Added new UpdateSlave registry operation. |
Fri, 01 Dec, 19:12 |
| Vinod Kone |
Re: Review Request 64010: Added additional parameters to Allocator::updateSlave(). |
Fri, 01 Dec, 19:23 |
| Mesos Reviewbot Windows |
Re: Review Request 64232: Add a temporary filter for overlay backend related tests. |
Fri, 01 Dec, 19:24 |
| Jie Yu |
Re: Review Request 63021: Added `getService()` function to launch CSI plugins. |
Fri, 01 Dec, 19:24 |
| Vinod Kone |
Re: Review Request 64010: Added additional parameters to Allocator::updateSlave(). |
Fri, 01 Dec, 19:28 |
| Vinod Kone |
Re: Review Request 64011: Updated master behaviour to update agent state on reregistration. |
Fri, 01 Dec, 19:30 |
| Vinod Kone |
Re: Review Request 64008: Activated AGENT_UPDATE master capability. |
Fri, 01 Dec, 19:33 |
| Jiang Yan Xu |
Re: Review Request 64250: Added new reasons task status update. |
Fri, 01 Dec, 19:36 |
| Jie Yu |
Re: Review Request 63021: Added `getService()` function to launch CSI plugins. |
Fri, 01 Dec, 19:37 |
| Vinod Kone |
Re: Review Request 64012: Added new --reconfiguration_compatibility slave flag and implementation. |
Fri, 01 Dec, 19:47 |
| Greg Mann |
Re: Review Request 64095: Added a generic actor to be used by status update managers. |
Fri, 01 Dec, 19:54 |
| Andrew Schwartzmeyer |
Review Request 64255: Removed `os/realpath.hpp` from `posix/os.hpp`. |
Fri, 01 Dec, 20:10 |
| Andrew Schwartzmeyer |
Review Request 64256: Included `stout/os/realpath.hpp` where `os::realpath()` is used. |
Fri, 01 Dec, 20:11 |
| Andrew Schwartzmeyer |
Review Request 64257: Included `stout/os/realpath.hpp` where `os::realpath()` is used. |
Fri, 01 Dec, 20:11 |
| Andrew Schwartzmeyer |
Review Request 64258: Included `stout/os/permissions.hpp` for the secret generator. |
Fri, 01 Dec, 20:13 |
| Meng Zhu |
Re: Review Request 64232: Add a temporary filter for overlay backend related tests. |
Fri, 01 Dec, 21:02 |
| Megha Sharma |
Re: Review Request 64250: Added new reasons task status update. |
Fri, 01 Dec, 21:06 |
| Greg Mann |
Re: Review Request 64093: Added operators for offer operation update protobuf classes. |
Fri, 01 Dec, 21:27 |
| Jiang Yan Xu |
Re: Review Request 64098: Send status updates when agent re-registers. |
Fri, 01 Dec, 21:29 |
| Akash Gupta |
Re: Review Request 64104: Added dependency of curl to agent. Enabled most health check tests. |
Fri, 01 Dec, 21:34 |
| Akash Gupta |
Re: Review Request 64102: Renamed curl target to libcurl, and staging of curl.exe on Windows. |
Fri, 01 Dec, 21:35 |
| Akash Gupta |
Re: Review Request 64103: Changed dependency of curl to libcurl for stout. |
Fri, 01 Dec, 21:35 |
| Megha Sharma |
Re: Review Request 64098: Send status updates when agent re-registers. |
Fri, 01 Dec, 21:37 |
| Jiang Yan Xu |
Re: Review Request 64250: Added new reasons task status update. |
Fri, 01 Dec, 21:45 |
| Akash Gupta |
Re: Review Request 63860: Windows: Mapped the Docker network info types. |
Fri, 01 Dec, 21:50 |
| Vinod Kone |
Re: Review Request 63765: Fixed an issue with the scheduler driver subscribe backoff time. |
Fri, 01 Dec, 21:54 |
| Ilya Pronin |
Re: Review Request 64250: Added new reasons task status update. |
Fri, 01 Dec, 21:55 |