[checkmk-commits] [tribe29/checkmk] b2d2a9: KUBE: fix false naming due copy-pasta in kube_name...

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: b2d2a9ff0ef5292e4a33c0e069e72f33a11de263 https://github.com/tribe29/checkmk/commit/b2d2a9ff0ef5292e4a33c0e069e72f33a… Author: Solomon Jacobs <solomon.jacobs(a)tribe29.com> Date: 2022-04-28 (Thu, 28 Apr 2022) Changed paths: M cmk/base/plugins/agent_based/kube_namespace_info.py M tests/unit/cmk/base/plugins/agent_based/test_kube_namespace_info.py Log Message: ----------- KUBE: fix false naming due copy-pasta in kube_namespace_info Change-Id: I07569d66299040862bfc31f3b6ca2496670ea333 Commit: f9936df383d35ec657288dc7d5b4bd088b854800 https://github.com/tribe29/checkmk/commit/f9936df383d35ec657288dc7d5b4bd088… Author: Solomon Jacobs <solomon.jacobs(a)tribe29.com> Date: 2022-04-28 (Thu, 28 Apr 2022) Changed paths: M cmk/base/plugins/agent_based/utils/kube.py M cmk/special_agents/utils_kubernetes/schemata/api.py Log Message: ----------- transform: fix incorrect schemata docstrings for Labels Fix consists of two parts: - validation_value was incorrect, because it excluded empty values. - Changes in cmk/special_agents/utils_kubernetes/schemata/api.py should match those in cmk/base/agent_based/plugins/utils/kube.py They now match. CMK-9882 Change-Id: Ie965f0d6ea776fc2553eff6343c10800cc351a40 Commit: 1a5fd6ce245a670a0edc6cbef101b933ec11c4f8 https://github.com/tribe29/checkmk/commit/1a5fd6ce245a670a0edc6cbef101b933e… Author: Solomon Jacobs <solomon.jacobs(a)tribe29.com> Date: 2022-04-28 (Thu, 28 Apr 2022) Changed paths: M tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kube_deployments.py A tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kube_namespace.py M tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kubernetes_api_nodes.py M tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kubernetes_api_pods.py Log Message: ----------- KUBE: add test parse_metadata parse_metadata is now tested for Deployments and Pods. CMK-9882 Change-Id: I01aacf6485682422b7708a07595142f028899c5d Commit: b90c5d7832d6aab142e6894efd959ee8dc6aa385 https://github.com/tribe29/checkmk/commit/b90c5d7832d6aab142e6894efd959ee8d… Author: Solomon Jacobs <solomon.jacobs(a)tribe29.com> Date: 2022-04-28 (Thu, 28 Apr 2022) Changed paths: M tests/unit/cmk/special_agents/agent_kube/conftest.py M tests/unit/cmk/special_agents/agent_kube/factory.py M tests/unit/cmk/special_agents/agent_kube/test_node.py Log Message: ----------- KUBE: move api_to_agent_pod out of conftest.py Change-Id: I07c29364ccec12a7aa5f5961a9988e1c57845224 Commit: 77f57087e95c746ea768666358f72d4255d936e2 https://github.com/tribe29/checkmk/commit/77f57087e95c746ea768666358f72d425… Author: Solomon Jacobs <solomon.jacobs(a)tribe29.com> Date: 2022-04-28 (Thu, 28 Apr 2022) Changed paths: M tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kube.py Log Message: ----------- KUBE: use Pydantic factories instead of default_pod Using factories improves maintainability. This is because new fields no longer affect the test (they are automatically populated). We also introduce the AAA pattern. Change-Id: I05a93978bfc039c52a1f023ce2219a96a29f6bb7 Commit: 4edc158b5e14612a57aedec3aa3a8400d188987d https://github.com/tribe29/checkmk/commit/4edc158b5e14612a57aedec3aa3a8400d… Author: Solomon Jacobs <solomon.jacobs(a)tribe29.com> Date: 2022-04-28 (Thu, 28 Apr 2022) Changed paths: M tests/unit/cmk/special_agents/agent_kube/test_agent_kube_api.py R tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kube.py A tests/unit/cmk/special_agents/agent_kubernetes/test_agent_kube_filter_from_namespace.py Log Message: ----------- KUBE: refactor _collect_cpu_resources test We use Pydantic factories instead of a handrolled function. Moreover, we now test in a more uniform fashion and test _collect_memory_resources alongside. Moreover, the test is moved to the same locations as test_collect_workload_resources_from_api_pods and renamed accordingly. Finally, the original test file is renamed to reflect the tests is now contains. CMK-9525 Change-Id: If1c0d2527b5532ba072226044f3a50eb83395dc6 Compare: https://github.com/tribe29/checkmk/compare/b201b3282100...4edc158b5e14