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