From: Taylor Blau Date: Thu, 14 Dec 2023 22:24:36 +0000 (-0500) Subject: pack-objects: add tracing for various packfile metrics X-Git-Tag: v2.44.0-rc0~84^2~4 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=54393e4e68f2b4cf1ebaab8a934e83c9384595af;p=thirdparty%2Fgit.git pack-objects: add tracing for various packfile metrics As part of the multi-pack reuse effort, we will want to add some tests that assert that we reused a certain number of objects from a certain number of packs. We could do this by grepping through the stderr output of `pack-objects`, but doing so would be brittle in case the output format changed. Instead, let's use the trace2 mechanism to log various pieces of information about the generated packfile, which we can then use to compare against desired values. Signed-off-by: Taylor Blau Signed-off-by: Junio C Hamano --- diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c index 7eb035eb7d..7aae9f104b 100644 --- a/builtin/pack-objects.c +++ b/builtin/pack-objects.c @@ -4595,6 +4595,13 @@ int cmd_pack_objects(int argc, const char **argv, const char *prefix) reuse_packfile_objects, (uintmax_t)reuse_packfiles_used_nr); + trace2_data_intmax("pack-objects", the_repository, "written", written); + trace2_data_intmax("pack-objects", the_repository, "written/delta", written_delta); + trace2_data_intmax("pack-objects", the_repository, "reused", reused); + trace2_data_intmax("pack-objects", the_repository, "reused/delta", reused_delta); + trace2_data_intmax("pack-objects", the_repository, "pack-reused", reuse_packfile_objects); + trace2_data_intmax("pack-objects", the_repository, "packs-reused", reuse_packfiles_used_nr); + cleanup: clear_packing_data(&to_pack); list_objects_filter_release(&filter_options);