| Mesos Reviewbot Windows |
Re: Review Request 69116: Added 'popen_tty' to test util functions for the new CLI. |
Thu, 01 Nov, 00:17 |
| Mesos Reviewbot |
Re: Review Request 69116: Added 'popen_tty' to test util functions for the new CLI. |
Thu, 01 Nov, 00:45 |
| Chun-Hung Hsiao |
Review Request 69224: Fixed a test flake in `HealthCheckTest.HealthyTaskNonShell`. |
Thu, 01 Nov, 03:15 |
| Mesos Reviewbot Windows |
Re: Review Request 69224: Fixed a test flake in `HealthCheckTest.HealthyTaskNonShell`. |
Thu, 01 Nov, 04:45 |
| Alexander Rukletsov |
Re: Review Request 69224: Fixed a test flake in `HealthCheckTest.HealthyTaskNonShell`. |
Thu, 01 Nov, 06:38 |
| Till Toenshoff via Review Board |
Re: Review Request 69224: Fixed a test flake in `HealthCheckTest.HealthyTaskNonShell`. |
Thu, 01 Nov, 16:17 |
| James Peach |
Review Request 69233: Fixed a GetContainers crash when using XFS disk isolation. |
Thu, 01 Nov, 20:33 |
| Jiang Yan Xu |
Re: Review Request 69233: Fixed a GetContainers crash when using XFS disk isolation. |
Thu, 01 Nov, 20:55 |
| Qian Zhang |
Review Request 69234: Made debug container runs as its parent container's user by default. |
Thu, 01 Nov, 21:25 |
| Qian Zhang |
Review Request 69235: Added a test `ROOT_UNPRIVILEGED_USER_DefaultExecutorCommandHealthCheck`. |
Thu, 01 Nov, 21:28 |
| Qian Zhang |
Review Request 69236: Fixed a coding error that a test waited on a wrong task status update. |
Thu, 01 Nov, 21:29 |
| Gilbert Song |
Re: Review Request 69236: Fixed a coding error that a test waited on a wrong task status update. |
Thu, 01 Nov, 21:41 |
| Mesos Reviewbot Windows |
Re: Review Request 69233: Fixed a GetContainers crash when using XFS disk isolation. |
Thu, 01 Nov, 22:07 |
| Kevin Klues |
Re: Review Request 69115: Added test for interactive 'task exec'. |
Thu, 01 Nov, 22:26 |
| Kevin Klues |
Re: Review Request 69115: Added test for interactive 'task exec'. |
Thu, 01 Nov, 22:28 |
| Kevin Klues |
Re: Review Request 69115: Added test for interactive 'task exec'. |
Thu, 01 Nov, 22:28 |
| Armand Grillet |
Review Request 69237: Simplified 'test_exec' in new CLI to better manage newlines. |
Thu, 01 Nov, 22:28 |
| Mesos Reviewbot Windows |
Re: Review Request 69236: Fixed a coding error that a test waited on a wrong task status update. |
Thu, 01 Nov, 23:04 |
| Mesos Reviewbot Windows |
Re: Review Request 69237: Simplified 'test_exec' in new CLI to better manage newlines. |
Fri, 02 Nov, 00:30 |
| Mesos Reviewbot |
Re: Review Request 69233: Fixed a GetContainers crash when using XFS disk isolation. |
Fri, 02 Nov, 01:31 |
| Mesos Reviewbot |
Re: Review Request 69236: Fixed a coding error that a test waited on a wrong task status update. |
Fri, 02 Nov, 03:43 |
| Jiang Yan Xu |
Re: Review Request 68706: Added master failover reregistration progress metrics. |
Fri, 02 Nov, 08:19 |
| Jiang Yan Xu |
Re: Review Request 68706: Added master failover reregistration progress metrics. |
Fri, 02 Nov, 08:24 |
| Benjamin Bannier |
Review Request 69242: Saved some disk space in mesos-tidy Docker image. |
Fri, 02 Nov, 20:50 |
| Benjamin Bannier |
Review Request 69243: Used a `switch` statement to match enum values. |
Fri, 02 Nov, 20:51 |
| Gilbert Song |
Re: Review Request 69234: Made debug container runs as its parent container's user by default. |
Fri, 02 Nov, 21:38 |
| Benjamin Mahler |
Re: Review Request 69092: Added default arguments to `FrameworkProfile` in allocator benchmark. |
Fri, 02 Nov, 21:57 |
| Benjamin Mahler |
Re: Review Request 69092: Added default arguments to `FrameworkProfile` in allocator benchmark. |
Fri, 02 Nov, 21:58 |
| Benjamin Mahler |
Re: Review Request 69093: Removed `used` argument in `AgentProfile` in the allocator benchmark. |
Fri, 02 Nov, 21:59 |
| Benjamin Mahler |
Re: Review Request 69093: Removed `used` argument in `AgentProfile` in the allocator benchmark. |
Fri, 02 Nov, 21:59 |
| Benjamin Mahler |
Re: Review Request 69096: Moved a few allocator test helpers to `tests/allocator.hpp`. |
Fri, 02 Nov, 22:02 |
| Mesos Reviewbot Windows |
Re: Review Request 69242: Saved some disk space in mesos-tidy Docker image. |
Fri, 02 Nov, 22:16 |
| Mesos Reviewbot Windows |
Re: Review Request 69243: Used a `switch` statement to match enum values. |
Fri, 02 Nov, 22:19 |
| Benjamin Mahler |
Re: Review Request 69097: Added an allocator benchmark for quota performance. |
Fri, 02 Nov, 22:22 |
| Chun-Hung Hsiao |
Re: Review Request 69036: WIP: Changed the semantics of `CREATE_DISK` and `DESTROY_DISK` operations. |
Fri, 02 Nov, 22:22 |
| Benjamin Mahler |
Re: Review Request 69098: Added a benchmark to compare quota and nonquota allocation performance. |
Fri, 02 Nov, 22:23 |
| Mesos Reviewbot Windows |
Re: Review Request 69207: Moved 'updated_tasks()' to new CLI tests base. |
Fri, 02 Nov, 22:23 |
| Chun-Hung Hsiao |
Re: Review Request 69242: Saved some disk space in mesos-tidy Docker image. |
Fri, 02 Nov, 22:26 |
| Chun-Hung Hsiao |
Re: Review Request 69243: Used a `switch` statement to match enum values. |
Fri, 02 Nov, 22:53 |
| Kevin Klues |
Re: Review Request 69116: Added 'popen_tty()' to test util functions for the new CLI. |
Fri, 02 Nov, 23:16 |
| Kevin Klues |
Re: Review Request 69207: Moved 'updated_tasks()' to new CLI tests base. |
Fri, 02 Nov, 23:21 |
| Chun-Hung Hsiao |
Re: Review Request 69161: Renamed a function argument to not reuse member name. |
Fri, 02 Nov, 23:35 |
| Kevin Klues |
Re: Review Request 69207: Moved 'updated_tasks()' to new CLI tests base. |
Fri, 02 Nov, 23:36 |
| Kevin Klues |
Re: Review Request 69237: Simplified newline handling in 'test_exec()' test for new CLI. |
Fri, 02 Nov, 23:39 |
| Mesos Reviewbot Windows |
Re: Review Request 69116: Added 'popen_tty()' to test util functions for the new CLI. |
Fri, 02 Nov, 23:44 |
| Kevin Klues |
Re: Review Request 69206: Updated new CLI commands for new CLI to return proper exit status. |
Fri, 02 Nov, 23:45 |
| Mesos Reviewbot Windows |
Re: Review Request 69237: Simplified newline handling in 'test_exec()' test for new CLI. |
Fri, 02 Nov, 23:45 |
| Mesos Reviewbot Windows |
Re: Review Request 69208: Updated new CLI task attach/exec exit strategy. |
Fri, 02 Nov, 23:51 |
| Kevin Klues |
Re: Review Request 69208: Updated new CLI task attach/exec exit strategy. |
Sat, 03 Nov, 00:04 |
| Mesos Reviewbot Windows |
Re: Review Request 69211: Improved the code comments for `getContainerDevicesPath`. |
Sat, 03 Nov, 01:22 |
| Mesos Reviewbot |
Re: Review Request 69242: Saved some disk space in mesos-tidy Docker image. |
Sat, 03 Nov, 01:54 |
| Mesos Reviewbot |
Re: Review Request 69243: Used a `switch` statement to match enum values. |
Sat, 03 Nov, 04:23 |
| James Peach |
Re: Review Request 69086: Moved container root construction to the isolators. |
Sat, 03 Nov, 18:49 |
| GitBox |
[GitHub] mlowicki opened a new pull request #318: Add info about Rhythm framework |
Sat, 03 Nov, 22:39 |
| GitBox |
[GitHub] vinodkone closed pull request #318: Add info about Rhythm framework |
Sun, 04 Nov, 04:28 |
| Alexander Rukletsov |
Re: Review Request 68131: Added MasterActorResponsiveness_BENCHMARK_Test. |
Sun, 04 Nov, 04:31 |
| Mesos Reviewbot Windows |
Re: Review Request 68131: Added MasterActorResponsiveness_BENCHMARK_Test. |
Sun, 04 Nov, 06:07 |
| Mesos Reviewbot |
Re: Review Request 68131: Added MasterActorResponsiveness_BENCHMARK_Test. |
Sun, 04 Nov, 07:04 |
| Jie Yu |
Re: Review Request 69086: Moved container root construction to the isolators. |
Mon, 05 Nov, 05:14 |
| Qian Zhang |
Re: Review Request 69234: Made nested container runs as its parent container's user by default. |
Mon, 05 Nov, 08:02 |
| Armand Grillet |
Review Request 69255: Updated PyInstaller requirement for new CLI to support Python 3.7. |
Mon, 05 Nov, 18:13 |
| Mesos Reviewbot Windows |
Re: Review Request 69255: Updated PyInstaller requirement for new CLI to support Python 3.7. |
Mon, 05 Nov, 19:52 |
| Benjamin Bannier |
Re: Review Request 69255: Updated PyInstaller requirement for new CLI to support Python 3.7. |
Mon, 05 Nov, 20:48 |
| Armand Grillet |
Re: Review Request 69255: Updated PyInstaller requirement for new CLI to support Python 3.7. |
Mon, 05 Nov, 22:22 |
| Benjamin Mahler |
Re: Review Request 67955: Added some new generic flag parsers. |
Tue, 06 Nov, 05:36 |
| Benjamin Mahler |
Re: Review Request 67956: Removed some generic flag parsers that are now in stout. |
Tue, 06 Nov, 05:36 |
| Benjamin Mahler |
Re: Review Request 67957: Refactored TemporaryDirectoryTest to be a mixin. |
Tue, 06 Nov, 05:49 |
| Benjamin Mahler |
Re: Review Request 67958: Added LibprocessTest for easily configuring the library for a test. |
Tue, 06 Nov, 05:58 |
| Benjamin Bannier |
Re: Review Request 67958: Added LibprocessTest for easily configuring the library for a test. |
Tue, 06 Nov, 06:07 |
| James Peach |
Re: Review Request 69086: Moved container root construction to the isolators. |
Tue, 06 Nov, 06:07 |
| Benjamin Mahler |
Re: Review Request 67959: Added support for instrumenting processes. |
Tue, 06 Nov, 06:13 |
| Benno Evers |
Re: Review Request 68131: Added MasterActorResponsiveness_BENCHMARK_Test. |
Tue, 06 Nov, 12:19 |
| Benjamin Mahler |
Re: Review Request 67958: Added LibprocessTest for easily configuring the library for a test. |
Tue, 06 Nov, 15:41 |
| Joseph Wu |
Review Request 69267: Fixed flaky SchedulerTest.MasterFailover. |
Wed, 07 Nov, 01:26 |
| Mesos Reviewbot Windows |
Re: Review Request 69211: Improved the code comments for `getContainerDevicesPath`. |
Wed, 07 Nov, 01:44 |
| Mesos Reviewbot Windows |
Re: Review Request 69267: Fixed flaky SchedulerTest.MasterFailover. |
Wed, 07 Nov, 03:17 |
| Benno Evers |
Re: Review Request 67022: Refreshed checkpointed SlaveInfo if `--reconfiguration_policy==any`. |
Wed, 07 Nov, 14:25 |
| Benno Evers |
Review Request 69272: Clarified some comments about the usage of offer filters. |
Wed, 07 Nov, 18:40 |
| Benno Evers |
Review Request 69273: Fixed flaky agent reconfiguration test. |
Wed, 07 Nov, 18:40 |
| Mesos Reviewbot Windows |
Re: Review Request 69273: Fixed flaky agent reconfiguration test. |
Wed, 07 Nov, 20:12 |
| Benjamin Bannier |
Review Request 69274: Reduced default oversubscription in parallel test runner. |
Wed, 07 Nov, 20:30 |
| Benjamin Bannier |
Review Request 69275: Added environment sanity check to parallel test runner. |
Wed, 07 Nov, 20:30 |
| Mesos Reviewbot |
Re: Review Request 67022: Refreshed checkpointed SlaveInfo if `--reconfiguration_policy==any`. |
Wed, 07 Nov, 21:11 |
| Benjamin Mahler |
Re: Review Request 69274: Reduced default oversubscription in parallel test runner. |
Wed, 07 Nov, 21:13 |
| Benjamin Mahler |
Re: Review Request 69275: Added environment sanity check to parallel test runner. |
Wed, 07 Nov, 21:14 |
| Armand Grillet |
Re: Review Request 69275: Added environment sanity check to parallel test runner. |
Wed, 07 Nov, 21:15 |
| Armand Grillet |
Re: Review Request 69274: Reduced default oversubscription in parallel test runner. |
Wed, 07 Nov, 21:16 |
| Benjamin Mahler |
Review Request 69276: Fixed flakiness in `FsTest.Used`. |
Wed, 07 Nov, 21:16 |
| Benjamin Bannier |
Review Request 69278: Added hierchical allocator benchmarks to the cmake build. |
Wed, 07 Nov, 21:17 |
| Benjamin Mahler |
Review Request 69277: Updated autotools build to use parallel test runner by default. |
Wed, 07 Nov, 21:19 |
| Jason Lai |
Re: Review Request 65811: Added Stout `path::normalize` function for POSIX paths. |
Wed, 07 Nov, 21:22 |
| Jason Lai |
Re: Review Request 68804: Added Stout `os::readlink` function for POSIX paths. |
Wed, 07 Nov, 21:22 |
| Jason Lai |
Re: Review Request 65812: Added an overloaded version of `os::realpath` to stout |
Wed, 07 Nov, 21:23 |
| James Peach |
Re: Review Request 69276: Fixed flakiness in `FsTest.Used`. |
Wed, 07 Nov, 21:24 |
| Jason Lai |
Re: Review Request 67175: Added support for marking slave mounts. |
Wed, 07 Nov, 21:24 |
| Jason Lai |
Re: Review Request 68832: Add unit tests for Stout `path::normalize` function in POSIX. |
Wed, 07 Nov, 21:26 |
| Joseph Wu |
Re: Review Request 69276: Fixed flakiness in `FsTest.Used`. |
Wed, 07 Nov, 21:27 |
| Vinod Kone |
Review Request 69279: Updated Build bot to write out test report xml files. |
Wed, 07 Nov, 21:28 |
| Joseph Wu |
Re: Review Request 69278: Added hierchical allocator benchmarks to the cmake build. |
Wed, 07 Nov, 21:29 |
| Benjamin Bannier |
Re: Review Request 69277: Updated autotools build to use parallel test runner by default. |
Wed, 07 Nov, 21:30 |