diff options
| author | Klaus Aehlig <klaus.aehlig@huawei.com> | 2025-06-03 12:48:43 +0200 |
|---|---|---|
| committer | Klaus Aehlig <klaus.aehlig@huawei.com> | 2025-06-03 18:41:23 +0200 |
| commit | d5faa5f51f0aa82fba17513528ea9725cd5e471a (patch) | |
| tree | 58ae211329fd11898fee6f2bb98eea5fe563602f /doc | |
| parent | 05eb0ab7cdede9cbac061c1152bf15fd220efd52 (diff) | |
| download | justbuild-d5faa5f51f0aa82fba17513528ea9725cd5e471a.tar.gz | |
Invocation server: also show counts in headings
Often, the simple number, e.g., for failed actions can already give
valuable information. For example, when investigating flakiness,
the count of failed actions already gives a hint whether a change
increased or decreased flakiness which can be valuable before even
investigating if the nature of the failure has changed. As we have
that information available and an additional number in a heading
does not clutter the page too much, let's just show it.
Diffstat (limited to 'doc')
| -rwxr-xr-x | doc/invocations-http-server/server.py | 3 | ||||
| -rw-r--r-- | doc/invocations-http-server/templates/invocation.html | 6 |
2 files changed, 6 insertions, 3 deletions
diff --git a/doc/invocations-http-server/server.py b/doc/invocations-http-server/server.py index 3ac1043f..f9f57aff 100755 --- a/doc/invocations-http-server/server.py +++ b/doc/invocations-http-server/server.py @@ -519,6 +519,7 @@ class InvocationServer: else: failed_build_actions.append(action_data(k, v, graph)) params["failed_actions"] = failed_build_actions + failed_test_actions + params["failed_actions_count"] = len(params["failed_actions"]) # non-failed actions with output output_actions = [] @@ -528,6 +529,7 @@ class InvocationServer: actions_considered.add(k) output_actions.append(action_data(k, v, graph)) params["output_actions"] = output_actions + params["output_actions_count"] = len(output_actions) # longest running non-cached non-failed actions without output candidates = [] @@ -554,6 +556,7 @@ class InvocationServer: action["name_prefix"] = "%5.1fs" % (t,) non_cached.append(action) params["non_cached"] = non_cached + params["non_cached_count"] = len(non_cached) start_time = profile.get("start time") build_start_time = profile.get("build start time") build_stop_time = profile.get("build stop time") diff --git a/doc/invocations-http-server/templates/invocation.html b/doc/invocations-http-server/templates/invocation.html index 957b59d7..bd5915b1 100644 --- a/doc/invocations-http-server/templates/invocation.html +++ b/doc/invocations-http-server/templates/invocation.html @@ -131,7 +131,7 @@ Invocation {{invocation | e}} <li> Cached: <tt>{{ action_count_cached | e }}</tt></li> </ul> -<h3>Failed actions</h3> +<h3> Failed actions ({{ failed_actions_count | e }})</h3> {% if failed_actions %} <ul> {% for action in failed_actions %} @@ -142,7 +142,7 @@ Invocation {{invocation | e}} <i>(none)</i> {% endif %} -<h3>Other actions with console output</h3> +<h3>Other actions with console output ({{ output_actions_count | e }})</h3> {% if output_actions %} <ul> {% for action in output_actions %} @@ -154,7 +154,7 @@ Invocation {{invocation | e}} {% endif %} -<h3>Remaining non-cached actions</h3> +<h3>Remaining non-cached actions ({{ non_cached_count | e}})</h3> {% if non_cached %} In order of decreasing run time. <ul> |