{"thread":{"id":"48407","subject":"[PATCH v2] perf/bisect_run_script: disable codespeed","startedAt":"2018-05-04T12:37:01Z","lastAt":"2018-05-04T12:37:01Z","messageCount":1,"participants":["Christian Couder"],"isPatch":true,"patchVersion":2,"patchTotal":null},"messages":[{"id":"346645","messageId":"20180504123636.13895-1-chriscool@tuxfamily.org","threadId":"48407","inReplyTo":null,"subject":"[PATCH v2] perf/bisect_run_script: disable codespeed","fromName":"Christian Couder","fromEmail":"christian.couder@gmail.com","sentAt":"2018-05-04T12:36:36Z","receivedAt":"2018-05-04T12:37:01Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"When bisecting a performance regression using a config file,\n`./bisect_regression --config my_perf.conf` for example, the\nconfig file can contain Codespeed configuration which would\ninstruct the 'aggregate.perl' script called by the 'run'\nscript to output results in the Codespeed format and maybe\nto try to send this output to a Codespeed server.\n\nThis is unfortunate because the 'bisect_run_script' relies\non the regular output from 'aggregate.perl' to mesure\nperformance, so let's disable Codespeed output and sending\nresults to a Codespeed server.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\n---\nPrevious version was:\n\nhttps://public-inbox.org/git/20180427135135.22931-1-chriscool@tuxfamily.org/\n\nThe only change compared to this previous version is a typo\nfix in the commit message:\n\ns/This in unfortunate/This is unfortunate/\n\n t/perf/bisect_run_script | 6 ++++++\n 1 file changed, 6 insertions(+)\n\ndiff --git a/t/perf/bisect_run_script b/t/perf/bisect_run_script\nindex 038255df4b..3ebaf15521 100755\n--- a/t/perf/bisect_run_script\n+++ b/t/perf/bisect_run_script\n@@ -24,6 +24,12 @@ result_file=\"$info_dir/perf_${script_number}_${bisect_head}_results.txt\"\n GIT_PERF_DIRS_OR_REVS=\"$bisect_head\"\n export GIT_PERF_DIRS_OR_REVS\n \n+# Don't use codespeed\n+GIT_PERF_CODESPEED_OUTPUT=\n+GIT_PERF_SEND_TO_CODESPEED=\n+export GIT_PERF_CODESPEED_OUTPUT\n+export GIT_PERF_SEND_TO_CODESPEED\n+\n ./run \"$script\" >\"$result_file\" 2>&1 || die \"Failed to run perf test '$script'\"\n \n rtime=$(sed -n \"s/^$script_number\\.$test_number:.*\\([0-9]\\+\\.[0-9]\\+\\)(.*).*\\$/\\1/p\" \"$result_file\")\n-- \n2.17.0.400.gd2f70daabc\n\n"}]}