[checkmk-commits] [Checkmk/checkmk] b8919b: werks: tool for mailing werk changes

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: b8919b63a5866f031c89471fb284a0e61a9ef2f6 https://github.com/Checkmk/checkmk/commit/b8919b63a5866f031c89471fb284a0e61… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: M cmk/utils/werks/__init__.py M cmk/utils/werks/__main__.py A cmk/utils/werks/mail/__init__.py A cmk/utils/werks/mail/templates/mail.txt.jinja2 Log Message: ----------- werks: tool for mailing werk changes The currently used mailing of werk changes had several drawbacks: - It was directly implemented in a git post-commit hook. This means: - hard to debug - hard to test changes - no error reporting in place - easy to forget about it - It was a standalone python script: - not checked in - reimplemented werks parsing - It had a feature that werks are only emailed once: for the version the werk was first committed to our main repo. If this is for example version 2.1.0, no additional mail for version 2.2.0 will be sent. With this commit we make it possible to change to a new mail sending approach: - Instead of sending out the mails as soon as the commits reach the main repo, mails are sent at midnight. - The mail sending script manages states via git notes: if the commit is a werks commit, and a mail has been sent a git note is added. - We now annotate if the werk was deleted/modified or created. - Send out a mail for every werk merged to any active branch. Before only a single mail for each werk id was sent. CMK-14325 Change-Id: I7fe0716589e7dd9aecda7add99eb3b12657dea7e Commit: c2b7161eb9ac7272ec6166d1539c7a9ffa0f1157 https://github.com/Checkmk/checkmk/commit/c2b7161eb9ac7272ec6166d1539c7a9ff… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: M cmk/utils/werks/mail/__init__.py Log Message: ----------- Consolidate getting fullname and address Change-Id: I8e23efc656a6263ba17dab0bc224d7fd311d9efc Commit: 07f0a35013647a4562b6025468edb9509ae2cee4 https://github.com/Checkmk/checkmk/commit/07f0a35013647a4562b6025468edb9509… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: M cmk/utils/werks/mail/__init__.py Log Message: ----------- Factor out building mail address Change-Id: Ied62360451400d772514a7044046aea0be625a4c Commit: 5e4af22faaa62dffcba8d0d5f9adee3a80710fa2 https://github.com/Checkmk/checkmk/commit/5e4af22faaa62dffcba8d0d5f9adee3a8… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: M cmk/utils/werks/mail/__init__.py Log Message: ----------- Adapt mail addresses to prior used logic * We used to sent, e.g. level 3 werks also to level 2 and 1 * further: refactor replacing the default addresses for testing Change-Id: I649022035dbea83fc02059234551f7a3825fddbc Commit: 398b0e39e27862eb39df0696a0b4354cfdcf26a3 https://github.com/Checkmk/checkmk/commit/398b0e39e27862eb39df0696a0b4354cf… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: A buildscripts/scripts/send-werks-to-mailing-lists.groovy Log Message: ----------- Add job for sending werks to mailing list Change-Id: I07cb983e5d2bbcb55444465778d323379dc98144 Commit: 59e48a13f48cf75683954a830e94d8b5b9c09293 https://github.com/Checkmk/checkmk/commit/59e48a13f48cf75683954a830e94d8b5b… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: M buildscripts/scripts/send-werks-to-mailing-lists.groovy Log Message: ----------- Switch to production ref/notes/werk_mail Change-Id: Idc86887c84ed9198e71e14b6a0ddce23dd7c9945 Commit: f5d2fd6724aec4112f24947a9075f8a1d61052dc https://github.com/Checkmk/checkmk/commit/f5d2fd6724aec4112f24947a9075f8a1d… Author: Timotheus Bachinger <timotheus.bachinger(a)checkmk.com> Date: 2023-09-13 (Wed, 13 Sep 2023) Changed paths: M buildscripts/scripts/send-werks-to-mailing-lists.groovy Log Message: ----------- Alter default params if we're timer triggered ... as this is the production mode Change-Id: Ifce6f985149561e8b1cfef58b55be8bc58edd0c1 Compare: https://github.com/Checkmk/checkmk/compare/88779aa92692...f5d2fd6724ae