Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix perf_speedy report (escaping ")
Browse files Browse the repository at this point in the history
The report currently chokes on quotes in the commit message (see
550aa48). Rather than trying to
correctly excape things in Bash, this PR delegates the quote handling to
jq, because having to deal with Bash embedded in YAML is hard enough.

CHANGELOG_BEGIN
CHANGELOG_END
garyverhaegen-da committed Jan 27, 2021
1 parent fef712b commit 90028bb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ci/cron/daily-compat.yml
Original file line number Diff line number Diff line change
@@ -116,7 +116,7 @@ jobs:
GCRED: $(GOOGLE_APPLICATION_CREDENTIALS_CONTENT)
- template: ../daily_tell_slack.yml
parameters:
success-message: '$(cat $(Build.StagingDirectory)/perf-results.json | jq . | jq -sR ''"perf for ''"$COMMIT_LINK"'':```\(.)```"'')'
success-message: $(jq --arg stats "$(cat $(Build.StagingDirectory)/perf-results.json)" --arg link "$COMMIT_LINK" -n '"perf for " + $link + ":```" + $stats + "```"')

- job: perf_http_json
timeoutInMinutes: 120

0 comments on commit 90028bb

Please sign in to comment.