| Benjamin Mahler |
Re: Review Request 57269: Added a test to ensure allocation roles are exposed in master API. |
Mon, 06 Mar, 22:38 |
| Greg Mann |
Re: Review Request 56754: Implemented a JWT secret generator. |
Mon, 06 Mar, 23:25 |
| Anindya Sinha |
Re: Review Request 55887: Check task user before allowing a task to be launched on the agent. |
Mon, 06 Mar, 23:30 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Mon, 06 Mar, 23:33 |
| Greg Mann |
Re: Review Request 56753: Implemented the JWT authenticator. |
Tue, 07 Mar, 00:02 |
| Michael Park |
Re: Review Request 57110: Updated the master to handle frameworks that changes its roles. |
Tue, 07 Mar, 00:07 |
| Michael Park |
Re: Review Request 57165: Minor cleanup for quota validation code. |
Tue, 07 Mar, 00:34 |
| Jiang Yan Xu |
Re: Review Request 55887: Check task user before allowing a task to be launched on the agent. |
Tue, 07 Mar, 00:50 |
| Vinod Kone |
Re: Review Request 57058: Updated default executor tests to exclusively use v1 protos. |
Tue, 07 Mar, 01:15 |
| Vinod Kone |
Re: Review Request 57059: Updated default executor tests. |
Tue, 07 Mar, 01:17 |
| Michael Park |
Review Request 57354: Added `initializer_list` constructors for `JSON::(Object|Array)`. |
Tue, 07 Mar, 01:33 |
| Michael Park |
Re: Review Request 57336: Updated `MultiRoleSchedulerUpgrade` to test framework updates. |
Tue, 07 Mar, 01:33 |
| Michael Park |
Re: Review Request 57336: Updated `MultiRoleSchedulerUpgrade` to test framework updates. |
Tue, 07 Mar, 01:34 |
| Michael Park |
Re: Review Request 57336: Updated `MultiRoleSchedulerUpgrade` to test framework updates. |
Tue, 07 Mar, 01:35 |
| Michael Park |
Re: Review Request 56805: Simplified interface for setting weights in allocator. |
Tue, 07 Mar, 01:38 |
| Adam B |
Re: Review Request 56805: Simplified interface for setting weights in allocator. |
Tue, 07 Mar, 01:44 |
| Joseph Wu |
Review Request 57358: Fixed potential for circular dependencies in Future continuations. |
Tue, 07 Mar, 02:50 |
| Mesos Reviewbot |
Re: Review Request 56733: Mount all supported subsystems in the containerizer tests. |
Tue, 07 Mar, 03:18 |
| Jay Guo |
Re: Review Request 57269: Added a test to ensure allocation roles are exposed in master API. |
Tue, 07 Mar, 03:35 |
| Jay Guo |
Re: Review Request 57269: Added a test to ensure allocation roles are exposed in master API. |
Tue, 07 Mar, 03:36 |
| Jay Guo |
Review Request 57360: Replace `.get().` in favor of `->` in tests. |
Tue, 07 Mar, 03:39 |
| Mesos Reviewbot |
Re: Review Request 57059: Updated default executor tests. |
Tue, 07 Mar, 05:17 |
| Michael Park |
Re: Review Request 57166: Updated role validation for hierarchical roles. |
Tue, 07 Mar, 05:26 |
| Jay Guo |
Review Request 57364: Fixed flaky test FaultToleranceTest.FrameworkReregister. |
Tue, 07 Mar, 06:48 |
| Greg Mann |
Re: Review Request 56757: Added the SecretGenerator module interface. |
Tue, 07 Mar, 07:13 |
| Avinash sridharan |
Re: Review Request 55790: Support the full CNI DNS specification. |
Tue, 07 Mar, 14:22 |
| Jan Schlicht |
Re: Review Request 56753: Implemented the JWT authenticator. |
Tue, 07 Mar, 15:10 |
| Neil Conway |
Re: Review Request 57364: Fixed flaky test FaultToleranceTest.FrameworkReregister. |
Tue, 07 Mar, 15:15 |
| Neil Conway |
Re: Review Request 57254: Updated DRFSorter to support hierarchical roles. |
Tue, 07 Mar, 15:26 |
| Jan Schlicht |
Re: Review Request 56666: Added a HMAC SHA256 generator. |
Tue, 07 Mar, 15:34 |
| Jan Schlicht |
Re: Review Request 56753: Implemented the JWT authenticator. |
Tue, 07 Mar, 15:36 |
| Jan Schlicht |
Re: Review Request 56754: Implemented a JWT secret generator. |
Tue, 07 Mar, 15:36 |
| Neil Conway |
Re: Review Request 57166: Updated role validation for hierarchical roles. |
Tue, 07 Mar, 15:37 |
| Neil Conway |
Re: Review Request 57166: Updated role validation for hierarchical roles. |
Tue, 07 Mar, 15:38 |
| Jan Schlicht |
Re: Review Request 56754: Implemented a JWT secret generator. |
Tue, 07 Mar, 15:40 |
| Jan Schlicht |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 15:48 |
| Jan Schlicht |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 15:51 |
| Jan Schlicht |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 15:57 |
| Alexander Rukletsov |
Re: Review Request 56208: Updated checks library with general check support. |
Tue, 07 Mar, 16:46 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 16:48 |
| James Peach |
Re: Review Request 55790: Support the full CNI DNS specification. |
Tue, 07 Mar, 17:01 |
| James Peach |
Re: Review Request 55790: Support the full CNI DNS specification. |
Tue, 07 Mar, 17:01 |
| Benjamin Bannier |
Review Request 57382: Removed unused lambda captures. |
Tue, 07 Mar, 17:04 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 17:08 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 17:11 |
| Gastón Kleiman |
Review Request 57384: Added `Containerizer::cleanupArtifacts`. |
Tue, 07 Mar, 17:13 |
| Gastón Kleiman |
Review Request 57385: Introduced proto changes needed for the DeleteNestedContainer API call. |
Tue, 07 Mar, 17:13 |
| Gastón Kleiman |
Review Request 57386: Introduced changes to the auth photos needed for DeleteNestedContainer. |
Tue, 07 Mar, 17:14 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the auth photos needed for DeleteNestedContainer. |
Tue, 07 Mar, 17:14 |
| Gastón Kleiman |
Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Tue, 07 Mar, 17:15 |
| Gastón Kleiman |
Review Request 57387: Implemented the Agent API call `DeleteNestedContainer`. |
Tue, 07 Mar, 17:16 |
| Gastón Kleiman |
Review Request 57388: Added documentation for the 'DeleteNestedContainer' Agent API call. |
Tue, 07 Mar, 17:17 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `DeleteNestedContainer`. |
Tue, 07 Mar, 17:17 |
| Avinash sridharan |
Re: Review Request 55790: Support the full CNI DNS specification. |
Tue, 07 Mar, 17:38 |
| Joseph Wu |
Re: Review Request 57358: WIP: Fixed potential for circular dependencies in Future continuations. |
Tue, 07 Mar, 18:09 |
| Mesos Reviewbot |
Re: Review Request 57360: Replace `.get().` in favor of `->` in tests. |
Tue, 07 Mar, 18:24 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 18:43 |
| Anindya Sinha |
Re: Review Request 53840: Added a metric in the allocator to track previous allocation cycle. |
Tue, 07 Mar, 18:52 |
| Jiang Yan Xu |
Re: Review Request 56733: Mount all supported subsystems in the containerizer tests. |
Tue, 07 Mar, 19:28 |
| Greg Mann |
Re: Review Request 56753: Implemented the JWT authenticator. |
Tue, 07 Mar, 19:36 |
| Greg Mann |
Re: Review Request 56754: Implemented a JWT secret generator. |
Tue, 07 Mar, 19:55 |
| Alexander Rukletsov |
Re: Review Request 56212: Added support for general checks to command executor. |
Tue, 07 Mar, 19:58 |
| Joseph Wu |
Re: Review Request 57358: Fixed FD leak in SSL server socket cleanup. |
Tue, 07 Mar, 20:18 |
| Joseph Wu |
Re: Review Request 57358: Fixed FD leak in SSL server socket cleanup. |
Tue, 07 Mar, 20:19 |
| Vinod Kone |
Re: Review Request 56665: Added a URL-safe base64 implementation. |
Tue, 07 Mar, 20:19 |
| James Peach |
Re: Review Request 56733: Mount all supported subsystems in the containerizer tests. |
Tue, 07 Mar, 20:19 |
| Alexander Rukletsov |
Re: Review Request 56016: Added a note about task status updates in scheduler and internal API. |
Tue, 07 Mar, 20:38 |
| Alexander Rukletsov |
Re: Review Request 56017: Added a helper for building a task status from an existing one. |
Tue, 07 Mar, 20:38 |
| Alexander Rukletsov |
Review Request 57394: Overloaded `<<` for `CheckInfo::Type`. |
Tue, 07 Mar, 20:38 |
| Alexander Rukletsov |
Re: Review Request 56208: Updated checks library with general check support. |
Tue, 07 Mar, 20:39 |
| Alexander Rukletsov |
Re: Review Request 57149: Added a warning if command executor gets an unknown acknowledgement. |
Tue, 07 Mar, 20:39 |
| Alexander Rukletsov |
Re: Review Request 56210: Reused previous task status for health updates in command executor. |
Tue, 07 Mar, 20:39 |
| Alexander Rukletsov |
Re: Review Request 56211: Renamed health checker in command executor for clarity. |
Tue, 07 Mar, 20:39 |
| Alexander Rukletsov |
Re: Review Request 56212: Added support for general checks to command executor. |
Tue, 07 Mar, 20:39 |
| Alexander Rukletsov |
Re: Review Request 57150: Simplified task id procurement in command executor. |
Tue, 07 Mar, 20:39 |
| Alexander Rukletsov |
Re: Review Request 56213: Added check tests for command executor. |
Tue, 07 Mar, 20:39 |
| Vinod Kone |
Re: Review Request 56666: Added a HMAC SHA256 generator. |
Tue, 07 Mar, 20:52 |
| Vinod Kone |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 21:37 |
| Vinod Kone |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 21:38 |
| Jiang Yan Xu |
Re: Review Request 56733: Mount all supported subsystems in the containerizer tests. |
Tue, 07 Mar, 22:05 |
| Aaron Wood via Review Board |
Re: Review Request 54993: Modify LevelDB patch to add ARM64/AArch64 support. |
Tue, 07 Mar, 22:06 |
| Jiang Yan Xu |
Re: Review Request 56730: Kill stray tasks when tearing down test cgroups. |
Tue, 07 Mar, 22:27 |
| Mesos Reviewbot |
Re: Review Request 57388: Added documentation for the 'DeleteNestedContainer' Agent API call. |
Tue, 07 Mar, 22:30 |
| Greg Mann |
Review Request 57395: Fixed a typo in libprocess 'Socket'. |
Tue, 07 Mar, 22:39 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 22:46 |
| Greg Mann |
Re: Review Request 56667: Added support for JSON Web Tokens. |
Tue, 07 Mar, 22:49 |
| Anindya Sinha |
Re: Review Request 55887: Check task user before allowing a task to be launched on the agent. |
Tue, 07 Mar, 23:19 |
| Anindya Sinha |
Re: Review Request 55887: Check task user before allowing a task to be launched on the agent. |
Tue, 07 Mar, 23:19 |
| Anindya Sinha |
Re: Review Request 55888: Test to ensure non-authorized users cannot launch tasks on agents. |
Tue, 07 Mar, 23:19 |
| Greg Mann |
Re: Review Request 57358: Fixed FD leak in SSL server socket cleanup. |
Tue, 07 Mar, 23:24 |
| Mesos Reviewbot |
Re: Review Request 55790: Support the full CNI DNS specification. |
Tue, 07 Mar, 23:31 |
| Gilbert Song |
Re: Review Request 57058: Updated default executor tests to exclusively use v1 protos. |
Tue, 07 Mar, 23:40 |
| Anindya Sinha |
Re: Review Request 53840: Metric in the allocator to track latency between allocation cycles. |
Tue, 07 Mar, 23:41 |
| Anindya Sinha |
Re: Review Request 53841: Added metrics for sorting of the sorters in the allocator. |
Tue, 07 Mar, 23:41 |
| Anindya Sinha |
Re: Review Request 53842: Add role specific metrics for sorting runs. |
Tue, 07 Mar, 23:41 |
| Anand Mazumdar |
Re: Review Request 57059: Updated default executor tests. |
Tue, 07 Mar, 23:46 |
| Anand Mazumdar |
Re: Review Request 57395: Fixed a typo in libprocess 'Socket'. |
Tue, 07 Mar, 23:49 |
| Gilbert Song |
Re: Review Request 57059: Updated default executor tests. |
Tue, 07 Mar, 23:58 |
| Joseph Wu |
Review Request 57399: Expanded the critical section inside `ProcessManager::spawn`. |
Wed, 08 Mar, 00:01 |
| Vinod Kone |
Re: Review Request 57059: Updated default executor tests. |
Wed, 08 Mar, 00:05 |