Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72217/#review219871 --- Ship it! Ship It! - Andrei Budnik On Март 10, 2020, 9:44 д.п., Greg Mann wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72217/ > --- > > (Updated Март 10, 2020, 9:44 д.п.) > > > Review request for mesos, Andrei Budnik and Qian Zhang. > > > Bugs: MESOS-10045 > https://issues.apache.org/jira/browse/MESOS-10045 > > > Repository: mesos > > > Description > --- > > Added tests for resource limit and shared cgroup validation. > > > Diffs > - > > src/tests/master_validation_tests.cpp > 8d5e74e82a2f9093b69120c1dbba33c56e25966b > > > Diff: https://reviews.apache.org/r/72217/diff/2/ > > > Testing > --- > > `make check` > > > Thanks, > > Greg Mann > >
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72217/#review219867 --- Bad patch! Reviews applied: [72161, 72162, 71858, 71884, 71885, 71886, 71943, 71944, 71950, 71951, 71952, 71953, 71955, 72216, 72217] Failed command: ['bash', '-c', "set -o pipefail; export OS='ubuntu:16.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers --disable-parallel-test-execution' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/jenkins/buildbot.sh 2>&1 | tee build_72217"] Error: .. ACKAGE_URL=\"\" -DPACKAGE=\"mesos\" -DVERSION=\"1.10.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_CXX11=1 -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DHAVE_OPENSSL_SSL_H=1 -DHAVE_FTS_H=1 -DHAVE_APR_POOLS_H=1 -DHAVE_LIBAPR_1=1 -DHAVE_LIBCURL=1 -DMESOS_HAS_JAVA=1 -DENABLE_NVML=1 -DHAVE_LIBSASL2=1 -DHAVE_SVN_VERSION_H=1 -DHAVE_LIBSVN_SUBR_1=1 -DHAVE_SVN_DELTA_H=1 -DHAVE_LIBSVN_DELTA_1=1 -DHAVE_ZLIB_H=1 -DHAVE_LIBZ=1 -DHAVE_PYTHON=\"2.7\" -DMESOS_HAS_PYTHON=1 -I. -I../../../../3rdparty/stout -I../../../../3rdparty/stout/include -I../boost-1.65.0 -I../elfio-3.2 -I../glog-0.4.0/src -I../googletest-release-1.8.0/googlemock/include -I../googletest-release-1.8.0/googletest/include -I../libarchive-3.3.2/libarchive -D__STDC_FORMAT_MACROS -I../picojson-1.3.0 -I../protobuf-3.5.0/src -I../rapidjson-1.1. 0/include -I/usr/include/subversion-1 -I/usr/include/apr-1 -I/usr/include/apr-1.0-Wall -Wsign-compare -Wformat-security -fstack-protector-strong -fPIC -fPIE -g1 -O0 -Wno-unused-local-typedefs -std=c++11 -c -o stout_tests-ip_tests.o `test -f 'tests/ip_tests.cpp' || echo '../../../../3rdparty/stout/'`tests/ip_tests.cpp g++ -DPACKAGE_NAME=\"mesos\" -DPACKAGE_TARNAME=\"mesos\" -DPACKAGE_VERSION=\"1.10.0\" -DPACKAGE_STRING=\"mesos\ 1.10.0\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE_URL=\"\" -DPACKAGE=\"mesos\" -DVERSION=\"1.10.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_CXX11=1 -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DHAVE_OPENSSL_SSL_H=1 -DHAVE_FTS_H=1 -DHAVE_APR_POOLS_H=1 -DHAVE_LIBAPR_1=1 -DHAVE_LIBCURL=1 -DMESOS_HAS_JAVA=1 -DENABLE_NVML=1 -DHAVE_LIBSASL2=1 -DHAVE_SVN_VERSION_H=1 -DHAVE_LIBSVN_SUBR_1=1 -DHAVE_SVN_DELTA_H=1 -DHAVE_LIBSVN_DELTA_1=1 -DHAVE_ZLIB_H=1 -DHAVE_LIBZ=1 -DHAVE_PYTHON=\"2.7\" -DMESOS_HAS_PYTHON=1 -I. -I../../../../3rdparty/stout -I../../../../3rdparty/stout/include -I../boost-1.65.0 -I../elfio-3.2 -I../glog-0.4.0/src -I../googletest-release-1.8.0/googlemock/include -I../googletest-rel ease-1.8.0/googletest/include -I../libarchive-3.3.2/libarchive -D__STDC_FORMAT_MACROS -I../picojson-1.3.0 -I../protobuf-3.5.0/src -I../rapidjson-1.1.0/include -I/usr/include/subversion-1 -I/usr/include/apr-1 -I/usr/include/apr-1.0 -Wall -Wsign-compare -Wformat-security -fstack-protector-strong -fPIC -fPIE -g1 -O0 -Wno-unused-local-typedefs -std=c++11 -c -o stout_tests-json_tests.o `test -f 'tests/json_tests.cpp' || echo '../../../../3rdparty/stout/'`tests/json_tests.cpp g++ -DPACKAGE_NAME=\"mesos\" -DPACKAGE_TARNAME=\"mesos\" -DPACKAGE_VERSION=\"1.10.0\" -DPACKAGE_STRING=\"mesos\ 1.10.0\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE_URL=\"\" -DPACKAGE=\"mesos\" -DVERSION=\"1.10.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_CXX11=1 -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DHAVE_OPENSSL_SSL_H=1 -DHAVE_FTS_H=1 -DHAVE_APR_POOLS_H=1 -DHAVE_LIBAPR_1=1 -DHAVE_LIBCURL=1 -DMESOS_HAS_JAVA=1 -DENABLE_NVML=1 -DHAVE_LIBSASL2=1 -DHAVE_SVN_VERSION_H=1 -DHAVE_LIBSVN_SUBR_1=1 -DHAVE_SVN_DELTA_H=1 -DHAVE_LIBSVN_DELTA_1=1 -DHAVE_ZLIB_H=1 -DHAVE_LIBZ=1 -DHAVE_PYTHON=\"2.7\" -DMESOS_HAS_PYTHON=1 -I. -I../../../../3rdparty/stout -I../../../../3rdparty/stout/include -I../boost-1.65.0 -I../elfio-3.2 -I../glog-0.4.0/src -I../googletest-release-1.8.0/googlemock/include -I../googletest-rel ease-1.8.0/googletest/include -I../libarchive-3.3.2/libarchive -D__STDC_FORMAT_MACROS -I../picojson-1.3.0 -I../protobuf-3.5.0/src -I../rapidjson-1.1.0/include -I/usr/include/subversion-1 -I/usr/include/apr-1 -I/usr/include/apr-1.0 -Wall -Wsign-compare -Wformat-security -fstack-protector-strong -fPIC -fPIE -g1 -O0 -Wno-unused-local-typedefs -std=c++11 -c -o stout_tests-jsonify_tests.o `test -f 'tests/jsonify_tests.cpp'
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72217/ --- (Updated March 10, 2020, 9:44 a.m.) Review request for mesos, Andrei Budnik and Qian Zhang. Bugs: MESOS-10045 https://issues.apache.org/jira/browse/MESOS-10045 Repository: mesos Description --- Added tests for resource limit and shared cgroup validation. Diffs (updated) - src/tests/master_validation_tests.cpp 8d5e74e82a2f9093b69120c1dbba33c56e25966b Diff: https://reviews.apache.org/r/72217/diff/2/ Changes: https://reviews.apache.org/r/72217/diff/1-2/ Testing --- `make check` Thanks, Greg Mann
Re: Review Request 72217: Added tests for resource limit and shared cgroup validation.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72217/#review219850 --- Bad patch! Reviews applied: [72161, 72162, 71858, 71884, 71885, 71886, 71943, 71944, 71950, 71951, 71952, 71953, 71955, 72215, 72216, 72217] Failed command: ['bash', '-c', "set -o pipefail; export OS='ubuntu:16.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers --disable-parallel-test-execution' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/jenkins/buildbot.sh 2>&1 | tee build_72217"] Error: .. KAGE_URL=\"\" -DPACKAGE=\"mesos\" -DVERSION=\"1.10.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_CXX11=1 -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DHAVE_OPENSSL_SSL_H=1 -DHAVE_FTS_H=1 -DHAVE_APR_POOLS_H=1 -DHAVE_LIBAPR_1=1 -DHAVE_LIBCURL=1 -DMESOS_HAS_JAVA=1 -DENABLE_NVML=1 -DHAVE_LIBSASL2=1 -DHAVE_SVN_VERSION_H=1 -DHAVE_LIBSVN_SUBR_1=1 -DHAVE_SVN_DELTA_H=1 -DHAVE_LIBSVN_DELTA_1=1 -DHAVE_ZLIB_H=1 -DHAVE_LIBZ=1 -DHAVE_PYTHON=\"2.7\" -DMESOS_HAS_PYTHON=1 -I. -I../../../../3rdparty/stout -I../../../../3rdparty/stout/include -I../boost-1.65.0 -I../elfio-3.2 -I../glog-0.4.0/src -I../googletest-release-1.8.0/googlemock/include -I../googletest-release-1.8.0/googletest/include -I../libarchive-3.3.2/libarchive -D__STDC_FORMAT_MACROS -I../picojson-1.3.0 -I../protobuf-3.5.0/src -I../rapidjson-1.1.0/ include -I/usr/include/subversion-1 -I/usr/include/apr-1 -I/usr/include/apr-1.0 -Wall -Wsign-compare -Wformat-security -fstack-protector-strong -fPIC -fPIE -g1 -O0 -Wno-unused-local-typedefs -std=c++11 -c -o stout_tests-ip_tests.o `test -f 'tests/ip_tests.cpp' || echo '../../../../3rdparty/stout/'`tests/ip_tests.cpp g++ -DPACKAGE_NAME=\"mesos\" -DPACKAGE_TARNAME=\"mesos\" -DPACKAGE_VERSION=\"1.10.0\" -DPACKAGE_STRING=\"mesos\ 1.10.0\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE_URL=\"\" -DPACKAGE=\"mesos\" -DVERSION=\"1.10.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_CXX11=1 -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DHAVE_OPENSSL_SSL_H=1 -DHAVE_FTS_H=1 -DHAVE_APR_POOLS_H=1 -DHAVE_LIBAPR_1=1 -DHAVE_LIBCURL=1 -DMESOS_HAS_JAVA=1 -DENABLE_NVML=1 -DHAVE_LIBSASL2=1 -DHAVE_SVN_VERSION_H=1 -DHAVE_LIBSVN_SUBR_1=1 -DHAVE_SVN_DELTA_H=1 -DHAVE_LIBSVN_DELTA_1=1 -DHAVE_ZLIB_H=1 -DHAVE_LIBZ=1 -DHAVE_PYTHON=\"2.7\" -DMESOS_HAS_PYTHON=1 -I. -I../../../../3rdparty/stout -I../../../../3rdparty/stout/include -I../boost-1.65.0 -I../elfio-3.2 -I../glog-0.4.0/src -I../googletest-release-1.8.0/googlemock/include -I../googletest-rel ease-1.8.0/googletest/include -I../libarchive-3.3.2/libarchive -D__STDC_FORMAT_MACROS -I../picojson-1.3.0 -I../protobuf-3.5.0/src -I../rapidjson-1.1.0/include -I/usr/include/subversion-1 -I/usr/include/apr-1 -I/usr/include/apr-1.0 -Wall -Wsign-compare -Wformat-security -fstack-protector-strong -fPIC -fPIE -g1 -O0 -Wno-unused-local-typedefs -std=c++11 -c -o stout_tests-json_tests.o `test -f 'tests/json_tests.cpp' || echo '../../../../3rdparty/stout/'`tests/json_tests.cpp g++ -DPACKAGE_NAME=\"mesos\" -DPACKAGE_TARNAME=\"mesos\" -DPACKAGE_VERSION=\"1.10.0\" -DPACKAGE_STRING=\"mesos\ 1.10.0\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE_URL=\"\" -DPACKAGE=\"mesos\" -DVERSION=\"1.10.0\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_CXX11=1 -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DHAVE_OPENSSL_SSL_H=1 -DHAVE_FTS_H=1 -DHAVE_APR_POOLS_H=1 -DHAVE_LIBAPR_1=1 -DHAVE_LIBCURL=1 -DMESOS_HAS_JAVA=1 -DENABLE_NVML=1 -DHAVE_LIBSASL2=1 -DHAVE_SVN_VERSION_H=1 -DHAVE_LIBSVN_SUBR_1=1 -DHAVE_SVN_DELTA_H=1 -DHAVE_LIBSVN_DELTA_1=1 -DHAVE_ZLIB_H=1 -DHAVE_LIBZ=1 -DHAVE_PYTHON=\"2.7\" -DMESOS_HAS_PYTHON=1 -I. -I../../../../3rdparty/stout -I../../../../3rdparty/stout/include -I../boost-1.65.0 -I../elfio-3.2 -I../glog-0.4.0/src -I../googletest-release-1.8.0/googlemock/include -I../googletest-rel ease-1.8.0/googletest/include -I../libarchive-3.3.2/libarchive -D__STDC_FORMAT_MACROS -I../picojson-1.3.0 -I../protobuf-3.5.0/src -I../rapidjson-1.1.0/include -I/usr/include/subversion-1 -I/usr/include/apr-1 -I/usr/include/apr-1.0 -Wall -Wsign-compare -Wformat-security -fstack-protector-strong -fPIC -fPIE -g1 -O0 -Wno-unused-local-typedefs -std=c++11 -c -o stout_tests-jsonify_tests.o `test -f
Review Request 72217: Added tests for resource limit and shared cgroup validation.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72217/ --- Review request for mesos, Andrei Budnik and Qian Zhang. Bugs: MESOS-10045 https://issues.apache.org/jira/browse/MESOS-10045 Repository: mesos Description --- Added tests for resource limit and shared cgroup validation. Diffs - src/tests/master_validation_tests.cpp 8d5e74e82a2f9093b69120c1dbba33c56e25966b Diff: https://reviews.apache.org/r/72217/diff/1/ Testing --- `make check` Thanks, Greg Mann