[checkmk-commits] [tribe29/checkmk] da8134: Do not define an unnamed namespace in a header file.

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: da81342b391d33f5ae0417bd1490466c39723c80 https://github.com/tribe29/checkmk/commit/da81342b391d33f5ae0417bd1490466c3… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2020-12-28 (Mon, 28 Dec 2020) Changed paths: M livestatus/src/Poller.h Log Message: ----------- Do not define an unnamed namespace in a header file. https://clang.llvm.org/extra/clang-tidy/checks/cert-dcl59-cpp.html Change-Id: Ib64886a40579e829ca1a631356e251c514ff25bf Commit: feb6abcba7a0747221e7509446f6641a09b17997 https://github.com/tribe29/checkmk/commit/feb6abcba7a0747221e7509446f6641a0… Author: Sven Panne <sven.panne(a)tribe29.com> Date: 2020-12-28 (Mon, 28 Dec 2020) Changed paths: M livestatus/src/Poller.h Log Message: ----------- Initialize all variables. Control flow analysis could figure it out that this variable is always assigned before being accessed, but somehow the linter doesn't seem to use CFA here. Change-Id: I8f8b38deb4c42dd2672ad8a25332c277dce497d4 Commit: 5d87bf6a8f85c064a81fe33b175364a1f032fef5 https://github.com/tribe29/checkmk/commit/5d87bf6a8f85c064a81fe33b175364a1f… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-12-28 (Mon, 28 Dec 2020) Changed paths: M tests/integration/cmk/base/test_automations.py Log Message: ----------- Make test accept another DNS resolver issue Depending on the DNS resolver settings the response is a bit different. Adding a new situation that can occur during manual tests. Change-Id: Idcb5df7a96d9701596bfd57a3391f0e6426ad9d3 Commit: eaa59ff97f9774a3bbe13c6b95f2bcb12a68c37d https://github.com/tribe29/checkmk/commit/eaa59ff97f9774a3bbe13c6b95f2bcb12… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-12-28 (Mon, 28 Dec 2020) Changed paths: M tests/integration/cmk/base/test_automations.py Log Message: ----------- Add more useful error message to failed automation tests Change-Id: I4158f438fe4f421a02ddc4d7175e4109f100cdcf Commit: 37178d2acb2c8e068d0bb30434ce7bb42f5ee43b https://github.com/tribe29/checkmk/commit/37178d2acb2c8e068d0bb30434ce7bb42… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-12-28 (Mon, 28 Dec 2020) Changed paths: M tests/testlib/web_session.py Log Message: ----------- Fix wrong file access when running tests undockered Change-Id: Ifb24786c81587cf9c087f7a5aa8d4616d18c1035 Commit: 809c0d0e4904ff896fc427c09627d96d8ade6505 https://github.com/tribe29/checkmk/commit/809c0d0e4904ff896fc427c09627d96d8… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-12-28 (Mon, 28 Dec 2020) Changed paths: M tests/integration/cmk/base/test_automations.py M tests/integration/cmk/base/test_check_variables.py M tests/integration/cmk/base/test_discovery_parameters.py M tests/integration/cmk/base/test_modes.py Log Message: ----------- Little more robust test setup finalization The cleanup code (after the yield) is not called in case some exception occurs during during activate changes, leaving configuration files of the fixture behind which will then break the test isolation. Would be better to separate single parts of the fixture into smaller fixtures to isolate them a bit more. Change-Id: I5d9602377692b7d92fb4c0422921a6a654fc4997 Compare: https://github.com/tribe29/checkmk/compare/317169d89bd3...809c0d0e4904