[checkmk-commits] [tribe29/checkmk] 0359a8: Remove layering violation in utils

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 0359a8f8a8354fdbc5d33a001e087d88bb30a5ed https://github.com/tribe29/checkmk/commit/0359a8f8a8354fdbc5d33a001e087d88b… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-05-28 (Thu, 28 May 2020) Changed paths: M cmk/base/automations/check_mk.py M cmk/base/snmp.py M cmk/base/snmp_scan.py M cmk/fetchers/factory.py M cmk/fetchers/snmp.py M cmk/utils/snmp_table.py M tests-py3/integration/cmk/base/snmp/test_snmp.py M tests-py3/integration/cmk/utils/test_snmp_table.py M tests-py3/unit/cmk/utils/test_snmp_table_unit.py Log Message: ----------- Remove layering violation in utils `cmk.utils.snmp_table` now depends on the SNMPBackend protocol and does not require any imports from `cmk.fetchers` anymore. Other functions are changed to have an API consistent with `cmk.utils.snmp_table`, that is, by taking an SNMPBackend as their last argument. The tests are also modified accordingly. We also get some clarification by using actual SNMPBackends instead of magical strings. With this change we do not have layering violations relating to the fetchers in `cmk.utils` or `cmk.fetchers` anymore. CMK-4504 Change-Id: I40ae5ac7c35b60d5d936b8814b40ce2714abdff2 Commit: 1c5e1043f59bf3b2360db02c9de970d78007dd7c https://github.com/tribe29/checkmk/commit/1c5e1043f59bf3b2360db02c9de970d78… Author: Lars Michelsen <lm(a)tribe29.com> Date: 2020-05-28 (Thu, 28 May 2020) Changed paths: M cmk/gui/wato/pages/rulesets.py Log Message: ----------- Fix reimport Change-Id: I2783b827d965f89f9cfaea00f3b94ac70711d17a Compare: https://github.com/tribe29/checkmk/compare/771d4030cf51...1c5e1043f59b