[checkmk-commits] [tribe29/checkmk] f11bfa: Move ABCSNMPBackend to `type_defs`

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: f11bfaaf9b6b03803e579d639d645afb92f2e2de https://github.com/tribe29/checkmk/commit/f11bfaaf9b6b03803e579d639d645afb9… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-05-26 (Tue, 26 May 2020) Changed paths: M cmk/fetchers/factory.py M cmk/fetchers/snmp_backend/__init__.py R cmk/fetchers/snmp_backend/_base.py M cmk/fetchers/snmp_backend/classic.py M cmk/fetchers/snmp_backend/stored_walk.py M cmk/utils/type_defs.py Log Message: ----------- Move ABCSNMPBackend to `type_defs` We should be able to use this in `cmk.utils.snmp_table` in order to break the last (I believe) layering violation introduced by this series of patches. Note that importing `fetchers` in `base` is not a layering violation. The other way around is one. In the end, we should have something like: +---------------------+ | base | +--------+------------+ | | fetchers | | +------------+ | utils | +---------------------+ which is probably ok. CMK-4504 Change-Id: Id78e1d57181ce4aefeb7e59a238200b791c1dac5 Commit: 5f6135655d7106756331707739b2be0168fd3862 https://github.com/tribe29/checkmk/commit/5f6135655d7106756331707739b2be016… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-05-26 (Tue, 26 May 2020) Changed paths: M cmk/base/modes/check_mk.py M cmk/base/snmp.py M cmk/fetchers/factory.py M cmk/utils/snmp_table.py Log Message: ----------- Force stored-walk backend in backends - This completes the decoupling of `utils.snmp_table` and `base.snmp`. - We do not need to explicitly pass the default value anymore. CMK-4504 Change-Id: If9ab68262786f8b2a21b2992e17f73b27502425b Compare: https://github.com/tribe29/checkmk/compare/caf0b8ae9e4f...5f6135655d71