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