[checkmk-commits] [Checkmk/checkmk] 122766: Hack to avoid funny GCC warnings for our old proto...

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: 12276682f8c8bb6a0fea7082ee67c17d66669ea7 https://github.com/Checkmk/checkmk/commit/12276682f8c8bb6a0fea7082ee67c17d6… Author: Sven Panne <sven.panne(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: M omd/packages/protobuf/protobuf.make Log Message: ----------- Hack to avoid funny GCC warnings for our old protobuf version. Change-Id: I2a4fd96f16fd2258a0af752dc54697829e63c318 Commit: 96f4066b833f0f7c4a81a7dbc69a3a5db7f07591 https://github.com/Checkmk/checkmk/commit/96f4066b833f0f7c4a81a7dbc69a3a5db… Author: Rebekka Seltmann <rebekka.seltmann(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: A .werks/15972 M cmk/gui/wato/pages/activate_changes.py M cmk/gui/wato/pages/user_profile/async_replication.py M cmk/gui/watolib/activate_changes.py M tests/unit/cmk/gui/conftest.py M tests/unit/cmk/gui/watolib/test_activate_changes.py M tests/unit/cmk/gui/watolib/test_config_sync.py Log Message: ----------- Activate Changes Redesign: Use threads instead of processes Since the memory of the parent process is copied when forking a new process, the number of processes created to simultaneously to activate changes per site is limited. With threads more sites can be activated in parallel. CMK-13856 Change-Id: I51dbdc77af4b977b515b8f0368ccb275f558823a Commit: 8fe417b2b0a8aa6d5b0c1c72fc89876cbcbe58dd https://github.com/Checkmk/checkmk/commit/8fe417b2b0a8aa6d5b0c1c72fc89876cb… Author: Rebekka Seltmann <rebekka.seltmann(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: M cmk/gui/watolib/activate_changes.py M tests/unit/cmk/gui/watolib/test_config_sync.py Log Message: ----------- Activate Changes Redesign: Precompute sync infos Since the config directories/files to be synced to the remote site are copied (as hard links), the sync info can be precomputed and reused for the copied files The implementation to iterate over these files and open them was also changed from pathlib to os.path since pathlib is slower CMK-13856 Change-Id: I56aaac7855bdfc7c3af66964a3314f513191b50d Commit: 486c90c19f02dc55ecccaad472d604722f12994e https://github.com/Checkmk/checkmk/commit/486c90c19f02dc55ecccaad472d604722… Author: Rebekka Seltmann <rebekka.seltmann(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: M cmk/gui/watolib/activate_changes.py M tests/unit/cmk/gui/watolib/test_config_sync.py Log Message: ----------- Activate Changes redesign: Do not wait for whole step to finish If one site takes longer to e.g. activate the remote changes, other sites should not be blocked in continuing their work. We now maintain a list of active tasks that are periodically asked if they have finished in order to move on to the next step as soon as possible CMK-13959 Change-Id: Ie43df6f09b5e0c6da2991df133eae63aaeca2288 Commit: 4467763f4c959b5c696dbebee9e913fea9b4db51 https://github.com/Checkmk/checkmk/commit/4467763f4c959b5c696dbebee9e913fea… Author: Rebekka Seltmann <rebekka.seltmann(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: M cmk/gui/watolib/activate_changes.py Log Message: ----------- Activate Changes: Improve inode lookup table creation Previously the sync file infos were pre-calculated on one the snapshot directories. However, due e.g. customer hooks, this might not contain all the files to be synced anymore. Look at all replication paths instead. CMK-14182 Change-Id: I847fa61fa4a98408e95a2bd5789a973674526307 Commit: 5cee59c112ced52eb518a6264124ca34b645a36e https://github.com/Checkmk/checkmk/commit/5cee59c112ced52eb518a6264124ca34b… Author: Rebekka Seltmann <rebekka.seltmann(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: M .werks/15972 M cmk/gui/plugins/config/base.py M cmk/gui/plugins/wato/check_mk_configuration.py M tests/unit/cmk/gui/test_gui_config.py M tests/unit/cmk/gui/watolib/test_watolib.py Log Message: ----------- Activate Changes Redesign: remove superfluous limit configuration With threads being used instead of processes, each activate uses much less memory. Because of that, the config variable for "Maximum parallel site activations" to limit the number of parallel activations has been removed CMK-13856 Change-Id: I476d16c21298a08263fcbf6626d16ee3871fb45d Commit: 2e7a6b0fb374778184ad68ec328c08b5c1dd066a https://github.com/Checkmk/checkmk/commit/2e7a6b0fb374778184ad68ec328c08b5c… Author: Rebekka Seltmann <rebekka.seltmann(a)checkmk.com> Date: 2023-08-18 (Fri, 18 Aug 2023) Changed paths: M cmk/gui/watolib/activate_changes.py Log Message: ----------- Activate Changes Redesign: Parallelize config directory cloning Although limited by the disk, the cp -al operation setting up the configuration directories to be copied to remote site can benefit from parallelization up to a certain degree CMK-14084 Change-Id: If03cf90a4400e92a331337b630b4c56786e2b5a4 Compare: https://github.com/Checkmk/checkmk/compare/ce95ecb0554f...2e7a6b0fb374