[checkmk-commits] [Checkmk/checkmk] fdd03a: downtimes: add a response schema

Branch: refs/heads/2.2.0 Home: https://github.com/Checkmk/checkmk Commit: fdd03a93edd011b74b9aa8826876533870f94bdc https://github.com/Checkmk/checkmk/commit/fdd03a93edd011b74b9aa882687653387… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-08-07 (Mon, 07 Aug 2023) Changed paths: M cmk/gui/plugins/openapi/endpoints/downtime/__init__.py A cmk/gui/plugins/openapi/endpoints/downtime/response_schemas.py Log Message: ----------- downtimes: add a response schema This werk adds a response schema to the downtime endpoints. This addresses two minor issues. 1. The API documentation now shows the response schema. 2. The domainType being returned has been updated Before: C+: "domainType": "dict" C-: After: C+: "domainType": "downtime" C-: See werk #15966 Change-Id: I721d4f9e0903667819825caf2a7e69018eb3d42f Commit: b618293f9a6a3fdf98fc8f52198c90ca5c734cc9 https://github.com/Checkmk/checkmk/commit/b618293f9a6a3fdf98fc8f52198c90ca5… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-08-07 (Mon, 07 Aug 2023) Changed paths: 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/unit/cmk/gui/plugins/openapi/test_openapi_downtime.py Log Message: ----------- downtimes: client now has to specify the site_id when deleting downtimes Downtime IDs are integers beginning from 1, both for the main site & for remote sites. This means that the same downtime ID can exist on more than one site at the same time. This caused problems when calling the DELETE endpoint as we weren't sure which ID the client wanted to delete. To get around this problem, this werk introduces a mandatory site_id field which the client has to specify when they want to delete a downtime. SUP-14495 See werk #15966 Change-Id: Iadbc80f2d84b55551b650e3db7b11514fe258b79 Commit: 659ea277ef27fc7932c1a985f0b3e5ef7853b4e9 https://github.com/Checkmk/checkmk/commit/659ea277ef27fc7932c1a985f0b3e5ef7… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-08-07 (Mon, 07 Aug 2023) Changed paths: M cmk/gui/plugins/openapi/endpoints/downtime/__init__.py M cmk/utils/livestatus_helpers/queries.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_downtime.py Log Message: ----------- downtimes: A site_id is now required to GET downtimes This werk adds a mandatory site_id field as a query parameter when calling the get downtime endpoint. This is due to the fact that downtime ids by themselves are not unique, but together with the site_id, they are. SUP-14495 See werk #15966 Change-Id: I5fcffaa77a8491efcc53f29524c5b0447d26eaf7 Commit: de48ed9117ae49e5ea76476e34ef3db962f388a2 https://github.com/Checkmk/checkmk/commit/de48ed9117ae49e5ea76476e34ef3db96… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-08-07 (Mon, 07 Aug 2023) Changed paths: M cmk/gui/plugins/openapi/endpoints/downtime/__init__.py M cmk/gui/plugins/openapi/endpoints/downtime/response_schemas.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_downtime.py Log Message: ----------- downtimes: the downtime response now includes the site_id This werk introduces the site_id to the downtime response for the GET endpoints 'show downtime' and 'show all downtimes'. See werk #15966 Change-Id: I29c80011abc7953114ac075d827ae8b5bd1bb899 Commit: a0c0c3539a050f6dd38ea06ac99ac232a1ac4c31 https://github.com/Checkmk/checkmk/commit/a0c0c3539a050f6dd38ea06ac99ac232a… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-08-07 (Mon, 07 Aug 2023) Changed paths: A .werks/15966 M cmk/gui/livestatus_utils/commands/downtimes.py M cmk/gui/plugins/openapi/endpoints/downtime/__init__.py M cmk/utils/livestatus_helpers/queries.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_downtime.py Log Message: ----------- 15966 downtimes: Add an optional site_id query param to the show all downtimes endpoint This change introduces a new optional query parameter on the show all downtimes endpoint. The client can now specify the site_id as a filter, so that they can list all downtimes for a specific site. Change-Id: Id2aea6869d24daa028d9aa5fce28333401805260 Commit: a86d0f85e4b5b243c1b7313fa8fdcb01baa3a8f3 https://github.com/Checkmk/checkmk/commit/a86d0f85e4b5b243c1b7313fa8fdcb01b… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2023-08-07 (Mon, 07 Aug 2023) Changed paths: M tests/testlib/rest_api_client.py M tests/unit/cmk/gui/plugins/openapi/test_openapi_downtime.py Log Message: ----------- downtimes: update tests and test client Change-Id: If687579ca39931cfc685675e4fdedf92711bc15d Compare: https://github.com/Checkmk/checkmk/compare/1ad97369696f...a86d0f85e4b5