We'll the connectivity check logic for git-receive-pack(1) in the
following commits to make it perform better. As a preparatory step, add
some benchmarks such that we can measure these changes.
Signed-off-by: Patrick Steinhardt <ps@xxxxxx>
---
t/perf/p5400-receive-pack.sh | 97 ++++++++++++++++++++++++++++++++++++
1 file changed, 97 insertions(+)
create mode 100755 t/perf/p5400-receive-pack.sh
diff --git a/t/perf/p5400-receive-pack.sh b/t/perf/p5400-receive-pack.sh
new file mode 100755
index 0000000000..a945e014a3
--- /dev/null
+++ b/t/perf/p5400-receive-pack.sh
@@ -0,0 +1,97 @@
+#!/bin/sh
+
+test_description="Tests performance of receive-pack"
+
+. ./perf-lib.sh
+
+test_perf_large_repo
+
+test_expect_success 'setup' '
+ # Create a main branch such that we do not have to rely on any specific
+ # branch to exist in the perf repository.
+ git switch --force-create main &&
+
+ # Set up a pre-receive hook such that no refs will ever be changed.
+ # This easily allows multiple perf runs, but still exercises
+ # server-side reference negotiation and checking for consistency.
+ mkdir hooks &&
+ write_script hooks/pre-receive <<-EOF &&
+ #!/bin/sh
+ echo "failed in pre-receive hook"
+ exit 1
+ EOF
+ cat >config <<-EOF &&
+ [core]
+ hooksPath=$(pwd)/hooks
+ EOF
+ GIT_CONFIG_GLOBAL="$(pwd)/config" &&
+ export GIT_CONFIG_GLOBAL &&
+
+ git switch --create updated &&
+ test_commit --no-tag updated
+'
+
+setup_empty() {
+ git init --bare "$2"
+}
+
+setup_clone() {
+ git clone --bare --no-local --branch main "$1" "$2"
+}
+
+setup_clone_bitmap() {
+ git clone --bare --no-local --branch main "$1" "$2" &&
+ git -C "$2" repack -Adb
+}
+
+# Create a reference for each commit in the target repository with extra-refs.
+# While this may be an atypical setup, biggish repositories easily end up with
+# hundreds of thousands of refs, and this is a good enough approximation.
+setup_extrarefs() {
+ git clone --bare --no-local --branch main "$1" "$2" &&
+ git -C "$2" log --all --format="tformat:create refs/commit/%h %H" |
+ git -C "$2" update-ref --stdin
+}
+
+# Create a reference for each commit in the target repository with extra-refs.
+# While this may be an atypical setup, biggish repositories easily end up with
+# hundreds of thousands of refs, and this is a good enough approximation.
+setup_extrarefs_bitmap() {
+ git clone --bare --no-local --branch main "$1" "$2" &&
+ git -C "$2" log --all --format="tformat:create refs/commit/%h %H" |
+ git -C "$2" update-ref --stdin &&
+ git -C "$2" repack -Adb
+}
+
+for repo in empty clone clone_bitmap extrarefs extrarefs_bitmap
+do
+ test_expect_success "$repo setup" '
+ rm -rf target.git &&
+ setup_$repo "$(pwd)" target.git
+ '
+
+ # If the target repository is the empty one, then the only thing we can
+ # do is to create a new branch.
+ case "$repo" in
+ empty)
+ refspecs="updated:new";;
+ *)
+ refspecs="updated:new updated:main main~10:main :main";;
+ esac
+
+ for refspec in $refspecs
+ do
+ test_expect_success "$repo seed $refspec" "
+ test_must_fail git push --force target.git '$refspec' \
+ --receive-pack='tee pack | git receive-pack' 2>err &&
+ grep 'failed in pre-receive hook' err
+ "
+
+ test_perf "$repo receive-pack $refspec" "
+ git receive-pack target.git <pack >negotiation &&
+ grep 'pre-receive hook declined' negotiation
+ "
+ done
+done
+
+test_done
--
2.32.0
Attachment:
signature.asc
Description: PGP signature
