Branch: refs/heads/master
Home:
https://github.com/Checkmk/checkmk
Commit: 1982221a9237e4b5156194af7b04f8c62d7bcfed
https://github.com/Checkmk/checkmk/commit/1982221a9237e4b5156194af7b04f8c62…
Author: Frans Fürst <frans.fuerst(a)checkmk.com>
Date: 2024-07-29 (Mon, 29 Jul 2024)
Changed paths:
M tests/code_quality/test_pipfile.py
Log Message:
-----------
fix Pipfile linter issue
Change-Id: Ia591d641ab3c5abcfb89f211cae6b9f412b962b5
Commit: 540c784f7d31e769e71a5cc7723b43a114d70b5a
https://github.com/Checkmk/checkmk/commit/540c784f7d31e769e71a5cc7723b43a11…
Author: Niko Wenselowski <niko.wenselowski(a)checkmk.com>
Date: 2024-07-29 (Mon, 29 Jul 2024)
Changed paths:
M buildscripts/scripts/bazel_execution_log_parser.py
Log Message:
-----------
Refactor ExecutionMetrics class
The constructor of the previous approach did have two bugs, which are
fixed by this.
In the process of this the class has been transformed into a NamedTuple,
which should decrease the required memory for this a bit.
First setting the value of `bool` to an attribute led to this value
always being seen as `True` on a truthiness-check.
This could have led to reporting higher values for cache hits, cacheable
and remotable entries from the log.
These fields are now set `False` by default, which should report more
reliable results.
Second the check for the presence of `targetLabel` did not work as intended,
because it checked for the key being part of the string `"targetLabel"`
instead of checking if the key was the string `"targetLabel"`.
The check for an existing target label is now fixed.
Change-Id: I0b14f9018bb9800bbefb16b2a6cc31a97301da38
Commit: 925d9cbfaeec4664588dd2e38fb49f67be4e9248
https://github.com/Checkmk/checkmk/commit/925d9cbfaeec4664588dd2e38fb49f67b…
Author: Niko Wenselowski <niko.wenselowski(a)checkmk.com>
Date: 2024-07-29 (Mon, 29 Jul 2024)
Changed paths:
M buildscripts/scripts/bazel_execution_log_parser.py
Log Message:
-----------
Only display a target missing the cache once
This avoids listing a target multiple times in the output, if it
has been reported missing the cache in different logs.
Change-Id: If6a84498b57a15730e58f59eef31ac7c4d740b38
Compare:
https://github.com/Checkmk/checkmk/compare/052cd58b806c...925d9cbfaeec
To unsubscribe from these emails, change your notification settings at
https://github.com/Checkmk/checkmk/settings/notifications