[checkmk-commits] [tribe29/checkmk] 1b8a8f: Replace DataSource.run() callers

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 1b8a8f1278a922045f8723e1e0ec3ccc5bd1e9b1 https://github.com/tribe29/checkmk/commit/1b8a8f1278a922045f8723e1e0ec3ccc5… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-08-24 (Mon, 24 Aug 2020) Changed paths: M cmk/base/data_sources/_abstract.py M cmk/base/data_sources/_data_sources.py M cmk/base/data_sources/snmp.py M cmk/base/inventory.py M tests/unit/cmk/base/data_sources/test_data_sources.py M tests/unit/cmk/base/data_sources/test_host_sections.py Log Message: ----------- Replace DataSource.run() callers Now, `DS.check(RawData)` and `DS.run()` are duplicates. But `DS.check(RawData)` is more consistent with the new API as it clarifies the distinction between fetching and checking. We therefore replace the calls to `run()` with calls to `fetch() -> RawData` and `check(RawData)`. In a following patch, we can finally remove `run()` and its callees. CMK-5243 Change-Id: I5713c1161ad39c3645a7f62caf5b29d69f70e6cf Commit: 426a8c5b0211015c58e70b9cb1c87bd99c1db6ad https://github.com/tribe29/checkmk/commit/426a8c5b0211015c58e70b9cb1c87bd99… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-08-24 (Mon, 24 Aug 2020) Changed paths: M cmk/base/data_sources/_abstract.py M cmk/base/data_sources/ipmi.py M cmk/base/data_sources/piggyback.py M cmk/base/data_sources/programs.py M cmk/base/data_sources/snmp.py M cmk/base/data_sources/tcp.py Log Message: ----------- Remove obsolete methods - run() - _run() - _execute() are obsoleted by `Fetcher.fetch() -> RawData` and `DataSource.check(RawData)` CMK-5243 Change-Id: I5f819aec228525f7fcf3ff4e404f19babe09961a Compare: https://github.com/tribe29/checkmk/compare/b6160c7428a5...426a8c5b0211