----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64195/#review192421 -----------------------------------------------------------
Ship it! src/tests/persistent_volume_tests.cpp Line 150 (original), 152 (patched) <https://reviews.apache.org/r/64195/#comment270565> This is to get a stable result from the `join` below, right? src/tests/slave_tests.cpp Line 860 (original), 860 (patched) <https://reviews.apache.org/r/64195/#comment270562> Huh... so these used to be an empty string? - Michael Park On Nov. 29, 2017, 5:45 p.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64195/ > ----------------------------------------------------------- > > (Updated Nov. 29, 2017, 5:45 p.m.) > > > Review request for mesos and Michael Park. > > > Bugs: MESOS-8237 > https://issues.apache.org/jira/browse/MESOS-8237 > > > Repository: mesos > > > Description > ------- > > Now that we strip the `Resource.allocation_info` for non-MULTI_ROLE > schedulers, it's simpler to default the tests to use the MULTI_ROLE > capability, since we've already updated the majority of the tests > to be aware of `Resource.allocation_info`. > > > Diffs > ----- > > src/examples/balloon_framework.cpp 0136344c160ea6aeacc57e61382640689a77ee1a > src/examples/disk_full_framework.cpp > 28f71c35b32b46c7bf2cce9fe971266f7bd73c8a > src/examples/dynamic_reservation_framework.cpp > 5ee3867228a70c8fa0b4b5092fb44cb556a670e0 > src/examples/long_lived_framework.cpp > e6742159108f9eae4f2548bb84d6495309737d9c > src/examples/no_executor_framework.cpp > fd920f58f5ccb903127e9c6b112be30f04baa069 > src/examples/persistent_volume_framework.cpp > 674d58a9e6fa34733f765d2925cedafeaab1eecd > src/examples/test_framework.cpp c6a293ed5f88bf4d73f23d97cbf9ffdd8b941f1a > src/examples/test_http_framework.cpp > 373035837f51e818629b1588d2ab8803c656f527 > src/tests/api_tests.cpp 66cb059c96ff9ecda594e13de9e5dc3909feeee3 > src/tests/containerizer/cgroups_isolator_tests.cpp > 421a72f1694a1a5c5e1bf4c525361dbd670c68af > src/tests/containerizer/docker_containerizer_tests.cpp > 7a42bb93508abf9020d0bcc5b3daa2f6834be2e3 > src/tests/containerizer/linux_filesystem_isolator_tests.cpp > 655f9f81f4bc55a85d7c5e27609a016254793380 > src/tests/default_executor_tests.cpp > 04d9e1b9d4682c3bb5355253e3a8e5b06d78e98f > src/tests/disk_quota_tests.cpp fc297995c42821aa4b0a7719a6a19c3bbf65db47 > src/tests/fault_tolerance_tests.cpp > 33a2220b4b4f019799db7b2482ce73b3454fa58e > src/tests/hook_tests.cpp 2e58d1133343a96b67d0816a54ff8eb874522ba3 > src/tests/master_allocator_tests.cpp > 3400d70bb0ba564eac43c4639eee0efd4d8059e6 > src/tests/master_authorization_tests.cpp > eff97f155e7472a0cd5994408ed73474392593ad > src/tests/master_quota_tests.cpp 058f6d24da50cbf3c28b091afa88f634a8102b62 > src/tests/master_tests.cpp 01f45a9bb6378c5fded31bf58fbcad93fe7ed719 > src/tests/master_validation_tests.cpp > 0e1c8b490ebe10bc50b8b6b530fa85128b967584 > src/tests/mesos.hpp f02c7c6962c9d0bee57712d8aad9997582c7404b > src/tests/oversubscription_tests.cpp > 2f403d3c9d2df7d5425b335781e8b62153854ef3 > src/tests/partition_tests.cpp e49c474167076b4136a161ed29b11db9a13455a7 > src/tests/persistent_volume_endpoints_tests.cpp > 883192d2badf8af87c95bc9a5fc3e711cd240193 > src/tests/persistent_volume_tests.cpp > b11d26e45c184cf3f0623629b898f67dc95c4fa0 > src/tests/reservation_endpoints_tests.cpp > a96cc714fd40421a7db9a4c0debf71546daf5d4c > src/tests/reservation_tests.cpp fa375cd45e4c68551801521176632a56acd6d0ea > src/tests/resource_provider_manager_tests.cpp > 0b7c4ad6bb0052847b884959e3171cd7ab382b45 > src/tests/role_tests.cpp 084555ae70b80330cdc8729cd62f8baddf291a01 > src/tests/scheduler_http_api_tests.cpp > 80e52fb22dbba4a0703f1b192205acaff89f883b > src/tests/scheduler_tests.cpp 45fc9c0cfccdb22c2e3e8d5de30c04575814a0e9 > src/tests/slave_authorization_tests.cpp > 11fd0d4e35523eca23a9603ecef0c9d0b65dde38 > src/tests/slave_recovery_tests.cpp f14c6ef69eb20a03454c8197df79b572a3c6d050 > src/tests/slave_tests.cpp a2274b691cf94b003c4bc15450d176a9c73517d5 > src/tests/upgrade_tests.cpp 7f434dbba858f636719eec24e92b306b76430c4c > > > Diff: https://reviews.apache.org/r/64195/diff/1/ > > > Testing > ------- > > Ran the tests, including with root. > > > Thanks, > > Benjamin Mahler > >