t/perf: add 'GIT_PERF_USE_SCALAR' run option
Add a 'GIT_PERF_USE_SCALAR' environment variable (and corresponding perf config 'useScalar') to register a repository created with any of: * test_perf_fresh_repo * test_perf_default_repo * test_perf_large_repo as a Scalar enlistment. This is intended to allow a developer to test the impact of Scalar on already-defined performance scenarios. Suggested-by: Derrick Stolee <derrickstolee@github.com> Signed-off-by: Victoria Dye <vdye@github.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
e2809233d1
commit
ba1b117eec
@ -95,6 +95,10 @@ You can set the following variables (also in your config.mak):
|
|||||||
Git (e.g., performance of index-pack as the number of threads
|
Git (e.g., performance of index-pack as the number of threads
|
||||||
changes). These can be enabled with GIT_PERF_EXTRA.
|
changes). These can be enabled with GIT_PERF_EXTRA.
|
||||||
|
|
||||||
|
GIT_PERF_USE_SCALAR
|
||||||
|
Boolean indicating whether to register test repo(s) with Scalar
|
||||||
|
before executing tests.
|
||||||
|
|
||||||
You can also pass the options taken by ordinary git tests; the most
|
You can also pass the options taken by ordinary git tests; the most
|
||||||
useful one is:
|
useful one is:
|
||||||
|
|
||||||
|
@ -49,6 +49,9 @@ export TEST_DIRECTORY TRASH_DIRECTORY GIT_BUILD_DIR GIT_TEST_CMP
|
|||||||
MODERN_GIT=$GIT_BUILD_DIR/bin-wrappers/git
|
MODERN_GIT=$GIT_BUILD_DIR/bin-wrappers/git
|
||||||
export MODERN_GIT
|
export MODERN_GIT
|
||||||
|
|
||||||
|
MODERN_SCALAR=$GIT_BUILD_DIR/bin-wrappers/scalar
|
||||||
|
export MODERN_SCALAR
|
||||||
|
|
||||||
perf_results_dir=$TEST_RESULTS_DIR
|
perf_results_dir=$TEST_RESULTS_DIR
|
||||||
test -n "$GIT_PERF_SUBSECTION" && perf_results_dir="$perf_results_dir/$GIT_PERF_SUBSECTION"
|
test -n "$GIT_PERF_SUBSECTION" && perf_results_dir="$perf_results_dir/$GIT_PERF_SUBSECTION"
|
||||||
mkdir -p "$perf_results_dir"
|
mkdir -p "$perf_results_dir"
|
||||||
@ -120,6 +123,10 @@ test_perf_create_repo_from () {
|
|||||||
# status" due to a locked index. Since we have
|
# status" due to a locked index. Since we have
|
||||||
# a copy it's fine to remove the lock.
|
# a copy it's fine to remove the lock.
|
||||||
rm .git/index.lock
|
rm .git/index.lock
|
||||||
|
fi &&
|
||||||
|
if test_bool_env GIT_PERF_USE_SCALAR false
|
||||||
|
then
|
||||||
|
"$MODERN_SCALAR" register
|
||||||
fi
|
fi
|
||||||
) || error "failed to copy repository '$source' to '$repo'"
|
) || error "failed to copy repository '$source' to '$repo'"
|
||||||
}
|
}
|
||||||
@ -130,7 +137,11 @@ test_perf_fresh_repo () {
|
|||||||
"$MODERN_GIT" init -q "$repo" &&
|
"$MODERN_GIT" init -q "$repo" &&
|
||||||
(
|
(
|
||||||
cd "$repo" &&
|
cd "$repo" &&
|
||||||
test_perf_do_repo_symlink_config_
|
test_perf_do_repo_symlink_config_ &&
|
||||||
|
if test_bool_env GIT_PERF_USE_SCALAR false
|
||||||
|
then
|
||||||
|
"$MODERN_SCALAR" register
|
||||||
|
fi
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -171,6 +171,9 @@ run_subsection () {
|
|||||||
get_var_from_env_or_config "GIT_PERF_MAKE_COMMAND" "perf" "makeCommand"
|
get_var_from_env_or_config "GIT_PERF_MAKE_COMMAND" "perf" "makeCommand"
|
||||||
get_var_from_env_or_config "GIT_PERF_MAKE_OPTS" "perf" "makeOpts"
|
get_var_from_env_or_config "GIT_PERF_MAKE_OPTS" "perf" "makeOpts"
|
||||||
|
|
||||||
|
get_var_from_env_or_config "GIT_PERF_USE_SCALAR" "perf" "useScalar" "--bool"
|
||||||
|
export GIT_PERF_USE_SCALAR
|
||||||
|
|
||||||
get_var_from_env_or_config "GIT_PERF_REPO_NAME" "perf" "repoName"
|
get_var_from_env_or_config "GIT_PERF_REPO_NAME" "perf" "repoName"
|
||||||
export GIT_PERF_REPO_NAME
|
export GIT_PERF_REPO_NAME
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user