[checkmk-commits] [Checkmk/checkmk] 29866b: Test equality of trees with empty nodes

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: 29866bbb03f834fad3ee98707ca2ea0667889f85 https://github.com/Checkmk/checkmk/commit/29866bbb03f834fad3ee98707ca2ea066… Author: Simon Jess <simon.jess(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M tests/unit/cmk/utils/structured_data/test_structured_data.py Log Message: ----------- Test equality of trees with empty nodes Change-Id: I1d83a57c6e0c05635582b13cbb969c1c6f162953 Commit: d7c0f27343d45c2a929ade9d48b591a5d0966614 https://github.com/Checkmk/checkmk/commit/d7c0f27343d45c2a929ade9d48b591a5d… Author: Simon Jess <simon.jess(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: A .werks/17134.md M cmk/utils/structured_data.py M tests/unit/cmk/utils/structured_data/test_structured_data.py Log Message: ----------- 17134 FIX HW/SW inventory: Do not archive or save trees with empty nodes Change-Id: I4da68f968ff9f4b35fef87670d3da77796c64d44 Commit: 994e73847673e1bcb95e388c021d1fd4a8f0aef1 https://github.com/Checkmk/checkmk/commit/994e73847673e1bcb95e388c021d1fd4a… Author: Andreas Boesl <andreas.boesl(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M packages/cmk-frontend-vue/src/form/components/forms/FormDictionary.vue Log Message: ----------- vue: fixed margin of dictionary checkbox Change-Id: Ib2f5d82b098a750bd24c7e2d95318d0237334e2e Commit: 0f376c7bd9c33ea4fcce05a63c5118824cbdca1a https://github.com/Checkmk/checkmk/commit/0f376c7bd9c33ea4fcce05a63c5118824… Author: Sergey Kipnis <sergey.kipnis(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M agents/windows/plugins/mssql.vbs Log Message: ----------- agent: remove MS SQL database name trimming - legacy plugin CMK-18125 Change-Id: I67324d2c8cbcf1335a6ae38097547a6d90875814 Commit: 70ffa052494b4bb9c8130a5906edc4e1232df09c https://github.com/Checkmk/checkmk/commit/70ffa052494b4bb9c8130a5906edc4e12… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M packages/cmk-frontend-vue/scripts/test_license_headers.py Log Message: ----------- better ignore mechanism for license header check added a second ignore based on glob. only use suffix based ignore for files that can be anywhere. also: better error message for license header check Change-Id: Ifcd429b93cfcb97525b6053a1515ac5715ebb963 Commit: e18f8a0b686826eec6bdf7251325537a6955d899 https://github.com/Checkmk/checkmk/commit/e18f8a0b686826eec6bdf7251325537a6… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M cmk/gui/htmllib/html.py M omd/packages/check_mk/check_mk.make M packages/cmk-frontend-vue/.f12 M packages/cmk-frontend-vue/src/assets/variables.css A packages/cmk-frontend-vue/src/stage1.ts R packages/cmk-frontend-vue/src/vue_stage1.ts M packages/cmk-frontend-vue/vite.config.ts M tests/unit/cmk/gui/conftest.py Log Message: ----------- ship css for cmk-frontend-vue this is a bigger change and does multiple things at once, but the most important thing is that previously we did not ship the css styles for cmk-frontend-vue. they were visible with the `vite dev` server, but the css was not included in the builds. this is the first time we make use of the manifest.json provided by the vite build. this way we can copy the whole dist folder as is and facilitate the cache busting mechanism provided by vite. the filenames contain hashes, so we don't have to add a query to the urls. biggest problem with this change was the relative external url of the icons: it is resolved relative to the css definition. but this differs between production build and development, as in production it is compiled into a file and during development its inlined into the dom. to solve this, the files are not directly accessed and no longer external, so vite can find them on the disk. this was previously not possible because of the theming mechanism in place, but the variables resolved this problem, if you inline the apache redirection that resolves icons not found in modern-dark with facelift. the second problem was: how to identify the css file and where to put it. in this change we use a new approach to utilize the manifest.json to read the filenames from there. this way we can simply copy the whole dist folder into the production build and read the filenames from there. maybe we have to introduce some performance optimizations, as we now read the manifest.json for each request. also modify the logic when to add vue javascript files slightly: previously we explicitly added vue as a js dependency, now we add the vue javascript files whenever the main javascript is added. CMK-19056 Change-Id: I168af7449db06ee4e679a6f8ab502277a398f8fa Commit: 4bb216920049a26f929099bc83f138f3c3c8d260 https://github.com/Checkmk/checkmk/commit/4bb216920049a26f929099bc83f138f3c… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M cmk/gui/htmllib/html.py Log Message: ----------- cache loading of manifest in production the manifest should only change with site updates, so it is okay to cache the result indefinitely per apache process. Change-Id: If29e8edd1612fadb54aba0615d54af485b11f9ac Commit: 1094f2fd8a51e58b0ca2c4d68b69ea45d3b0c8c1 https://github.com/Checkmk/checkmk/commit/1094f2fd8a51e58b0ca2c4d68b69ea45d… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M packages/cmk-frontend-vue/src/assets/variables.css Log Message: ----------- simplify icon variables no need to speficy icons twice if the reference the same files Change-Id: I7281a4b585c261c20364d6c9ade21b732ea7be7a Commit: 0107ad33602255fcee91608884baf51fa254cc8d https://github.com/Checkmk/checkmk/commit/0107ad33602255fcee91608884baf51fa… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2024-09-23 (Mon, 23 Sep 2024) Changed paths: M packages/cmk-werks/BUILD Log Message: ----------- Remove left-over filegroup ... this should not be needed anymore as we're having the wheel build in bazel Change-Id: I522c6b305b2dc0705ec4a05c9b4548fc3b93380f Compare: https://github.com/Checkmk/checkmk/compare/5f3faa97343d...0107ad336022 To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications