----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64436/ -----------------------------------------------------------
Review request for mesos and Greg Mann. Repository: mesos Description ------- The default executor authentication mechanism depends on SSL support being enabled in the build. Multiple tests that invoke the default executor have compilation time checks for SSL support so they can disable the Mesos agent authenticate_http_readwrite flag in this case. Rather than duplicating this check to multiple tests, set it to an appropriate default in the MesosTest fixture. Diffs ----- src/tests/api_tests.cpp 53c705ed4775846ba7eb42da57f962895f5669aa src/tests/check_tests.cpp 2a71f41bad1a7eea2763addf5fc35d648bd46cc6 src/tests/container_daemon_tests.cpp 6e5efc952edeab0a03b33dcff81fec484126d068 src/tests/containerizer/environment_secret_isolator_tests.cpp fd1cd468bd1e4c2455bb3315b085d38b2543bc6f src/tests/containerizer/linux_capabilities_isolator_tests.cpp d3c456826db2c1844ebfd55acbd766d6fb7eaede src/tests/containerizer/nvidia_gpu_isolator_tests.cpp 5e40a940770cb45c9ac181b953a4e57e4b053445 src/tests/containerizer/ports_isolator_tests.cpp 343803028afce01874a054879f5ec39f2b85286a src/tests/containerizer/posix_rlimits_isolator_tests.cpp f639cac900f8ac5d0bc2eb856e90cb4881d85ec3 src/tests/containerizer/volume_host_path_isolator_tests.cpp 271bbbe71f16388d1331a9db120ba71b86cefa5f src/tests/default_executor_tests.cpp d3e3ef770656a36f3469f2ec655102a05adcae8a src/tests/health_check_tests.cpp 56721fac4a464f3cad40dbf4e6bc4fb7b1a780d4 src/tests/master_slave_reconciliation_tests.cpp 1a9b40a9a269313fc6c858ab5b2a9901c5e12352 src/tests/master_tests.cpp 0dfa8c716bc35a386dd7fa87b595360487ab8b15 src/tests/mesos.cpp 7ed08a5ac2a0efdbd2a93fb48a70a23d0973dc51 src/tests/scheduler_tests.cpp ecfce4d112464a76c14603e40c20585ec25e0ca7 Diff: https://reviews.apache.org/r/64436/diff/1/ Testing ------- make check (Fedora 27) Thanks, James Peach
