You are viewing a plain text version of this content. The canonical link for it is here.
- Re: Review Request 62775: Added a test `ROOT_NoTransitionFromKillingToFinished`. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/01 00:02:21 UTC, 4 replies.
- Re: Review Request 63331: Added documentation for the `network/ports` isolator. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/01 00:47:33 UTC, 1 replies.
- Re: Review Request 63447: Fixed bulleted list formatting. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/01 02:39:02 UTC, 0 replies.
- Re: Review Request 63353: Don't clear the executor ID of non-command executors on re-registration. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/01 04:43:39 UTC, 4 replies.
- Re: Review Request 63453: Patched concurrentqueue to check for libstdc++ instead of GCC. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/01 06:39:58 UTC, 1 replies.
- Re: Review Request 63414: Added a helper to get the resoruce provider ID from an offer operation. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 14:29:39 UTC, 2 replies.
- Review Request 63463: Added support for credential secret to test-http-framework. - posted by Till Toenshoff <to...@me.com> on 2017/11/01 14:39:08 UTC, 2 replies.
- Re: Review Request 62837: Added a test `DefaultExecutorTest.KillMultipleTasks`. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/01 15:00:25 UTC, 6 replies.
- Review Request 63468: Added TERMINATED state to processes. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/11/01 15:17:22 UTC, 3 replies.
- Re: Review Request 62774: Added a test `ROOT_DOCKER_NoTransitionFromKillingToFinished`. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/01 16:47:34 UTC, 1 replies.
- Re: Review Request 62616: Binary packaging for Centos 7. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/01 18:31:29 UTC, 0 replies.
- Review Request 63474: Added a regression test for MESOS-8135. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/01 20:54:02 UTC, 6 replies.
- Review Request 63476: Revert "Added a test `ROOT_NoTransitionFromKillingToFinished`". - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/01 21:04:50 UTC, 4 replies.
- Review Request 63472: Replaced `concurrentqueue-1.0.0-beta` with `concurrentqueue-7b69a8f`. - posted by Michael Park <mp...@apache.org> on 2017/11/01 21:45:48 UTC, 5 replies.
- Review Request 63479: Migrated `3rdparty/mesos-3rdparty` to `mesos/3rdparty`. - posted by Michael Park <mp...@apache.org> on 2017/11/01 21:52:11 UTC, 3 replies.
- Re: Review Request 63174: Added a benchmark for agent reregistration during master failover. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/01 22:06:32 UTC, 5 replies.
- Re: Review Request 63413: Added streaming functions for offer operation ID. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:10:16 UTC, 0 replies.
- Review Request 63480: Added the initial implementation for applying offer operations. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:14:01 UTC, 3 replies.
- Re: Review Request 63432: Updated the comment about slaves.removed in master. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:14:30 UTC, 1 replies.
- Review Request 63481: Added latest_status field to OfferOperation protobuf. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:15:35 UTC, 2 replies.
- Review Request 63482: Added streaming function for OfferOperationState. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:16:17 UTC, 2 replies.
- Review Request 63483: Added some protobuf helper for offer operations. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:17:02 UTC, 2 replies.
- Review Request 63484: Changed slave_id field to be optional in OfferOperationStatusUpdate. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:17:36 UTC, 2 replies.
- Review Request 63485: Added initial code for offer operation status update in master. - posted by Jie Yu <yu...@gmail.com> on 2017/11/01 23:18:42 UTC, 7 replies.
- Review Request 63491: Added resource version to resource provider UpdateTotalResources call. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:05 UTC, 10 replies.
- Re: Review Request 61183: Triggered 'UpdateSlaveMessage' when 'ResourceProviderManager' updates. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:06 UTC, 9 replies.
- Review Request 63492: Synchronized agent clock with master via 'UpdateSlaveMessage'. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:06 UTC, 8 replies.
- Review Request 63493: Transmitted agent resource versions in (re)registration. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:08 UTC, 6 replies.
- Review Request 63494: Removed unused declaration. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:09 UTC, 1 replies.
- Review Request 63495: Added comparison operators for 'ResourceVerionUUID'. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:09 UTC, 2 replies.
- Review Request 63496: Added tests for agent resource version transmission. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 01:57:10 UTC, 16 replies.
- Review Request 63497: Fixed the unit test that missed to check the TASK_STARTING update. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/02 03:16:39 UTC, 3 replies.
- Re: Review Request 63389: WIP: Added a mock resource provider manager. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/02 03:28:08 UTC, 2 replies.
- Review Request 63499: Fixed the unit test that broke Windows build. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/02 05:27:13 UTC, 13 replies.
- Review Request 63501: Initial `3rdparty/README.md`. - posted by Michael Park <mp...@apache.org> on 2017/11/02 07:54:35 UTC, 3 replies.
- Review Request 63511: Introduced ResourceConversion to represent conversion to Resources. - posted by Jie Yu <yu...@gmail.com> on 2017/11/02 13:54:26 UTC, 5 replies.
- Re: Review Request 60888: Added recovery logic for standalone containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 15:43:44 UTC, 3 replies.
- Re: Review Request 60889: Added basic tests for launching standalone containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 15:45:19 UTC, 0 replies.
- Re: Review Request 60890: Defined API for launching standalone containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 15:50:50 UTC, 4 replies.
- Re: Review Request 62145: Implemented Standalone Container API. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 15:57:36 UTC, 4 replies.
- Re: Review Request 63056: Parameterized test for nested container launch. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 16:00:22 UTC, 5 replies.
- Re: Review Request 63057: Moved and refactored some nested container tests. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 16:00:59 UTC, 1 replies.
- Re: Review Request 63058: Changed failure response of LAUNCH containers API. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 16:01:11 UTC, 1 replies.
- Re: Review Request 63063: Modified Containerizer::launch interface to allow repeated launch. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 16:02:45 UTC, 3 replies.
- Review Request 63514: Updated tests with Containerizer::launch interface change. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 16:04:02 UTC, 2 replies.
- Review Request 63515: Updated xfs/disk, gpu, and cgroups isolators' prepare function. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/02 16:04:44 UTC, 2 replies.
- Review Request 63519: Allowed toggling of agent capabilities via command line flags. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/02 17:47:34 UTC, 13 replies.
- Review Request 63522: Removed unused variable `DefaultExecutorTest.CommitSuicideOnKillTask`. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/02 18:22:44 UTC, 3 replies.
- Re: Review Request 63023: WIP: Added a test CSI plugin. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/02 18:27:46 UTC, 0 replies.
- Review Request 63520: Added building the `stout-tests` target to `mesos-tidy`. - posted by Michael Park <mp...@apache.org> on 2017/11/02 18:47:42 UTC, 3 replies.
- Re: Review Request 63434: Added IPv6 capabilities for TCP and HTTP healthchecks. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/02 18:48:51 UTC, 9 replies.
- Review Request 63523: Updated `Allocator::updateAllocation` to take `ResourceConversion`s. - posted by Jie Yu <yu...@gmail.com> on 2017/11/02 19:36:19 UTC, 7 replies.
- Re: Review Request 63353: Don't clear the executor ID of executors on re-registration. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/02 19:52:28 UTC, 1 replies.
- Re: Review Request 63422: Updated os::unsetenv to clear the old environment value. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/02 20:23:42 UTC, 1 replies.
- Re: Review Request 63268: Windows: Fixed variable casing in `windows/os.hpp`. - posted by Aaron Wood via Review Board <no...@reviews.apache.org> on 2017/11/02 20:51:31 UTC, 4 replies.
- Re: Review Request 63273: Windows: Added `os::get_job_processes` to stout. - posted by Aaron Wood via Review Board <no...@reviews.apache.org> on 2017/11/02 21:18:28 UTC, 1 replies.
- Re: Review Request 63278: Windows: Documented the `cpuset` and `mem` isolators. - posted by Aaron Wood via Review Board <no...@reviews.apache.org> on 2017/11/02 21:48:33 UTC, 4 replies.
- Re: Review Request 63279: Increased check tests task resources for Windows. - posted by Aaron Wood via Review Board <no...@reviews.apache.org> on 2017/11/02 21:50:21 UTC, 7 replies.
- Review Request 63526: Fixed TASK_UNKNOWN status ambiguity. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/11/02 21:53:03 UTC, 2 replies.
- Review Request 63527: Added MESOS-8165 to the 1.5.0 CHANGELOG. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/11/02 21:53:10 UTC, 7 replies.
- Review Request 63529: Added missing lock free event queue and LIFO CMake options. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/02 22:54:38 UTC, 1 replies.
- Re: Review Request 56721: Implemented pruneImages with a mark and sweep in docker store. - posted by Gilbert Song <so...@gmail.com> on 2017/11/02 23:18:13 UTC, 16 replies.
- Review Request 63538: Moved expectation for scheduler subscription events in some tests. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/03 14:32:15 UTC, 3 replies.
- Review Request 63540: Added flag protobuf message for agent capabilities. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/03 15:21:48 UTC, 2 replies.
- Review Request 63543: Added bintray publishing scripts. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/11/03 17:35:03 UTC, 7 replies.
- Re: Review Request 63340: Moved Java build code to `java/CMakeLists.txt`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/03 17:39:26 UTC, 0 replies.
- Re: Review Request 60621: Added new stout capability: os::copyfile(source, dest). - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/03 18:29:11 UTC, 5 replies.
- Re: Review Request 63342: Fixed CMake binary dependencies. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/03 21:25:14 UTC, 4 replies.
- Re: Review Request 63437: Slave heartbeat add message sender distinguish. - posted by Vinod Kone <vi...@apache.org> on 2017/11/03 22:26:24 UTC, 0 replies.
- Re: Review Request 63400: Fix bad links. - posted by Vinod Kone <vi...@apache.org> on 2017/11/03 22:33:35 UTC, 8 replies.
- Re: Review Request 63423: Cleared the executor auth token after using it. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/03 22:33:57 UTC, 0 replies.
- Review Request 63550: Included commit message in `push-commits.py`. - posted by Vinod Kone <vi...@apache.org> on 2017/11/04 00:12:22 UTC, 6 replies.
- Re: Review Request 62762: Added `LocalResourceProvider::principal()` for authentication. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 00:49:26 UTC, 9 replies.
- Re: Review Request 62636: Generated authentication tokens for local resource providers. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 00:51:03 UTC, 17 replies.
- Re: Review Request 63018: Added filesystem layout for local resource providers. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 00:52:39 UTC, 4 replies.
- Re: Review Request 63377: Added filesystem layout for storage resource providers. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 00:58:41 UTC, 1 replies.
- Re: Review Request 63021: Added functions to launch CSI plugin in storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:06:23 UTC, 0 replies.
- Re: Review Request 63023: Added a test CSI plugin. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:09:07 UTC, 1 replies.
- Review Request 63498: Allowed relative path in `Resource.disk.source.{path, mount}.root. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:13:34 UTC, 0 replies.
- Re: Review Request 63387: Added publish/unpublish in storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:17:48 UTC, 1 replies.
- Re: Review Request 63388: Handling offer operations in storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:23:08 UTC, 1 replies.
- Review Request 63554: WIP: Added a publish function in resource provider manager. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:35:50 UTC, 0 replies.
- Re: Review Request 63390: Create/destroy volume test for storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:40:23 UTC, 0 replies.
- Review Request 63555: Publish resource provider resources before container launch or update. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:55:10 UTC, 4 replies.
- Review Request 63556: Added a test to check volume publish before running tasks. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/04 01:57:01 UTC, 2 replies.
- Re: Review Request 61128: Improved log messages in master when adding/removing tasks/executors. - posted by Mesos Reviewbot <re...@mesos.apache.org> on 2017/11/04 04:56:05 UTC, 1 replies.
- Review Request 63560: Removed unnecessary overhead of constructing another temporary string. - posted by Michael Park <mp...@apache.org> on 2017/11/04 17:34:44 UTC, 3 replies.
- Review Request 63565: MesosTidy: Added some `google-*` checks to the default set of checks. - posted by Michael Park <mp...@apache.org> on 2017/11/06 00:10:29 UTC, 5 replies.
- Review Request 63566: MesosTidy: Added braces around statements. - posted by Michael Park <mp...@apache.org> on 2017/11/06 00:10:35 UTC, 2 replies.
- Review Request 63567: MesosTidy: Removed redundant `get()` on smart pointers in libprocess. - posted by Michael Park <mp...@apache.org> on 2017/11/06 00:10:41 UTC, 3 replies.
- Review Request 63568: MesosTidy: removed redundant `get()` on smart pointers in mesos. - posted by Michael Park <mp...@apache.org> on 2017/11/06 00:10:46 UTC, 9 replies.
- Review Request 63569: CMake: Compile the 3rdparty dependencies with the same compiler. - posted by Michael Park <mp...@apache.org> on 2017/11/06 01:17:06 UTC, 8 replies.
- Review Request 63570: CMake: Fixed the debug build. - posted by Michael Park <mp...@apache.org> on 2017/11/06 01:19:29 UTC, 1 replies.
- Re: Review Request 62734: Added --markdown and --target-people to post-reviews.py. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/06 01:35:10 UTC, 2 replies.
- Re: Review Request 63232: Converted SlaveInfo resources from the registry during master recovery. - posted by Michael Park <mp...@apache.org> on 2017/11/06 03:24:03 UTC, 1 replies.
- Review Request 63577: Fixed a task status update race in default executor tests. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/06 13:31:51 UTC, 14 replies.
- Re: Review Request 62938: Added ZooKeeper leader resolution to CLI. - posted by Kevin Klues <kl...@gmail.com> on 2017/11/06 14:48:27 UTC, 4 replies.
- Review Request 63584: Added agent ID to offer operation protobuf helper. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/06 14:50:34 UTC, 4 replies.
- Review Request 63581: Created virtual environment for linters in /support. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/06 14:52:52 UTC, 6 replies.
- Review Request 63582: Removed pylint from the CLI requirements. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/06 16:53:07 UTC, 1 replies.
- Review Request 63585: Improved support/mesos-style.py structure. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/06 17:00:00 UTC, 5 replies.
- Re: Review Request 62734: Added Markdown, bug, and reviewers support to `post-reviews.py`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/06 18:06:15 UTC, 3 replies.
- Review Request 63588: Updated the offer operation bookeeping in the master. - posted by Jie Yu <yu...@gmail.com> on 2017/11/06 18:14:58 UTC, 3 replies.
- Re: Review Request 63341: Set `BUILD_FLAGS` flag in CMake. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/06 18:34:27 UTC, 0 replies.
- Review Request 63589: Fixed bug in tests leading to orphaned containers. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/11/06 18:41:33 UTC, 5 replies.
- Review Request 63590: Enabled CredentialsTests on Windows. - posted by John Kordich via Review Board <no...@reviews.apache.org> on 2017/11/06 18:49:41 UTC, 2 replies.
- Review Request 63591: Flattened unnecessary `Optional>` to simply `Shared`. - posted by Michael Park <mp...@apache.org> on 2017/11/06 18:54:34 UTC, 1 replies.
- Review Request 63592: Implemented updateOfferOperation in the master. - posted by Jie Yu <yu...@gmail.com> on 2017/11/06 18:55:00 UTC, 4 replies.
- Review Request 63594: Improve fetcher exit status log message. - posted by James Peach <jp...@apache.org> on 2017/11/06 20:36:33 UTC, 0 replies.
- Re: Review Request 63538: Removed expectation for scheduler connected events in some tests. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/06 20:42:47 UTC, 4 replies.
- Re: Review Request 63594: Improved the fetcher exit status log message. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/06 21:00:41 UTC, 0 replies.
- Re: Review Request 55334: Checkpoint and recover `ContainerConfig` in Mesos containerizer. - posted by Gilbert Song <so...@gmail.com> on 2017/11/06 21:43:44 UTC, 6 replies.
- Re: Review Request 55335: Backfilled required fields in TaskInfo in MesosContainerizer* tests. - posted by Gilbert Song <so...@gmail.com> on 2017/11/06 21:50:15 UTC, 4 replies.
- Review Request 63597: Refactored XFS quota tests for more reuse for upcoming XFS related tests. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/06 22:10:49 UTC, 2 replies.
- Re: Review Request 59687: Added tests for recovering ContainerConfig. - posted by Gilbert Song <so...@gmail.com> on 2017/11/06 22:53:46 UTC, 6 replies.
- Re: Review Request 63276: Windows: Added `Cpu` and `Mem` isolators. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/06 23:18:33 UTC, 5 replies.
- Re: Review Request 63277: Windows: Ported CPU and memory isolator tests. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/06 23:19:01 UTC, 3 replies.
- Re: Review Request 63278: Windows: Documented the `cpu` and `mem` isolators. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/06 23:19:13 UTC, 3 replies.
- Review Request 63598: Set container info from executor by default if available. - posted by Julien Pepy <j....@criteo.com> on 2017/11/07 01:12:59 UTC, 2 replies.
- Re: Review Request 63253: Fixed DefaultExecutorTest for proper URI handling on Windows. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/07 02:09:11 UTC, 5 replies.
- Re: Review Request 62635: Added an optional `unsafe` parameter to `http::encode`. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/07 02:36:00 UTC, 0 replies.
- Review Request 63604: Removed an unnecessary serialization in protobuf process. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/07 03:40:22 UTC, 1 replies.
- Review Request 63605: Support moving in data during ProcessBase::send. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/07 03:40:23 UTC, 1 replies.
- Review Request 63606: Avoid an extra copy during ProtobufProcess::send. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/07 04:23:14 UTC, 3 replies.
- Review Request 63611: Refactored scheduler test driver to avoid using uninitialized object. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/07 12:58:38 UTC, 2 replies.
- Review Request 63612: Modified formatting of v1 scheduler test driver for consistency. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/07 12:58:41 UTC, 2 replies.
- Review Request 63613: Refactored executor test driver to avoid using uninitialized object. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/07 12:58:42 UTC, 3 replies.
- Review Request 63614: Removed unnecessary check in v1 scheduler library. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/07 12:58:45 UTC, 2 replies.
- Review Request 63615: Passed scheduler as a shared pointer into the callback. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/07 12:58:46 UTC, 4 replies.
- Re: Review Request 62880: Marked the agent as RESOURCE_PROVIDER capable. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/07 13:49:53 UTC, 0 replies.
- Re: Review Request 63495: Added comparison operators for 'ResourceVersionUUID'. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/07 14:11:10 UTC, 0 replies.
- Review Request 63620: Updated offer operation handling to set resource versions. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/07 14:18:45 UTC, 4 replies.
- Review Request 63621: Added a devolve function for 'OfferOperationStatus'. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/07 14:19:40 UTC, 2 replies.
- Review Request 63622: Provided handling for offer operation updates. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/07 14:20:31 UTC, 12 replies.
- Re: Review Request 62161: Update boost version. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/07 15:14:56 UTC, 3 replies.
- Re: Review Request 63292: Added lamdba::zip. - posted by Alexander Rojas <al...@mesosphere.io> on 2017/11/07 15:26:39 UTC, 0 replies.
- Review Request 63625: Added a test for resource conversion using a resource provider. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/07 15:36:21 UTC, 10 replies.
- Review Request 63628: Implemented index_sequence and related functionality. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:10 UTC, 4 replies.
- Review Request 63629: Added partial function application implementation. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:14 UTC, 2 replies.
- Review Request 63630: Added support for callable once functors. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:21 UTC, 1 replies.
- Review Request 63631: Separated event visiting and consumption. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:26 UTC, 0 replies.
- Review Request 63632: Migrated to event consumer interface. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:34 UTC, 0 replies.
- Review Request 63633: Added ability to protect nested bind expressions from composition. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:39 UTC, 2 replies.
- Review Request 63634: Changed dispatch to use callable once functors. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:45 UTC, 0 replies.
- Review Request 63635: Prepared defer for use in callable once contexts. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:50 UTC, 0 replies.
- Review Request 63636: Added placeholder implementation. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:51:59 UTC, 1 replies.
- Review Request 63637: Added callable once support in defer. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:52:09 UTC, 0 replies.
- Review Request 63638: Added callable once support in Future. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:52:20 UTC, 1 replies.
- Review Request 63639: Enabled rvalue reference parameters in protobuf handlers. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 16:52:28 UTC, 1 replies.
- Review Request 63641: Used move for events consumption in master. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/07 17:42:30 UTC, 2 replies.
- Review Request 63642: Added a test for ExecutorID validation in ReregisterSlaveMessage. - posted by James Peach <jp...@apache.org> on 2017/11/07 18:47:59 UTC, 7 replies.
- Re: Review Request 62997: Added checkpoint and recover capability for layers in provisioner. - posted by Gilbert Song <so...@gmail.com> on 2017/11/07 19:46:12 UTC, 9 replies.
- Re: Review Request 63492: Synchronized agent resource versions via 'UpdateSlaveMessage'. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/07 22:00:57 UTC, 2 replies.
- Review Request 63649: Introduced `.clang-tidy` configuration file. - posted by Michael Park <mp...@apache.org> on 2017/11/07 22:02:47 UTC, 2 replies.
- Re: Review Request 63175: Do not generate UnavailableResources for inactive frameworks. - posted by James Peach <jp...@apache.org> on 2017/11/07 23:58:00 UTC, 3 replies.
- Review Request 63652: Added d_type check in containerizer backend validation. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/07 23:59:51 UTC, 21 replies.
- Re: Review Request 62143: Added validation for Standalone Container APIs. - posted by Jie Yu <yu...@gmail.com> on 2017/11/08 00:11:07 UTC, 2 replies.
- Review Request 63654: Added a `net::socketpair` helper to stout. - posted by James Peach <jp...@apache.org> on 2017/11/08 01:12:38 UTC, 4 replies.
- Review Request 63655: Switched to `net::socketpair` in `ns::clone`. - posted by James Peach <jp...@apache.org> on 2017/11/08 01:14:06 UTC, 4 replies.
- Review Request 63661: Updated tests to use `createCallSubscribe`. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/08 10:47:00 UTC, 2 replies.
- Review Request 63662: Dropped scheduler call if a connection has been interrupted. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/08 11:04:17 UTC, 4 replies.
- Re: Review Request 60891: Added ACLs and AuthZ for standalone containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/08 16:15:49 UTC, 4 replies.
- Review Request 63672: Added a two-parameter SendSubscribe action in test. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/08 17:31:26 UTC, 2 replies.
- Review Request 63673: Allowed multiple master detections in MasterFailover test. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/08 17:31:27 UTC, 2 replies.
- Review Request 63674: Updated a comment about resubscribing completed frameworks. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/08 17:31:38 UTC, 8 replies.
- Review Request 63675: Added a non-allocating variant of `os::clone`. - posted by James Peach <jp...@apache.org> on 2017/11/08 17:46:37 UTC, 2 replies.
- Review Request 63677: Changed `os:Stack` to allocate with `mmap`. - posted by James Peach <jp...@apache.org> on 2017/11/08 17:47:07 UTC, 6 replies.
- Review Request 63678: Improved the signal safety of `ns::clone`. - posted by James Peach <jp...@apache.org> on 2017/11/08 17:51:00 UTC, 6 replies.
- Review Request 63679: Added test helpers for storage operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/08 17:55:26 UTC, 1 replies.
- Re: Review Request 63054: Updated test filters to include value-parameterized tests. - posted by Jie Yu <yu...@gmail.com> on 2017/11/08 18:30:47 UTC, 0 replies.
- Review Request 63680: Added a container daemon to monitor a long-running standalone container. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/08 19:55:45 UTC, 15 replies.
- Review Request 63682: Updated tests to toggle pipe inheritance. - posted by James Peach <jp...@apache.org> on 2017/11/08 21:14:26 UTC, 2 replies.
- Review Request 63689: Used `Master::_apply` consistently for all offer operations. - posted by Jie Yu <yu...@gmail.com> on 2017/11/09 00:02:49 UTC, 8 replies.
- Re: Review Request 62579: Added a test `BlkioIsolatorTest.ROOT_BlkioUsage`. - posted by Gilbert Song <so...@gmail.com> on 2017/11/09 01:10:01 UTC, 0 replies.
- Re: Review Request 62579: Added a test `CgroupsIsolatorTest.ROOT_CGROUPS_BlkioUsage`. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/09 02:03:22 UTC, 3 replies.
- Review Request 63694: Added context to agent processing calls. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/09 09:50:11 UTC, 7 replies.
- Re: Review Request 63554: Added a publish function in resource provider manager. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/09 10:43:57 UTC, 1 replies.
- Re: Review Request 62214: Added JavaScript linter. - posted by Kevin Klues <kl...@gmail.com> on 2017/11/09 15:22:11 UTC, 2 replies.
- Review Request 63703: Removed unnecessary semicolon in controllers.js. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/09 16:18:36 UTC, 0 replies.
- Re: Review Request 63703: Linted controllers.js. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/09 16:35:20 UTC, 1 replies.
- Re: Review Request 62166: Updated RC tagging+voting mechanism. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/11/09 23:16:42 UTC, 2 replies.
- Review Request 63710: Fixed flaky RescindRevocableOfferWithIncreasedRevocable test. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/09 23:45:14 UTC, 2 replies.
- Re: Review Request 63172: Added and used more v1 scheduler API test helpers. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/09 23:57:35 UTC, 0 replies.
- Re: Review Request 63173: Refactored and fixed `DefaultExecutorTest.CommitSuicideOnKillTask`. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/09 23:58:00 UTC, 0 replies.
- Review Request 63723: Upated excluded files by JavaScript linter. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/10 12:59:52 UTC, 1 replies.
- Review Request 63724: Updated JavaScript linter rules. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/10 13:47:13 UTC, 1 replies.
- Re: Review Request 63703: Linted JavaScript files. - posted by Kevin Klues <kl...@gmail.com> on 2017/11/10 14:02:37 UTC, 0 replies.
- Review Request 63726: Fixed broken python linter. - posted by Kevin Klues <kl...@gmail.com> on 2017/11/10 14:56:54 UTC, 4 replies.
- Review Request 63728: Fix IoSwitchboard tests. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/10 15:07:13 UTC, 3 replies.
- Re: Review Request 63365: Re-ordered some configuration options alphabetically. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/10 15:09:29 UTC, 0 replies.
- Review Request 63733: Don't print full usage for invocation errors. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/10 15:40:16 UTC, 1 replies.
- Re: Review Request 62893: Expand description of the --recovery_timeout flag. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/10 15:45:02 UTC, 2 replies.
- Re: Review Request 63366: Added jemalloc release tarball and build rules. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/10 15:52:52 UTC, 1 replies.
- Re: Review Request 63368: Added MemoryProfiler class to Libprocess. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/10 16:31:04 UTC, 3 replies.
- Review Request 63737: Handled the resource conversion for new operations in master. - posted by Jie Yu <yu...@gmail.com> on 2017/11/10 17:55:02 UTC, 2 replies.
- Re: Review Request 63269: Windows: Added `os::set_job_cpu_limit` to stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/11 00:56:54 UTC, 5 replies.
- Re: Review Request 63271: Windows: Added `os::set_job_memory_limit` to stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/11 01:57:15 UTC, 0 replies.
- Re: Review Request 63272: Windows: Added `os::get_job_info` to stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/11 01:59:50 UTC, 0 replies.
- Re: Review Request 63274: Windows: Added `os::get_job_memory` to stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/11 02:39:38 UTC, 0 replies.
- Review Request 63751: Triggered 'ApplyOfferOperatioMessage' for agent local resources. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/13 10:42:16 UTC, 1 replies.
- Review Request 63752: Updated documentation on protobuf version requirements. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/13 13:35:04 UTC, 1 replies.
- Re: Review Request 63751: Triggered 'ApplyOfferOperationMessage' for agent local resources. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/13 14:21:09 UTC, 14 replies.
- Review Request 63761: Replaced std::shared_ptr with std::unique_ptr in dispatch. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/13 17:02:33 UTC, 1 replies.
- Re: Review Request 63731: Reconciled pending resource provider operations in agent. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/13 18:33:56 UTC, 20 replies.
- Re: Review Request 63730: Passed operations from resource provider to agent. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/13 18:33:56 UTC, 6 replies.
- Re: Review Request 63732: Reconciled offer operations between agent and master. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/13 18:33:57 UTC, 25 replies.
- Re: Review Request 63370: Added new --memory_profiling flag to agent and master binaries. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/13 19:32:06 UTC, 0 replies.
- Re: Review Request 63214: Fixed GPU tests. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/13 20:20:50 UTC, 0 replies.
- Review Request 63765: Fixed an issue with the scheduler driver subscribe backoff time. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/13 21:32:50 UTC, 6 replies.
- Re: Review Request 63275: Windows: Abstracted out `os::name_job` in stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/13 21:40:27 UTC, 1 replies.
- Review Request 63766: Clarified log message when selecting backend. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/13 22:10:55 UTC, 1 replies.
- Review Request 63768: Added 'ReconcileOfferOperations' reponse to scheduler API. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/13 23:16:54 UTC, 3 replies.
- Review Request 63767: Added a missing protobuf field to the unversioned definitions. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/13 23:17:14 UTC, 3 replies.
- Re: Review Request 63271: Windows: Added `os::set_job_mem_limit` to stout. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 00:20:17 UTC, 1 replies.
- Review Request 63781: Updated Docker path for Containerizer::launch interface change. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/14 01:38:16 UTC, 1 replies.
- Re: Review Request 63515: Updated some isolators' usage of ExecutorInfo. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/14 01:41:12 UTC, 2 replies.
- Re: Review Request 63274: Windows: Added `os::get_job_mem` to stout. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 01:56:47 UTC, 1 replies.
- Review Request 63779: Windows: Fixed `SlaveReregisterTaskExecutorIds` test. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 02:00:33 UTC, 2 replies.
- Review Request 63791: Handled the resource conversion for new operations in master. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/14 09:31:44 UTC, 2 replies.
- Re: Review Request 63753: Upgraded vendored protobuf to 3.5.0. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/14 13:00:56 UTC, 2 replies.
- Review Request 63794: Added a new member field `ipv6` to the `CheckerProcess` class. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/14 13:27:04 UTC, 6 replies.
- Review Request 63795: Made `mesos-tcp-connect` support IPv6. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/14 13:30:05 UTC, 16 replies.
- Review Request 63796: Made `CheckerProcess` support IPv6 for HTTP/TCP check. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/14 13:31:57 UTC, 7 replies.
- Review Request 63797: Changed resource checkpointing to no longer failover. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/14 14:11:10 UTC, 1 replies.
- Review Request 63798: Added resource provider support for all offer operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/14 14:11:56 UTC, 8 replies.
- Review Request 63799: Improved log messages of offer operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/14 14:12:39 UTC, 2 replies.
- Review Request 63801: Windows: Fixed build of Google Test with Visual Studio Preview. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 18:04:17 UTC, 3 replies.
- Review Request 63804: Added protos for operation reconciliation between master and agent. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/14 18:41:22 UTC, 4 replies.
- Re: Review Request 63768: Added 'ReconcileOfferOperations' response to scheduler API. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/14 20:48:05 UTC, 3 replies.
- Re: Review Request 63804: Added plumbing for operation reconciliation between master and agent. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/14 22:38:59 UTC, 3 replies.
- Review Request 63818: Added quick start instructions for CMake. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:12:30 UTC, 5 replies.
- Review Request 63809: Windows: Fixed symlink code to not need admin privileges. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:14:05 UTC, 2 replies.
- Review Request 63810: Windows: Added internal `fullpath` API to normalize paths. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:14:34 UTC, 5 replies.
- Review Request 63811: Windows: Added `get_handle_follow` which follows symlinks. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:14:47 UTC, 2 replies.
- Review Request 63812: Windows: Fixed `os::realpath` to behave like POSIX version. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:14:58 UTC, 4 replies.
- Review Request 63813: Windows: Fixed name of default executor. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:15:14 UTC, 3 replies.
- Review Request 63814: Windows: Fixed `os::host_default_path()`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:16:19 UTC, 4 replies.
- Review Request 63815: Windows: Fixed environment priorities in `shell.hpp`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:16:43 UTC, 4 replies.
- Review Request 63816: Windows: Fixed MESOS-6816 to enable `ExecutorEnvironmentVariables`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:17:06 UTC, 4 replies.
- Review Request 63817: Windows: Enabled more agent tests. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/14 23:21:19 UTC, 8 replies.
- Review Request 63823: Initialized and subscribed storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/15 02:37:59 UTC, 4 replies.
- Re: Review Request 63021: Added `getService()` function to launch CSI plugins. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/15 02:40:50 UTC, 5 replies.
- Re: Review Request 63022: Imported resources from CSI plugins in storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/15 02:42:36 UTC, 3 replies.
- Re: Review Request 63389: Added a mock resource provider manager. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/15 02:46:32 UTC, 0 replies.
- Review Request 63829: Modified Containerizer::remove to allow top-level containers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/15 08:41:59 UTC, 1 replies.
- Review Request 63828: Added authorization tests for standalone container APIs. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/15 08:42:00 UTC, 2 replies.
- Review Request 63833: Added context to new agent processing calls. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/15 12:12:58 UTC, 1 replies.
- Review Request 63842: Allowed removing non-terminal offer operations. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/15 17:31:07 UTC, 21 replies.
- Review Request 63843: Implemented a test of offer operation reconcilation. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/15 17:31:08 UTC, 11 replies.
- Review Request 63844: Removed acknowledged offer operation status updates. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/15 17:31:10 UTC, 6 replies.
- Review Request 63846: Updated test PortMappingMesosTest.CGROUPS_ROOT_CleanUpOrphan. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/15 18:16:00 UTC, 2 replies.
- Review Request 63741: Fixed framework subscription with suppressed roles. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/15 18:41:37 UTC, 0 replies.
- Review Request 63830: Fixed 'NoOffersWithAllRolesSuppressed' test. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/15 18:45:01 UTC, 4 replies.
- Review Request 63848: Added test to check standalone containers in GET_CONTAINERS. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/15 19:42:44 UTC, 2 replies.
- Re: Review Request 62777: Fixed build dependency for `protobuf_tests.proto`. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/15 21:30:12 UTC, 2 replies.
- Review Request 63850: Used fully qualified namespace in the scheduler library for consistency. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/15 21:53:05 UTC, 4 replies.
- Re: Review Request 62637: Added an object approver to authorize requests from resource providers. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/15 22:54:46 UTC, 4 replies.
- Review Request 63851: Changed backend validation logic from exclusive to inclusive. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/15 23:21:21 UTC, 0 replies.
- Re: Review Request 60620: Modifed os::write to write binary files on Windows. - posted by Michael Park <mp...@apache.org> on 2017/11/15 23:34:39 UTC, 0 replies.
- Review Request 63852: Added "Task" prefix to status update manager related classes/methods. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/15 23:41:37 UTC, 2 replies.
- Review Request 63853: Added the "task" prefix to the name of the status update manager files. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/15 23:42:12 UTC, 5 replies.
- Re: Review Request 63385: Added utility functions for CSI responses and volume ID and metadata. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/16 00:33:13 UTC, 0 replies.
- Re: Review Request 63019: Encapsulated common error handling in CSI clients. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/16 00:47:36 UTC, 1 replies.
- Re: Review Request 56722: Added a new operator API for `PRUNE_IMAGES`. - posted by Zhitao Li <zh...@gmail.com> on 2017/11/16 04:03:28 UTC, 6 replies.
- Re: Review Request 60471: Added tests for pruneImages for containerizer and provisioner. - posted by Zhitao Li <zh...@gmail.com> on 2017/11/16 04:05:53 UTC, 6 replies.
- Re: Review Request 61473: Do not kill non partition aware tasks. - posted by Megha Sharma <ms...@apple.com> on 2017/11/16 05:14:47 UTC, 26 replies.
- Review Request 63831: Fixed a bug that removed the suppressed framework from sorter. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/16 07:05:21 UTC, 4 replies.
- Re: Review Request 60622: Added new stout functions for path and URI conversions. - posted by Michael Park <mp...@apache.org> on 2017/11/16 18:08:39 UTC, 0 replies.
- Re: Review Request 60623: Converted "file://" URI handling to use new uri function. - posted by Michael Park <mp...@apache.org> on 2017/11/16 18:08:46 UTC, 0 replies.
- Re: Review Request 60624: Enabled HDFS compilation and associated tests. - posted by Michael Park <mp...@apache.org> on 2017/11/16 18:08:49 UTC, 0 replies.
- Review Request 63859: Fixed signal vals & WEXITSTATUS on Windows stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/16 18:17:19 UTC, 2 replies.
- Review Request 63860: Fixed docker network settings for Windows. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/16 18:18:20 UTC, 2 replies.
- Review Request 63861: Updated networking doc for Windows docker containers. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/16 18:18:31 UTC, 2 replies.
- Review Request 63862: Ported docker_tests.cpp to Windows. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/16 18:18:40 UTC, 3 replies.
- Review Request 63887: Fixed `wait()` and `destroy()` in composing containerizer. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/11/16 20:42:21 UTC, 4 replies.
- Review Request 63888: Updated composing containerizer tests. - posted by Andrei Budnik <ab...@mesosphere.com> on 2017/11/16 20:42:22 UTC, 5 replies.
- Review Request 63889: Use annotated tags in vote.sh and release.sh. - posted by Kapil Arya <ka...@mesosphere.io> on 2017/11/16 20:46:46 UTC, 3 replies.
- Re: Review Request 60628: Enabled fetcher_tests.cpp unit test module on Windows platform. - posted by Michael Park <mp...@apache.org> on 2017/11/16 23:08:58 UTC, 0 replies.
- Re: Review Request 60626: Eliminated os::shell calls from HDFS for Windows compatibility. - posted by Jie Yu <yu...@gmail.com> on 2017/11/16 23:34:02 UTC, 0 replies.
- Re: Review Request 63376: Started `LocalResourceProviderDaemon` after obtaining the slave ID. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/17 00:13:59 UTC, 2 replies.
- Review Request 63897: WIP. Add prototype of master changes. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/17 00:19:42 UTC, 2 replies.
- Re: Review Request 55727: Checkpoint and track docker image layer sizes. - posted by Zhitao Li <zh...@gmail.com> on 2017/11/17 01:43:54 UTC, 0 replies.
- Review Request 63895: Fixed flaky test UnreachableAgentReregisterAfterFailover test. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/17 05:42:43 UTC, 1 replies.
- Review Request 63900: Downgraded the logging level of socket shutdown failures. - posted by Jiang Yan Xu <ya...@jxu.me> on 2017/11/17 06:54:24 UTC, 2 replies.
- Review Request 63911: Bumped up the number of args allowed in some libprocess templates. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/17 13:07:03 UTC, 1 replies.
- Review Request 63912: Changed agent (re-)registration to set resource versions. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/17 13:07:48 UTC, 4 replies.
- Review Request 63910: Added 3 tests for TCP/HTTP(S) health check support for Docker container. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/17 13:25:47 UTC, 16 replies.
- Review Request 63913: Replaced std::shared_ptr with std::unique_ptr in Future. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/17 14:02:21 UTC, 1 replies.
- Review Request 63914: Changed agent reregistration to work with message directly. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/17 14:02:33 UTC, 1 replies.
- Review Request 63915: Reduced tasks copying during agent reregistration. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/17 14:02:47 UTC, 2 replies.
- Re: Review Request 63901: WIP: Added operator API to update and remove resource provider configs. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/17 14:39:02 UTC, 1 replies.
- Re: Review Request 62853: Added test for `PRUNE_IMAGES` operator API call. - posted by Zhitao Li <zh...@gmail.com> on 2017/11/17 16:48:13 UTC, 2 replies.
- Review Request 63917: Renamed resource provider message UpdateTotalResources to UpdateState. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/17 17:24:16 UTC, 3 replies.
- Review Request 63918: Updated ResourceProviderMessage stringification with recent changes. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/17 17:24:21 UTC, 3 replies.
- Re: Review Request 63859: Windows: Fixed mock signal values in stout. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/17 22:35:48 UTC, 1 replies.
- Re: Review Request 63860: Windows: Mapped the Docker network info types. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/17 22:38:00 UTC, 1 replies.
- Re: Review Request 63861: Windows: Updated networking doc. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/17 22:38:58 UTC, 1 replies.
- Review Request 63930: MesosTidy: Turned on [misc-use-after-move]. - posted by Michael Park <mp...@apache.org> on 2017/11/18 00:03:07 UTC, 1 replies.
- Review Request 63931: MesosTidy [misc-use-after-move]: Fixed `HashMapTest` test. - posted by Michael Park <mp...@apache.org> on 2017/11/18 00:03:24 UTC, 1 replies.
- Review Request 63932: MesosTidy [misc-use-after-move]: Added `NOLINT` to false-positives. - posted by Michael Park <mp...@apache.org> on 2017/11/18 00:03:29 UTC, 1 replies.
- Review Request 63933: MesosTidy [misc-use-after-move]: Arguments moved into chained calls. - posted by Michael Park <mp...@apache.org> on 2017/11/18 00:03:35 UTC, 2 replies.
- Review Request 63939: Added a script to build Mesos docker image. - posted by Jie Yu <yu...@gmail.com> on 2017/11/19 06:28:51 UTC, 1 replies.
- Review Request 63940: Added a Future constructor for Try>. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/20 00:23:04 UTC, 1 replies.
- Re: Review Request 61149: Added stringification support for Future. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/20 00:37:50 UTC, 1 replies.
- Review Request 63942: Added a state machine abstraction. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/11/20 01:02:05 UTC, 1 replies.
- Review Request 63941: Added support for enum's to hashset and hashmap. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/11/20 01:02:18 UTC, 1 replies.
- Review Request 63943: Renamed `TaskStatusEq()` to `TaskStatusTaskIdEq()`. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/20 02:44:09 UTC, 3 replies.
- Re: Review Request 61155: Added http::Server. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/20 02:50:59 UTC, 0 replies.
- Re: Review Request 61157: Refactored ProcessManager::handle for future use with http::Server. - posted by Mesos Reviewbot Windows <re...@mesos.apache.org> on 2017/11/20 03:04:39 UTC, 1 replies.
- Re: Review Request 61154: Added safe downcasts for Socket. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/20 09:33:30 UTC, 1 replies.
- Re: Review Request 63104: Added a helper to extract resources from storage operations. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/20 12:47:24 UTC, 0 replies.
- Review Request 63951: Added noexcept specifier to Option. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/20 14:38:29 UTC, 1 replies.
- Review Request 63959: Optimized resources logging and conversion in master. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/20 16:51:25 UTC, 3 replies.
- Review Request 63960: Preallocated buffer for resources conversion. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/20 17:20:18 UTC, 2 replies.
- Review Request 63963: Windows: Fixed HTTP server build and tests. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/20 21:40:30 UTC, 3 replies.
- Review Request 63966: Bumped the CSI version to ec29890. - posted by Jie Yu <yu...@gmail.com> on 2017/11/20 22:41:46 UTC, 1 replies.
- Review Request 63967: Updated the gRPC client to return both status and payload. - posted by Jie Yu <yu...@gmail.com> on 2017/11/20 22:42:05 UTC, 1 replies.
- Review Request 63968: Updated the CSI client to use the new gRPC client interface. - posted by Jie Yu <yu...@gmail.com> on 2017/11/20 22:42:32 UTC, 2 replies.
- Re: Review Request 62635: Added an optional `additional_chars` parameter to `http::encode`. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/20 23:44:47 UTC, 1 replies.
- Review Request 63969: Storage local resource provider is built when gRPC is enabled on Linux. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/21 00:06:37 UTC, 1 replies.
- Re: Review Request 63904: Updated the top-level `CSIPluginInfo` proto. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/21 05:32:40 UTC, 3 replies.
- Re: Review Request 63377: Added filesystem layout for CSI plugins. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/21 05:36:41 UTC, 4 replies.
- Review Request 63976: Modified `downgradeResources` to use protobuf reflection. - posted by Michael Park <mp...@apache.org> on 2017/11/21 06:07:14 UTC, 0 replies.
- Review Request 63977: Used the protobuf-reflection-based `downgradeResources` utility. - posted by Michael Park <mp...@apache.org> on 2017/11/21 06:07:16 UTC, 0 replies.
- Re: Review Request 62502: Added an event to publish resources. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/21 11:55:52 UTC, 1 replies.
- Re: Review Request 63953: Added logging based on container class. - posted by Armand Grillet <ag...@mesosphere.io> on 2017/11/21 13:12:43 UTC, 5 replies.
- Review Request 63984: Fixed build dependencies in libprocess. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/21 13:32:18 UTC, 5 replies.
- Review Request 63986: Refactored 'MockResourceProvider' to include default functions. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/21 15:29:49 UTC, 1 replies.
- Review Request 63988: Fixed indentation in a test header. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/21 17:10:32 UTC, 3 replies.
- Review Request 63989: Added the OFFER_OPERATION_DROPPED state. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/21 17:21:29 UTC, 1 replies.
- Review Request 63990: Added the RESOURCE_PROVIDER capability to the agent. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/21 17:23:09 UTC, 0 replies.
- Review Request 63991: Added helpers to create and forward offer operation updates. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/21 17:29:20 UTC, 0 replies.
- Review Request 63992: Handled incorrect offer operation IDs in master's ACCEPT handler. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/21 17:31:40 UTC, 0 replies.
- Review Request 63994: Added a new test for validation of offer operation IDs. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/21 17:33:30 UTC, 1 replies.
- Re: Review Request 63741: Fixed a bug in devolving framework subscription with suppressed roles. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/21 18:04:47 UTC, 3 replies.
- Re: Review Request 63060: Added utility functions and structures for CSI version and capabilities. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/21 19:55:38 UTC, 2 replies.
- Review Request 63997: Added new allocator method to explicitly bookkeep existing allocations. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/21 21:08:36 UTC, 0 replies.
- Review Request 63999: Added env var to set default flags for parallel test runner. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/21 21:09:05 UTC, 3 replies.
- Review Request 64000: Allowed stringification of hashmaps of stringifiable types. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/21 21:10:35 UTC, 4 replies.
- Review Request 64001: Made sure all true allocator agent updates return correct status. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/21 21:10:39 UTC, 3 replies.
- Review Request 64003: Made quota resource allocation fine-grained. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/21 21:14:01 UTC, 0 replies.
- Re: Review Request 52006: Supported docker/volume isolator to be nested aware. - posted by Gilbert Song <so...@gmail.com> on 2017/11/21 22:08:24 UTC, 1 replies.
- Review Request 64006: Moved registry operations into separate header. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/21 23:20:31 UTC, 7 replies.
- Review Request 64007: Added master capabilities. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/21 23:21:00 UTC, 1 replies.
- Review Request 64008: Activate AGENT_CONFIGURATION_CHANGE capability. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/21 23:50:57 UTC, 1 replies.
- Review Request 64009: Added new UpdateSlave registry operation. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/21 23:53:12 UTC, 8 replies.
- Review Request 64010: Added new overload for reviveOffers to allocator API. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/22 00:11:20 UTC, 1 replies.
- Review Request 64014: Added reservations in `ResourceProviderInfo`. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/22 00:21:42 UTC, 0 replies.
- Review Request 64018: Added tests for the d_type support validation. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/22 01:26:06 UTC, 3 replies.
- Re: Review Request 63385: Added utility functions for CSI Plugin info and volume attributes. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/22 05:21:50 UTC, 4 replies.
- Re: Review Request 63823: Initialization and registration for storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/22 08:47:22 UTC, 1 replies.
- Re: Review Request 63959: Optimized resources logging in master. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/22 13:24:12 UTC, 1 replies.
- Review Request 64028: Removed duplicate resources conversions. - posted by Dmitry Zhuk <dz...@twopensource.com> on 2017/11/22 13:24:19 UTC, 5 replies.
- Review Request 64032: Promoted log level to warning for disconnected events in exec.cpp. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/22 16:15:43 UTC, 2 replies.
- Review Request 64033: Terminated executors if kill task received before launch task. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/22 16:15:51 UTC, 6 replies.
- Review Request 64011: Updated master behaviour to update agent state on reregistration. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/22 23:31:14 UTC, 9 replies.
- Review Request 64012: Added new --configuration_compatibility slave flag and implementation. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/22 23:31:56 UTC, 4 replies.
- Re: Review Request 64014: Added default reservations in `ResourceProviderInfo`. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/23 00:47:13 UTC, 4 replies.
- Review Request 64044: Killed unused CSI plugin containers for storage local resource provider. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/23 02:25:10 UTC, 0 replies.
- Review Request 64053: Fixed a flaky resource provider test. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/23 15:19:16 UTC, 2 replies.
- Review Request 64054: Refactored offer operation handling for speculative operations. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/23 15:39:38 UTC, 9 replies.
- Review Request 64056: Added a helper to extract consumed resources from offer operations. - posted by Benjamin Bannier <bb...@apache.org> on 2017/11/24 09:31:32 UTC, 2 replies.
- Re: Review Request 63997: Added a new allocator method to add resources to agents. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/24 14:06:42 UTC, 6 replies.
- Review Request 64062: Added 6 nested container tests for `docker/volume` isolator. - posted by Qian Zhang <zh...@gmail.com> on 2017/11/24 14:23:26 UTC, 0 replies.
- Review Request 64067: Fixed a bug for starting local resource provider daemon multiple times. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/24 18:33:00 UTC, 1 replies.
- Review Request 64068: Increased executor log verbosity in tests. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/24 19:12:56 UTC, 1 replies.
- Review Request 64069: Ensured command executor always honors shutdown request. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/24 19:12:57 UTC, 0 replies.
- Review Request 64070: Ensured executor adapter propagates error and shutdown messages. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/24 19:13:03 UTC, 0 replies.
- Re: Review Request 64033: Terminated driver-based executors if kill arrives before launch task. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/24 19:13:15 UTC, 3 replies.
- Review Request 61158: Introduced http::Server in process.cpp. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/11/26 22:28:04 UTC, 1 replies.
- Review Request 64073: Fixed ignored Socket::accept discard when using LibeventSSLSocketImpl. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/11/26 22:28:07 UTC, 1 replies.
- Review Request 64074: Added ENABLE_HTTP_SERVER option for cmake. - posted by Benjamin Hindman <be...@berkeley.edu> on 2017/11/26 22:28:08 UTC, 1 replies.
- Review Request 64075: Added the `State` protobuf for resource providers. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/27 00:33:14 UTC, 0 replies.
- Re: Review Request 63211: Moved the logic of setting '/' as rslave into the launch helper. - posted by Jie Yu <yu...@gmail.com> on 2017/11/27 04:28:47 UTC, 2 replies.
- Re: Review Request 63200: Added ContainerMountInfo to avoid pre_exec_commands for mounts. - posted by Jie Yu <yu...@gmail.com> on 2017/11/27 04:48:57 UTC, 2 replies.
- Re: Review Request 63210: Moved mounts prepare to a helper in launch.cpp. - posted by Jie Yu <yu...@gmail.com> on 2017/11/27 04:49:15 UTC, 1 replies.
- Re: Review Request 63212: Added a findByTarget method for fs::MountInfoTable. - posted by Jie Yu <yu...@gmail.com> on 2017/11/27 04:49:50 UTC, 6 replies.
- Re: Review Request 63213: Added mount propagation support for HOST_PATH volumes. - posted by Jie Yu <yu...@gmail.com> on 2017/11/27 04:50:03 UTC, 2 replies.
- Re: Review Request 64001: Made sure all true allocator agent updates trigger allocations. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/27 10:51:51 UTC, 0 replies.
- Review Request 64081: Fix a code snippet in libprocess documentation. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/27 11:06:02 UTC, 1 replies.
- Review Request 64084: Made explicit that a 'switch' covers all branches. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/27 14:04:23 UTC, 2 replies.
- Review Request 64086: Removed currently unneeded 'AWAIT_READY's in 'MockResourceProvider'. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/27 14:55:43 UTC, 6 replies.
- Re: Review Request 64081: Fixed a code snippet in libprocess documentation. - posted by Alexander Rukletsov <ru...@gmail.com> on 2017/11/27 15:27:59 UTC, 0 replies.
- Re: Review Request 63862: Windows: Ported docker_tests.cpp. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/27 17:45:31 UTC, 1 replies.
- Review Request 64093: Added operators for offer operation update protobuf classes. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/27 19:45:33 UTC, 3 replies.
- Review Request 64094: Added the `OfferOperationStatusUpdateRecord` protobuf message. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/27 19:51:37 UTC, 2 replies.
- Review Request 64095: Added a generic actor to be used by status update managers. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/27 19:52:41 UTC, 2 replies.
- Review Request 64096: Implemented the `OfferOperationStatusUpdateManager`. - posted by Gaston Kleiman <ga...@mesosphere.io> on 2017/11/27 19:53:26 UTC, 1 replies.
- Review Request 63971: Defined a module interface for translating volume profiles. - posted by Joseph Wu <jo...@mesosphere.io> on 2017/11/27 21:33:11 UTC, 4 replies.
- Re: Review Request 61473: Send status updates when agent re-registers. - posted by Megha Sharma <ms...@apple.com> on 2017/11/27 22:02:31 UTC, 0 replies.
- Review Request 64098: Send status updates when agent re-registers. - posted by Megha Sharma <ms...@apple.com> on 2017/11/27 22:08:07 UTC, 9 replies.
- Re: Review Request 64007: Added infrastructure to support master capabilities. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/27 23:29:30 UTC, 3 replies.
- Re: Review Request 64010: Added an optional SlaveInfo parameter to Allocator::updateSlave(). - posted by Benno Evers <be...@mesosphere.com> on 2017/11/27 23:29:58 UTC, 1 replies.
- Re: Review Request 64008: Activated AGENT_UPDATE master capability. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/27 23:30:28 UTC, 4 replies.
- Review Request 64102: Renamed curl target to libcurl, and staging of curl.exe on Windows. - posted by John Kordich via Review Board <no...@reviews.apache.org> on 2017/11/27 23:49:37 UTC, 0 replies.
- Review Request 64103: Changed dependency of curl to libcurl for stout. - posted by John Kordich via Review Board <no...@reviews.apache.org> on 2017/11/27 23:49:42 UTC, 0 replies.
- Review Request 64104: Added dependency of curl to agent. Enabled most health check tests. - posted by John Kordich via Review Board <no...@reviews.apache.org> on 2017/11/27 23:49:45 UTC, 0 replies.
- Review Request 64101: Skip registry update if SlaveInfo didn't change. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/28 00:37:19 UTC, 4 replies.
- Re: Review Request 63385: Added utility functions for volume attributes and printing CSI messages. - posted by Chun-Hung Hsiao <ch...@mesosphere.io> on 2017/11/28 00:57:50 UTC, 0 replies.
- Review Request 64107: Renamed agent flag '--executor_secret_key' to '--jwt_secret_key'. - posted by Jie Yu <yu...@gmail.com> on 2017/11/28 02:28:40 UTC, 2 replies.
- Review Request 64109: Renamed secret key to JWT secret key in the code base. - posted by Jie Yu <yu...@gmail.com> on 2017/11/28 02:46:35 UTC, 2 replies.
- Re: Review Request 64065: Allowed resubscription of resource providers. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/28 11:17:20 UTC, 2 replies.
- Review Request 64132: Allowed the injection of secret generator into the Slave. - posted by Jie Yu <yu...@gmail.com> on 2017/11/28 20:33:39 UTC, 3 replies.
- Review Request 64136: Introduced an allocator helper function to track used resources. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/28 21:31:55 UTC, 5 replies.
- Review Request 64135: Removed stringification for hashmaps of strings. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/28 21:31:57 UTC, 1 replies.
- Re: Review Request 64000: Added generic stringification of hashmap of stringifiable types. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/28 21:32:07 UTC, 2 replies.
- Review Request 64139: Refactored MockSlave creation to use cluster::Slave interfaces. - posted by Jie Yu <yu...@gmail.com> on 2017/11/29 00:37:43 UTC, 1 replies.
- Review Request 64151: Added an agent endpoint to list resource providers. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/29 11:21:32 UTC, 2 replies.
- Review Request 64158: Used helper functions instead of switches for resource extraction. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/29 13:52:24 UTC, 1 replies.
- Review Request 64163: Made sure only non-terminal operations use resources. - posted by Benjamin Bannier <be...@mesosphere.io> on 2017/11/29 16:27:45 UTC, 2 replies.
- Review Request 64168: Introduced a 'OperationUUID' type. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/29 17:07:54 UTC, 2 replies.
- Review Request 64142: Updated master ACCEPT handler for offer operation update acks. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/29 17:37:14 UTC, 0 replies.
- Re: Review Request 64142: Updated master ACCEPT handler to disallow offer operation feedback. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/29 17:40:27 UTC, 3 replies.
- Review Request 64144: Made master acknowledge offer operation updates when 'id' isn't set. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/29 17:47:42 UTC, 2 replies.
- Review Request 64143: Added ACKNOWLEDGE event to the resource provider API. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/29 17:50:55 UTC, 1 replies.
- Review Request 64146: Added offer operation update acknowledgement to the agent. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/29 17:54:03 UTC, 2 replies.
- Review Request 64145: Added status update acknowledgement to resource provider manager. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/29 18:00:09 UTC, 1 replies.
- Re: Review Request 64010: Added additional parameters to Allocator::updateSlave(). - posted by Benno Evers <be...@mesosphere.com> on 2017/11/29 18:17:58 UTC, 1 replies.
- Review Request 64169: Add checks for slave gone. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/29 18:29:24 UTC, 1 replies.
- Review Request 64170: Fall back to binding devices if mknod fails. - posted by James Peach <jp...@apache.org> on 2017/11/29 18:40:22 UTC, 1 replies.
- Re: Review Request 64012: Added new --reconfiguration_compatibility slave flag and implementation. - posted by Benno Evers <be...@mesosphere.com> on 2017/11/29 19:10:56 UTC, 1 replies.
- Review Request 64171: Fixed the multi taskgroup shared pid ns test due to slowness. - posted by Gilbert Song <so...@gmail.com> on 2017/11/29 19:19:13 UTC, 2 replies.
- Review Request 64176: Added a test for ContainerDaemon. - posted by Jie Yu <yu...@gmail.com> on 2017/11/29 21:35:57 UTC, 2 replies.
- Review Request 64177: Added a test in ContainerDaemon for testing failed authorization. - posted by Jie Yu <yu...@gmail.com> on 2017/11/29 22:03:15 UTC, 4 replies.
- Review Request 64180: Windows: Fixed `fs::symlink` to support relative and broken paths. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/29 22:45:00 UTC, 1 replies.
- Review Request 64181: Windows: Fixed `os::rmdir` bugs. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/29 22:45:40 UTC, 4 replies.
- Review Request 64182: Windows: Cleaned up `os::exists`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/29 22:45:53 UTC, 1 replies.
- Re: Review Request 63809: Windows: Fixed `fs::symlink` to not need admin privileges. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/29 22:46:42 UTC, 1 replies.
- Re: Review Request 63811: Windows: Added internal `get_handle_follow` which follows symlinks. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/29 22:47:49 UTC, 1 replies.
- Review Request 64183: Fixed Windows (and maybe clang) build break. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/29 23:14:23 UTC, 3 replies.
- Review Request 64187: Included `stout/os/realpath.hpp` in `flags.hpp`. - posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/11/30 00:05:24 UTC, 2 replies.
- Review Request 64188: Fixed Windows `HealthCheckTest.HealthyTaskViaTCP` test failure. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/30 00:41:57 UTC, 3 replies.
- Review Request 64194: Stripped AllocationInfo from offers to non-MULTI_ROLE schedulers. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/30 01:45:22 UTC, 0 replies.
- Review Request 64195: Updated the tests to use MULTI_ROLE frameworks by default. - posted by Benjamin Mahler <bm...@apache.org> on 2017/11/30 01:45:27 UTC, 0 replies.
- Review Request 64197: Tracked layers in memory for provisioned containers. - posted by Zhitao Li <zh...@gmail.com> on 2017/11/30 06:32:11 UTC, 0 replies.
- Re: Review Request 61172: Added mesos.http and mesos.exceptions for CLI. - posted by Eric Chung <ci...@gmail.com> on 2017/11/30 10:16:37 UTC, 0 replies.
- Re: Review Request 64151: Added a V1 API call to list resource providers. - posted by Jan Schlicht <ja...@mesosphere.io> on 2017/11/30 14:11:43 UTC, 0 replies.
- Review Request 64217: Added scheduler driver benchmarks. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/11/30 17:49:17 UTC, 3 replies.
- Review Request 64218: Terminated the Slave first before cleaning up its dependencies. - posted by Jie Yu <yu...@gmail.com> on 2017/11/30 18:13:55 UTC, 2 replies.
- Re: Review Request 63804: Added plumbing for master to reconcile offer operations with agent. - posted by Greg Mann <gr...@mesosphere.io> on 2017/11/30 18:20:25 UTC, 2 replies.
- Re: Review Request 59294: Optionally scale egress bandwidth with CPU. - posted by Ilya Pronin <ip...@twopensource.com> on 2017/11/30 19:12:15 UTC, 0 replies.
- Review Request 64226: Added a `ns::supported` convenience API. - posted by James Peach <jp...@apache.org> on 2017/11/30 22:03:35 UTC, 0 replies.
- Review Request 64232: Add a temporary filter for overlay backend related tests. - posted by Meng Zhu <mz...@mesosphere.io> on 2017/11/30 23:17:46 UTC, 0 replies.
- Review Request 64233: Removed some dead code in the master. - posted by Jie Yu <yu...@gmail.com> on 2017/11/30 23:38:07 UTC, 2 replies.
- Re: Review Request 63277: Windows: Ported `Cpu` and `Mem` isolator tests. - posted by Akash Gupta <ak...@hotmail.com> on 2017/11/30 23:53:40 UTC, 0 replies.