[checkmk-commits] [Checkmk/checkmk] 7d68fc: 16156 FIX downtimes: site_id only required when de...

Branch: refs/heads/2.2.0 Home: https://github.com/Checkmk/checkmk Commit: 7d68fc42c4bf1b8292a76e260c9a909ea14d2373 https://github.com/Checkmk/checkmk/commit/7d68fc42c4bf1b8292a76e260c9a909ea… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-10-23 (Mon, 23 Oct 2023) Changed paths: A .werks/16156 M cmk/gui/livestatus_utils/commands/downtimes.py M cmk/gui/plugins/openapi/endpoints/downtime/__init__.py M cmk/gui/plugins/openapi/endpoints/downtime/request_schemas.py M tests/testlib/rest_api_client.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_downtime.py Log Message: ----------- 16156 FIX downtimes: site_id only required when deleting downtimes by_id We recently introduced a change with werk 15966 which means that the site_id is now required when deleting downtimes via the REST API. However, only the 'by_id' option leaves the downtime selection ambigious, while the options 'params' or 'query' already have the ability to pinpoint the downtimes requested, therefore, the site_id should not be required in these cases. This werk addresses this oversight and introduces a change where the site_id is only required when deleteing by_id. Change-Id: If1c72ad254d37d5fd9322065cffe179d31a05891